[PATCH] profile support
Mon Aug 29 21:12:00 GMT 2011
On 2011-08-30 AM 5:23, Christopher Faylor wrote:
>> +#if !defined(PROFILE)
>> void *rv = malloc(size);
>> + void *rv = LocalAlloc(LMEM_FIXED,size);
> Since the code is in gmon.c then I don't see a reason to #ifdef it but I
> still don't understand the motivation for the change. This is a major
> amount of code and it is desperately missing comments.
> Maybe Corinna will disagree but I think there is way too much code
> change here for me to be comfortable with including it. It looks like
> it would be an ongoing maintenance issue, requiring constant vigilance
> to avoid code rot. And, it would have to be very carefully studied to
> make sure there aren't more gotchas like 'if "" ""'.
> Sorry, but I don't see us including this.
Yes, I see. but there would be months for the missing comments, and i
have not enough time to test this patch for now.
and there were bugs in previous patch. so i attach the revised one.
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
More information about the Cygwin-patches