Review Request JDK-8172808: Handle sun.security.util.Resources bundle in ResourcesMgr in the same way as AuthResources

Sean Mullan sean.mullan at oracle.com
Wed Jan 25 16:50:10 UTC 2017


Looks good.

--Sean

On 1/25/17 11:37 AM, Mandy Chung wrote:
> http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8172808/webrev.01/
>
> This includes a simple patch from Adam to fixup the copyright headers in the fix for JDK-8168075.
>
> Mandy
>
>> On Jan 24, 2017, at 2:25 PM, Mandy Chung <mandy.chung at oracle.com> wrote:
>>
>> Adam, Sean,
>>
>> Can you review this simple fix:
>>  http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8172808/webrev.00/
>>
>> This change updates ResourcesMgr to handle both Resources
>> and AuthResources consistently.  This removes doPrivileged
>> block because it is not really needed.  This is covered by
>> an existing test, jdk/test/sun/security/util/Resources/Format.java.
>>
>> I ran all core tests to verify as well.
>>
>> Mandy
>>
>



More information about the security-dev mailing list