RFR: 8334599: Improve code from JDK-8302671 [v2]
Alexey Ivanov
aivanov at openjdk.org
Mon Jul 22 18:59:32 UTC 2024
On Thu, 11 Jul 2024 08:36:23 GMT, Julian Waters <jwaters at openjdk.org> wrote:
>> src/java.desktop/windows/native/libawt/windows/awt_Component.cpp line 3365:
>>
>>> 3363: }
>>> 3364: static void
>>> 3365: resetKbdState(BYTE (&kstate) [AwtToolkit::KB_STATE_SIZE]) {
>>
>> I don't know what is most used syntax for this type. I'd rather keep them together without a space between `()` and `[]`.
>
> Will do
Here, I'd remove the space after the opening parenthesis: `resetKbdState(BYTE`.
It's not worth changing at this point, though…
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19798#discussion_r1687016681
More information about the client-libs-dev
mailing list