<AWT Dev> <AWT dev>[9] Review request for 8157827: AWT_Desktop/Automated/Exceptions/BasicTest loads incorrect GTK version when jdk.gtk.version=3
Semyon Sadetsky
semyon.sadetsky at oracle.com
Thu May 26 07:12:41 UTC 2016
I am sorry for any misunderstanding I may have caused.
That was your request in
http://mail.openjdk.java.net/pipermail/swing-dev/2016-April/005773.html :
>I think we can/should quickly fix the new ordinal() usage and file a
>separate bug for the GTKEngine usage.
I just fulfilled it. But, my fault, I missed one place to convert
ordinal() to getNumber().
Yes, the 8154992 is about the remaining ordinal() refactoring that we
postponed because there are plenty of them in the code and it may take a
noticeable time to fix them all.
--Semyon
On 5/25/2016 11:09 PM, Phil Race wrote:
> Where did getNumber() come from ? Did you miss a file in the review ?
>
> Under what circumstances does this fail as I did not notice such a
> failure.
>
> And yet this sounds like what was pointed out by Sergey during the GTK
> 3 review here :
> http://mail.openjdk.java.net/pipermail/swing-dev/2016-March/005495.html
> and I thought was going to be fixed before it was pushed as requested
> here :-
> http://mail.openjdk.java.net/pipermail/swing-dev/2016-April/005773.html
>
> The request for a separate bug which you filed here :-
> https://bugs.openjdk.java.net/browse/JDK-8154992
> was for the pre-existing usages.
>
> -phil.
>
> On 05/25/2016 10:54 AM, Semyon Sadetsky wrote:
>> Hello,
>>
>> Please review fix for JDK9:
>>
>> bug: https://bugs.openjdk.java.net/browse/JDK-8157827
>>
>> webrev: http://cr.openjdk.java.net/~ssadetsky/8157827/webrev.00/
>>
>> In the Desktop initialization procedure GtkVersions#ordinal() was
>> used instead of GtkVersions#getNumber().
>>
>> --Semyon
>>
>
More information about the awt-dev
mailing list