Updated review request for CR 8048362 Test doPrivileged with accomplice

raghu k.nair raghu.k.nair at oracle.com
Wed Aug 20 17:10:14 UTC 2014


Hello,
  Please review the updated webrev : 
http://cr.openjdk.java.net/~tyan/raghu/8048362/webrev.04/ 
<http://cr.openjdk.java.net/%7Etyan/raghu/8048362/webrev.04/>
  I have updated the path of JavaToolUtils in DoPrivAccompliceTest.java.

Thanks,
Raghu
On 8/12/2014 11:06 PM, raghu k.nair wrote:
> Hello ,
>  The latest webrev is available at 
> http://cr.openjdk.java.net/~rhalade/8048362/webrev.03/ 
> <http://cr.openjdk.java.net/%7Erhalade/8048362/webrev.03/> The changes 
> include review comments from Vinnie .
>
> Thanks,
> Raghu
> On 8/8/2014 7:21 PM, Vincent Ryan wrote:
>> Code looks fine.
>> I noticed a few typos: DoPrivTest.java l.26 and 
>> DoPrivAccompliceTest.java l.34-38
>>
>>
>> On 4 Aug 2014, at 08:11, raghu k.nair <raghu.k.nair at oracle.com> wrote:
>>
>>> Hello ,
>>> Please review the updated tests for doPrivilaged with accomplice. 
>>> Updated the test to use @compile tag rather than compiling using API.
>>>
>>> Bug -https://bugs.openjdk.java.net/browse/JDK-8048362
>>> webrev - http://cr.openjdk.java.net/~rhalade/8048362/webrev.01/ 
>>> <http://cr.openjdk.java.net/%7Erhalade/8048362/webrev.01/>
>>>
>>> Thanks,
>>> Raghu
>>>
>




More information about the security-dev mailing list