On Wed, 28 Feb 2024 05:37:23 GMT, Chris Plummer <[email protected]> wrote:
>> test/jdk/com/sun/jdi/InterruptHangTest.java line 134:
>>
>>> 132:
>>> 133: if (interruptorThread != null) {
>>> 134: synchronized(InterruptHangTarg.sync) {
>>
>> Nit: A space is missed after `synchronized` keyword.
>
> ok
Actually there are quite a few cases of this in this test, and most are
pre-existing. The test consistently does not use a space, and I see this is
pretty common in other JDI tests. Do we have an official policy on this?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/17989#discussion_r1506748630
