On Wed, 18 Oct 2023 20:28:53 GMT, Christoph Langer <clan...@openjdk.org> wrote:

>> Johannes Bechberger has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Add suggested modification
>
> src/jdk.jdwp.agent/share/native/libjdwp/debugInit.c line 631:
> 
>> 629:  * Initialize debugger back end modules
>> 630:  *
>> 631:  * @param opt_info optional event info to use, might be null
> 
> No java code here, so Javadoc syntax isn't appropriate. I suggest to remove 
> the `@param` notation.

This syntax can be found in 188 other C/C++ files in the OpenJDK, like the 
macroAssembler and stubGenerator files.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16145#discussion_r1364566031

Reply via email to