Re: Trash directory on removable media - Bug 138058



On 2/5/07, Alexander Larsson <alexl redhat com> wrote:
Still missing spaces in a few places like:

+gchar*
+get_trash_uri_for_volume(GnomeVFSVolume *volume)
and
+       if (gnome_vfs_volume_handles_trash(volume)) {

I think I got them all now :)

+       uri = 0;
Use NULL here, not 0


Done.

get_trash_uri_for_volume(GnomeVFSVolume *volume):
This function leaks vol_uri if there is no trash dir directory.


Moved the free outside the if

delete_callback():
The callback owns debuting_uris, so you must free it.


Done

You should ref volume/drive over delete callback to make sure it doesn't
go away due to other reasons while showing the dialog.


Done

Do you think this warrants a gconf option to disable the prompt?

Jesse

Attachment: nautilus-empty-trash-on-unmount.patch
Description: Binary data



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]