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

[Anthony Petrov]

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