Re: Review request for 6895456: Eliminate dependency on java.io.ObjectStreamClass during boot

2009-10-27 Thread Alan Bateman
Mandy Chung wrote: Fixed 6895456: Eliminate dependency on java.io.ObjectStreamClass during boot Webrev at: http://cr.openjdk.java.net/~mchung/6895456/webrev.00/ java.io.ObjectStreamClass is not needed if not doing serialization. Move getSignatureClass static method to

Re: Review request for 6895456: Eliminate dependency on java.io.ObjectStreamClass during boot

2009-10-27 Thread Mandy Chung
Alan Bateman wrote: I agree it would be good to avoid loading/initializing ObjectStreamClass (as you say, we have to load ObjectStreamField as the serialPersistentFields is part of Class, String, and other APIs). My only concern is that ObjectStreamField doesn't seem to be the right place

Re: Review request for 6895456: Eliminate dependency on java.io.ObjectStreamClass during boot

2009-10-27 Thread Mandy Chung
Duplicating this method may be a better option for this case (that I typically try to avoid). I can make this change. The new webrev at: http://cr.openjdk.java.net/~mchung/6895456/webrev.01/ Mandy

Review request for 6895456: Eliminate dependency on java.io.ObjectStreamClass during boot

2009-10-26 Thread Mandy Chung
Fixed 6895456: Eliminate dependency on java.io.ObjectStreamClass during boot Webrev at: http://cr.openjdk.java.net/~mchung/6895456/webrev.00/ java.io.ObjectStreamClass is not needed if not doing serialization. Move getSignatureClass static method to java.io.ObjectStreamField class.