Looks good. Thanks Vyom. -Chris.
> On 16 Jan 2017, at 09:10, Vyom Tewari <vyom.tew...@oracle.com> wrote: > > Hi All, > > Please review below the small fix for the issue. > > BugId : https://bugs.openjdk.java.net/browse/JDK-8167178 > > The compatibility impact is minimum as no code in JDK is currently depend on > it. > > I will file a CCC today. > > Thanks, > > Vyom > > > --- a/src/java.naming/share/classes/javax/naming/CompoundName.java Fri Dec 23 > 09:31:24 2016 +0530 > +++ b/src/java.naming/share/classes/javax/naming/CompoundName.java Mon Jan 16 > 13:36:48 2017 +0530 > @@ -153,7 +153,7 @@ > * This field is initialized by the constructors and cannot be null. > * It should be treated as a read-only variable by subclasses. > */ > - protected transient NameImpl impl; > + private transient NameImpl impl; > /** > * Syntax properties for this compound name. > * This field is initialized by the constructors and cannot be null. >
