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

Jiangli Zhou jiangli.zhou at oracle.com
Tue Sep 2 17:02:07 UTC 2014


Hi Ioi,

The changes look ok to me.

Thanks,
Jiangli


On 08/31/2014 11: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