On Tue, 3 Dec 2024 15:52:30 GMT, Alexey Ivanov <[email protected]> wrote:
>> `Properties` doesn't allow `null` values. >> We can replace containsKey+getProperty with getProperty+null check. >> It's clearer and a bit faster. > > src/java.desktop/share/classes/java/awt/Cursor.java line 303: > >> 301: String key = prefix + DOT_FILE_SUFFIX; >> 302: >> 303: String fileName = >> systemCustomCursorProperties.getProperty(key); > > You can keep the `final` modifier, probably. I really like how this line now fits in 80 characters limit. But if you insist - I will return it. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/21824#discussion_r1868181704
