RFR: 8304685: Fix whitespace parsing in libjdwp [v2]

Kevin Walls kevinw at openjdk.org
Fri May 19 19:00:58 UTC 2023


On Fri, 12 May 2023 20:29:50 GMT, Kevin Walls <kevinw at openjdk.org> wrote:

>> Problem call to skipWhitespace in dbgsysExec src/jdk.jdwp.agent/unix/native/libjdwp/exec_md.c:91
>> 
>> We call skipWhitespace even when the loop will terminate, should guard against the last skipWhitespace call on the last iteration.
>> 
>> Could be done differently with the skipWhitespace call at the start of the loop, but skipped on the first iteration, but leaving the statements in the loop in the same order seems more readable.
>
> Kevin Walls has updated the pull request incrementally with one additional commit since the last revision:
> 
>   move skipWhitespace to start

Thanks for the reviews!

-------------

PR Comment: https://git.openjdk.org/jdk/pull/13964#issuecomment-1555097491


More information about the serviceability-dev mailing list