On Wed, 22 Apr 2026 07:08:36 GMT, Joel Sikström <[email protected]> wrote:

>> 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.

Good catch. This is way to easy to miss in the Github PR diff.

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

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

Reply via email to