RFR (XS) 8056971 - Minor class loading clean-up

David Holmes david.holmes at oracle.com
Sun Aug 31 22:34:47 UTC 2014


Looks okay.

src/share/vm/classfile/sharedPathsMiscInfo.cpp seems to have an empty 
set of changes

Thanks,
David


On 1/09/2014 4:55 AM, Ioi Lam wrote:
> Please review a very small fix:
>
> http://cr.openjdk.java.net/~iklam/8056971_classloader_cleanup/
>
> Bug: Minor class loading clean-up
>
>      https://bugs.openjdk.java.net/browse/JDK-8056971
>
> Summary of fix:
>
>      Code review after the push of JDK-8046070 (Class Data Sharing
> cleanup and refactoring)
>      has found some minor issues:
>
>      + Misplacement of #if INCLUE_CDS
>      + typos
>      + unnecessary C string duplication
>        -> in fixing this I converted a bunch of unnecessary
>           "char*" to "const char*"
>
> Tests:
>
>      JPRT
>      UTE vm.quick.testlist, vm.parallel_class_loading.testlist
>
> Thanks
> - Ioi


More information about the hotspot-runtime-dev mailing list