Re: 8227231: JDWP help information shows use of obsolete Xdebug flag

2019-10-10 Thread Daniil Titov
Hi David, Serguei, and Alex, Thank you for reviewing this change and the CSR. Best regards, Daniil On 10/9/19, 5:26 PM, "David Holmes" wrote: +1 on each Thanks, David On 10/10/2019 9:04 am, Alex Menkov wrote: > LGTM > I reviewed CSR as well > > --ale

Re: RFR: 8227231: JDWP help information shows use of obsolete Xdebug flag

2019-10-09 Thread serguei.spit...@oracle.com
Hi Daniil, Looks good. Thanks, Serguei On 10/9/19 15:45, Daniil Titov wrote: Please review a small fix [1] that removes references to obsolete XDebug flag from JDWP help returned by "java -Xrunjdwp:help" command. A new CSR [2] was created for these changes and needs to be reviewed. [1] Webr

Re: RFR: 8227231: JDWP help information shows use of obsolete Xdebug flag

2019-10-09 Thread David Holmes
+1 on each Thanks, David On 10/10/2019 9:04 am, Alex Menkov wrote: LGTM I reviewed CSR as well --alex On 10/09/2019 15:45, Daniil Titov wrote: Please review a small fix [1] that removes references to obsolete XDebug flag from JDWP help returned by "java -Xrunjdwp:help" command. A new CSR [2

Re: RFR: 8227231: JDWP help information shows use of obsolete Xdebug flag

2019-10-09 Thread Alex Menkov
LGTM I reviewed CSR as well --alex On 10/09/2019 15:45, Daniil Titov wrote: Please review a small fix [1] that removes references to obsolete XDebug flag from JDWP help returned by "java -Xrunjdwp:help" command. A new CSR [2] was created for these changes and needs to be reviewed. [1] Webrev:

RFR: 8227231: JDWP help information shows use of obsolete Xdebug flag

2019-10-09 Thread Daniil Titov
Please review a small fix [1] that removes references to obsolete XDebug flag from JDWP help returned by "java -Xrunjdwp:help" command. A new CSR [2] was created for these changes and needs to be reviewed. [1] Webrev: http://cr.openjdk.java.net/~dtitov/8227231/webrev.01/ [2] CSR: https://bugs.op