On Wed, 22 Apr 2026 07:05:10 GMT, SendaoYan <[email protected]> wrote:

>> Joel Sikström has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Update copyright
>
> test/jdk/com/sun/jdi/ClassesByName2Test.java line 141:
> 
>> 139: 
>> 140:     private static boolean isHiddenClass(String className) {
>> 141:       return className.contains("/");
> 
> The default indendation for this file is 4 whitespaces. But the new lines of 
> this proposed change is 2 whitespaces.
> Does it necessary to maintain consistency.

That's a good catch. Fixed.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/30825#discussion_r3122154050

Reply via email to