RFR: JDK-8162340: Better class stream parsing

John Rose john.r.rose at oracle.com
Thu Jul 21 22:16:17 UTC 2016


Excellent.  Thanks for chasing down all the details.

The cleanups are welcome, and the principled restriction
on loading into the "java/" packages will help us sleep at night.

Reviewed.
— John

On Jul 21, 2016, at 10:23 AM, Karen Kinnear <karen.kinnear at oracle.com> wrote:
> 
> Please review:
> https://bugs.openjdk.java.net/browse/JDK-8162340
> 
> http://cr.openjdk.java.net/~acorn/8162340.hs/webrev/
> http://cr.openjdk.java.net/~acorn/8162340.jdk.test/webrev
> 
> When moving the check from resolve_from_stream to set_package, I removed
> the no longer need parsed_class (!) which I should have cleaned up when
> tightening the placeholder table cleanup years ago.
> 
> Tests run:
> 1. updated VMAnonymousClass test
> 2. jcks
> 3. rbt hs-nightly-runtime linux-x64 in progress
> 
> thanks,
> Karen
> 
> 



More information about the hotspot-runtime-dev mailing list