Re: RFR: 8296445: C++ syntax error in jdwpTransport.h

2022-11-06 Thread pandaapo
On Mon, 7 Nov 2022 01:33:53 GMT, Dingli Zhang wrote: > Hi @pandaapo thanks for your patch! You need to change the name of this PR to > `8296445: C++ syntax error in jdwpTransport.h` to match the issue in JBS. Thanks. @DingliZhang - PR: https://git.openjdk.org/jdk/pull/11008

Re: RFR: JDK-8296445: Fix a syntax error.

2022-11-06 Thread Dingli Zhang
On Sun, 6 Nov 2022 23:02:35 GMT, pandaapo wrote: > Fix a syntax error in the code in jdwpTransport.h Hi @pandaapo thanks for your patch! You need to change the name of this PR to `8296445: C++ syntax error in jdwpTransport.h` to match the issue in JBS. - PR: https://git.openjdk.o

RFR: JDK-8296445: Fix a syntax error.

2022-11-06 Thread pandaapo
Fix a syntax error in the code in jdwpTransport.h - Commit messages: - Fix bug: syntax error. Changes: https://git.openjdk.org/jdk/pull/11008/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=11008&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8296445 Stats: 1 line i

Re: [PATCH] syntax error in jdwpTransport.h

2022-11-06 Thread David Holmes
Thanks for the report I have filed: https://bugs.openjdk.org/browse/JDK-8296445 for this. Cheers, David On 6/11/2022 4:33 pm, 小破 wrote: *Bug description* The code in jdwpTransport.h has syntax error. For example, use `g++ -c jdwpTransport.h` to compile it, this error message will be produced