<AWT Dev> A patch so multiple selected files can have different directories.

Matthew Smith melkor at orangepalantir.org
Mon Jan 30 11:51:51 PST 2012


I made the changes suggested and tried it out. Everything seems ok.


On 01/30/2012 10:08 AM, Anthony Petrov wrote:
> Hi Matthew,
> I've published a webrev at:
> http://cr.openjdk.java.net/~anthony/8-10-GTKFileDialogMultiSelRecentFiles-7132194.0/ 
> I have a few stylistic comments:
> 1. Lines 187, 235: please avoid using reverse order for conditionals. 
> I.e. (list == NULL) is preferred to (NULL == list).
> 2. Line 208: always put the if/else (and while()) statements into 
> blocks {} (and add a new line after {.)
> 3. Lines 208, 235,236, 240, 246: please use proper spacing (e.g.. "if 
> (cond) {", "a = b;", etc.)
> 4. (this one is less stylistic): at line 208 what prevents us from 
> using "if (entry[0] == '/')"? This seems more simple and robust than 
> using strchr() and pointer arithmetics.
> Other than that the fix looks fine to me, thank you! Once you fix the 
> issues above and send us an updated patch, I can push the fix to the 
> repository.
> Comments from other AWT team members (and anyone interested) are welcome!
> -- 
> best regards,
> Anthony
> On 01/24/12 00:50, Matthew Smith wrote:
>> openjdk/jdk/src/solaris/native/sun/awt/sun_awt_X11_GtkFileDialogPeer.c
>> This patch is intended to address the issue of newer versions of gtk
>> where the file dialog lets you select files w/out selecting a directory
>> via Search or Recently Used. When this happens the 'current directory'
>> is returned as null, and the Files selected will be returned with the
>> current working directory appended to the path.
>> The .java file will show the incorrect behavior if you run it and use
>> Recently Used to select a file (gnome 3).
>> This patch does not change the behavior when a directory is selected, or
>> when the dialog is canceled. If you select a file from the Recently Used
>> option the directory will be set to the root directory, and the filename
>> will be the complete path and filename. When multiple files are selected
>> from a folder the original behavior occurs. If multiple files are
>> selected from Recently Used then the directory is again set to root, and
>> all of the fiIes have the complete name.
>> This work around can be verified with the .java file provided.
>> Finally this is an updated version of the patch, originally I used
>> 'bool' instead of a 'gboolean' I did compile it and test that it works.
>> Thank you
>> mbs

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: patch.txt
Url: http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20120130/4fa69ecf/patch.txt 

More information about the awt-dev mailing list