Re: RfR JDK-8056925 Add jaccessinspector and jaccesswalker to the bin directory

2015-11-20 Thread Phil Race
looks good. -phil. On 11/20/2015 03:20 PM, Pete Brunet wrote: Thanks for looking at this Phil. How's this: http://cr.openjdk.java.net/~ptbrunet/JDK-8056925/webrev.01/ Pete On 11/20/15 3:07 PM, Phil Race wrote: On 11/20/2015 09:24 AM, Pete Brunet wrote: Please review: http://cr.openjdk.java

Re: RfR JDK-8056925 Add jaccessinspector and jaccesswalker to the bin directory

2015-11-20 Thread Pete Brunet
Thanks for looking at this Phil. How's this: http://cr.openjdk.java.net/~ptbrunet/JDK-8056925/webrev.01/ Pete On 11/20/15 3:07 PM, Phil Race wrote: > On 11/20/2015 09:24 AM, Pete Brunet wrote: >> Please review: >> http://cr.openjdk.java.net/~ptbrunet/JDK-8056925/webrev.00/ > > 42 char *printEr

RfR JDK-8056925 Add jaccessinspector and jaccesswalker to the bin directory

2015-11-20 Thread Pete Brunet
Please review: http://cr.openjdk.java.net/~ptbrunet/JDK-8056925/webrev.00/

Re: RfR JDK-8056925 Add jaccessinspector and jaccesswalker to the bin directory

2015-11-20 Thread Phil Race
On 11/20/2015 09:24 AM, Pete Brunet wrote: Please review: http://cr.openjdk.java.net/~ptbrunet/JDK-8056925/webrev.00/ 42 char *printError(char *msg) { 43 LPVOID lpMsgBuf = NULL; 44 static char retbuf[256]; 45 46 if (msg != NULL) { 47 strncpy((char *)retbuf, msg,

Re: RfR JDK-8056925 Add jaccessinspector and jaccesswalker to the bin directory

2015-11-20 Thread Vivi An
Looks good to me. Thanks Vivi On 11/20/2015 9:24 AM, Pete Brunet wrote: Please review: http://cr.openjdk.java.net/~ptbrunet/JDK-8056925/webrev.00/