Re: RFR: JDK-8211121: Remove sun.reflect.ReflectionFactory::newInstanceForSerialization

2018-09-26 Thread mandy chung
On 9/26/18 10:01 AM, Lance Andersen wrote: On Sep 26, 2018, at 1:19 AM, mandy chung > wrote: The imports statement of jdk.internal.misc.SharedSecrets and JavaSecurityAccess can be removed. Thank you Mandy,  thought I clicked to remove unused imports in

Re: RFR: JDK-8211121: Remove sun.reflect.ReflectionFactory::newInstanceForSerialization

2018-09-26 Thread Lance Andersen
> On Sep 26, 2018, at 1:19 AM, mandy chung wrote: > > The imports statement of jdk.internal.misc.SharedSecrets and > JavaSecurityAccess can be removed. Thank you Mandy, thought I clicked to remove unused imports in netbeans but guess I did not. Here is the revised diff $ hg diff

Re: RFR: JDK-8211121: Remove sun.reflect.ReflectionFactory::newInstanceForSerialization

2018-09-26 Thread Alan Bateman
On 26/09/2018 00:10, Lance Andersen wrote: Hi all: JDK-8211121 removes sun.reflect.ReflectionFactory::newInstanceForSerialization which was only used by the java.corba module. It was missed as part of the initial removal of the Java EE modules. Way back, this was in the CORBA bridge. I

Re: RFR: JDK-8211121: Remove sun.reflect.ReflectionFactory::newInstanceForSerialization

2018-09-25 Thread mandy chung
The imports statement of jdk.internal.misc.SharedSecrets and JavaSecurityAccess can be removed. Otherwise looks fine. Mandy On 9/25/18 4:10 PM, Lance Andersen wrote: Hi all: JDK-8211121 removes sun.reflect.ReflectionFactory::newInstanceForSerialization which was only used by the java.corba

Re: RFR: JDK-8211121: Remove sun.reflect.ReflectionFactory::newInstanceForSerialization

2018-09-25 Thread Joseph D. Darcy
Hi Lance, Looks fine; thanks, -Joe On 9/25/2018 4:10 PM, Lance Andersen wrote: Hi all: JDK-8211121 removes sun.reflect.ReflectionFactory::newInstanceForSerialization which was only used by the java.corba module. It was missed as part of the initial removal of the Java EE modules. The diff

RFR: JDK-8211121: Remove sun.reflect.ReflectionFactory::newInstanceForSerialization

2018-09-25 Thread Lance Andersen
Hi all: JDK-8211121 removes sun.reflect.ReflectionFactory::newInstanceForSerialization which was only used by the java.corba module. It was missed as part of the initial removal of the Java EE modules. The diff for the change: — s$ hg diff