RFR: JDK-8069355 Registry permission check refactoring

Sergey Nazarkin sergey.nazarkin at oracle.com
Fri Jan 23 09:20:41 UTC 2015


Thank you,  Jen. I've changed to static function access.

/Sergey

On 23.01.2015 1:29, Jen Dority wrote:
> Hi Sergey,
>
> In RegistryImpl.java, the static checkPermission() method is accessed 
> once through "Registry." and once through "super." I don't see a 
> reason for the difference. If there isn't one, it would be clearer to 
> access it in a consistent manner.
>
> Otherwise, changes look good. No need for another review if you decide 
> to address the "super." issue.
>
> Jen
>
> On 1/22/2015 10:03 AM, Sergey Nazarkin wrote:
>> reminder
>>
>> /Sergey
>>
>> On 21.01.2015 13:17, Sergey Nazarkin wrote:
>>> issue: https://bugs.openjdk.java.net/browse/JDK-8069355
>>> webrev: http://cr.openjdk.java.net/~snazarki/8069355/
>>>
>>> Simplify and refactor Registry permission verification
>>>
>>
>



More information about the dio-dev mailing list