On Tue, 1 Oct 2024 21:13:35 GMT, David Holmes <dhol...@openjdk.org> wrote:

>> SendaoYan has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   merge two comments enclosed with /* ... */ on two lines
>
> test/jdk/com/sun/tools/attach/java.policy.allow line 17:
> 
>> 15: 
>> 16:     /* to read configuration file in META-INF/services, write/delete 
>> .attach_pid<pid>, 
>> 17:     and read symbolic link of /proc/self/ns/mnt */
> 
> Suggestion:
> 
>     /* 
>      * To read configuration file in META-INF/services, write/delete 
> .attach_pid<pid>, 
>      * and read symbolic link of /proc/self/ns/mnt.
>      */

Thanks for the review. The comment has been changed according your advice.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21269#discussion_r1784795554

Reply via email to