RFR: 8167178 Exported elements referring to inaccessible types in java.naming

[Roger Riggs]

Hi Vyom,

Please also correct or remove the comment saying it should be treated as 
read-only by subclasses.

Roger


On 1/16/2017 4:56 AM, Chris Hegarty wrote:
> Looks good. Thanks Vyom.
>
> -Chris.
>
>> On 16 Jan 2017, at 09:10, Vyom Tewari <vyom.tewari at 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.
>>