7030249: Eliminate use of LoadLibrary and other clean-ups

Kelly O'Hair kelly.ohair at oracle.com
Tue Mar 29 15:44:43 UTC 2011


On Mar 29, 2011, at 6:48 AM, Alan Bateman wrote:

> Kelly O'Hair wrote:
>> I'll admit that I'm not following this C code horribly well, but overall it does look much cleaner.
>> As well as I can follow it, it looks good.
>> 
>> In the Makefile, could we get some comments as to why the lib's are needed and also some
>> comments as to what the delayload means and why it is important? I could guess, but I'd prefer
>> some makefile comments to help the next poor soul looking at it. ;^)
>>  
> Thanks Kelly. I'll add a comment to make this clearer athough this isn't really new as there are other places in the build where /delayload is used too.

Yes, I realize that. But in general, some of the past build issues were created because we have done a poor job
of documenting what the makefiles are doing, so I'm just trying to encourage more makefile comments from
now on. Sometimes the reasons for doing things can help determine a better future decision on removal or change.
Just a sentence on why the delayload is needed here for this library, not what the delayload feature is.
Or maybe a simple sentence on why these libraries are needed for this library and not all libraries.

-kto

> 
> -Alan.




More information about the core-libs-dev mailing list