On Mon, 15 Apr 2024 14:05:24 GMT, Martin Doerr <mdo...@openjdk.org> wrote:

>> src/java.base/aix/classes/jdk/internal/loader/ClassLoaderHelper.java line 40:
>> 
>>> 38:     * This method returns false so that loading of shared library 
>>> continues if
>>> 39:     * libname.so is not present.
>>> 40:     */
>> 
>> The '*' should be in the same column. Please compare to 
>> ClassLoaderHelper.java for other OSs.
>
> '*' should are in the same column, now, but it is still not the same coding 
> style as in other files. Why can't you do it exactly like 
> https://github.com/openjdk/jdk/blob/273df6286a9cc4dcef4cd339234206b2ad922386/src/java.base/unix/classes/jdk/internal/loader/ClassLoaderHelper.java#L43
>  ?

Thanks. I didn't notice the one extra space.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17945#discussion_r1566037541

Reply via email to