Gtk+ 3 has been migrated to use the XDG Basedir spec to store bookmarks. We should update the user dirs to do the same. http://git.gnome.org/browse/xdg-user-dirs-gtk/tree/parse.c#n169
Created attachment 73210 [details] [review] part 1
Created attachment 73211 [details] [review] part 2
Created attachment 73212 [details] [review] part 2
Comment on attachment 73212 [details] [review] part 2 Review of attachment 73212 [details] [review]: ----------------------------------------------------------------- ::: parse.c @@ +16,5 @@ > + legacy_filename = g_build_filename (g_get_home_dir (), ".gtk-bookmarks", NULL); > + legacy_exists = g_file_test (legacy_filename, G_FILE_TEST_EXISTS); > + > + if (legacy_exists) > + return legacy_filename; When returning here, you leak filename
Is gtk2 also updated to read from the new location? If not it seems that we're not upgrading gtk2 apps and we should also replace any existing old file.
(In reply to comment #5) > Is gtk2 also updated to read from the new location? If not it seems that > we're not upgrading gtk2 apps and we should also replace any existing old > file. Yeah, GTK2 is also updated to read from the new location since 2.24.14. I'll post a new patch for the leak found by Matthias.
Created attachment 73386 [details] [review] part 2
already pushed with the memleak fix
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.