<AWT Dev>  Review request for 7123476: DesktopOpenTests:When enter the file path and click the open button, it crash
anthony.petrov at oracle.com
Wed Mar 28 08:01:17 PDT 2012
> 2 * Copyright (c) 2005, 2011, Oracle and/or its affiliates. All rights reserved.
I think this file has just been created in 2012 :)
> 30 fprintf(stderr, "gnome_load\n");
Please remove all the debugging output, or put it under the #ifdef
> 70 fprintf(stderr, "can not find symble gnome_url_show\n");
> 447 return TRUE;
I think it also makes sense to check fp_gtk_show_uri for NULL before
returning TRUE here.
The rest of the fix looks fine to me. Thank you!
On 3/27/2012 5:03 PM, Denis S. Fokin wrote:
> Hi Anthony,
> here is a new version of the fix.
> I took into account you suggestions. Now the implementation loads gtk
> API if it exists on the library path. If it does not exists we try to
> load gnome API. If it is not successful we do not support the
> I introduced a couple of files to keep gnome interface separately like
> we do with gtk. I expect that we remove them as soon as all our
> supported OS configurations will be have installed the proper GTK
> library by default.
> As for the synchronization section, I do not see how to use the
> fp_gdk_threads_* functions with gnome API so I put the critical under
> gtk-specific if-clause section.
> Thank you,
> On 2/1/2012 9:03 PM, Anthony Petrov wrote:
>> Hi Denis,
>> The gtk_show_uri() is available since GTK 2.14. Did you verify if all
>> platforms supposed to be supported by JDK 8 have this version of GTK
>> libraries installed by default? I'm mostly concerned about Solaris
>> systems, as well as corporate Linux desktops. If this is not the case,
>> then perhaps using this function should be conditional, and with the old
>> GTK library we should fall back to using the old API. You may notice
>> that, for example, for the file chooser we have an explicit check for
>> GTK 2.8.0 and use the new
>> gtk_file_chooser_set_do_overwrite_confirmation() API only when it's
>> I like that we move all the GTK-related utility code to the
>> gtk2_interface files. A few comments:
>> 1. Please use the TRUE and FALSE constants instead of 1 and 0 as a
>> return value for gtk2_show_uri_load().
>> 2. Should the fprintf() call be #ifdef'ed for INTERNAL_BUILD's only?
>> best regards,
>> On 2/1/2012 7:39 PM, Denis S. Fokin wrote:
>>> Hi AWT team,
>>> Please review a fix for the CR 7123476 at
>>> CR URL:
>>> The Gnome API is deprecated so we need to migrate on GTK function. See
>>> the next thread.
>>> Thank you,
More information about the awt-dev