<AWT Dev> [9] Review request for 8149115: [hidpi] Linux: display-wise scaling factor should probably be taken into account
Semyon Sadetsky
semyon.sadetsky at oracle.com
Fri Jul 8 06:25:53 UTC 2016
I have changed variables declaration style in systemScale.c, because it
produced warnings in Solaris build.
http://cr.openjdk.java.net/~ssadetsky/8149115/webrev.03/
--Semyon
On 7/7/2016 9:57 AM, Alexandr Scherbatiy wrote:
> The fix looks good to me.
>
> Thanks,
> Alexandr.
>
> On 7/6/2016 11:46 PM, Alexander Zvegintsev wrote:
>> Still looks good.
>>
>> --
>> Thanks,
>> Alexander.
>>
>> On 06.07.2016 21:42, Semyon Sadetsky wrote:
>>> Thanks, Alexander. Please see the updated webrev
>>> http://cr.openjdk.java.net/~ssadetsky/8149115/webrev.02/
>>>
>>> --Semyon
>>>
>>>
>>> On 7/6/2016 9:03 PM, Alexander Zvegintsev wrote:
>>>> The fix looks good to me.
>>>>
>>>> Just a minor comment: multiple NULL checks in get_schema_value
>>>> could be wrapped in CHECK_NULL_RETURN macro, e.g.:
>>>>
>>>> CHECH_NULL_RETURN(fp_g_settings_schema_has_key
>>>> = dlsym(lib_handle, "g_settings_schema_has_key"), NULL);
>>>> CHECH_NULL_RETURN(fp_g_settings_new_full
>>>> = dlsym(lib_handle, "g_settings_new_full"), NULL);
>>>>
>>>>
>>>> On 7/6/16 5:27 PM, Alexandr Scherbatiy wrote:
>>>>>
>>>>> The fix looks good to me.
>>>>>
>>>>> Thanks,
>>>>> Alexandr.
>>>>>
>>>>> On 7/6/2016 4:46 PM, Semyon Sadetsky wrote:
>>>>>> On 7/6/2016 12:26 PM, Alexandr Scherbatiy wrote:
>>>>>>
>>>>>>> On 7/5/2016 9:59 PM, Semyon Sadetsky wrote:
>>>>>>>> Hello,
>>>>>>>>
>>>>>>>> Please review fix for JDK9:
>>>>>>>>
>>>>>>>> bug: https://bugs.openjdk.java.net/browse/JDK-8149115
>>>>>>>>
>>>>>>>> webrev: http://cr.openjdk.java.net/~ssadetsky/8149115/webrev.00/
>>>>>>>>
>>>>>>>> Currently the hidpi support on linux only reads GDK_SCALE
>>>>>>>> environment variable to get the native scale. Although, Gnome3
>>>>>>>> and Unity DE use own settings to control the interface scale.
>>>>>>>> The fix adds possibility to read those settings to make java
>>>>>>>> apps hidpi scale similar to the native apps. Currently only
>>>>>>>> integer scale values are supported. If native scale is not an
>>>>>>>> integer value it is rounded to the nearest integer.
>>>>>>> systemScale.c
>>>>>>> Should the while loop have a break statement in the
>>>>>>> getDesktopScale(...) method?
>>>>>> That's make sense. Please look at the updated webrev:
>>>>>> http://cr.openjdk.java.net/~ssadetsky/8149115/webrev.01/
>>>>>>
>>>>>> --Semyon
>>>>>>
>>>>>>>
>>>>>>> Thanks,
>>>>>>> Alexandr.
>>>>>>>
>>>>>>>>
>>>>>>>> --Semyon
>>>>>>>>
>>>>>>>
>>>>>>
>>>>>
>>>>
>>>
>>
>
More information about the awt-dev
mailing list