RFR: JDK-8294427 - Check boxes and radio buttons have rendering issues on Windows in High DPI env [v9]

Alexey Ivanov aivanov at openjdk.org
Tue Jun 6 20:14:16 UTC 2023


On Tue, 2 May 2023 15:27:16 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>> Rajat Mahajan has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   code changes as per code review
>
> src/java.desktop/windows/native/libawt/windows/ThemeReader.cpp line 243:
> 
>> 241: (JNIEnv* env, jclass klass, jstring widget, jint dpi) {
>> 242: 
>> 243:     LPCTSTR str = (LPCTSTR)JNU_GetStringPlatformChars(env, widget, NULL);
> 
> I'd rather keep the original formatting for casts as well as for `JNIEnv *env` in the declaration.

This isn't fully resolved: the space between the type and the expression should be preserved on lines 243 and 254 (in the updated version).


LPCTSTR str = (LPCTSTR) JNU_GetStringPlatformChars(env, widget, NULL);
return (jlong) htheme;

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13701#discussion_r1220245608



More information about the client-libs-dev mailing list