GNOME Bugzilla – Bug 164113
Feature Request: DirectoryHandle::read_next() return null RefPtr instead of 'exist' arg
Last modified: 2011-01-16 23:36:14 UTC
Currently, the following has to be done: bool exist; Glib::RefPtr<Gnome::Vfs::FileInfo> info; while ((info = directory_handle.read_next(exist)) && exist) { ... } The 'exist' argument is cumbersome and seems redundant. Compare if instead read_next() just returned a null RefPtr, the above could be: Glib::RefPtr<Gnome::Vfs::FileInfo> info; while (info = directory_handle.read_next()) { ... }
Created attachment 36106 [details] [review] Patch for above feature
Created attachment 36108 [details] [review] Again, as overloaded read_next() function
This looks good, but unfortunately it has missed the API freeze for GNOME 2.9/2.10. I will apply it when we start GNOME 2.11/2.12 in March.
Applied to HEAD. Thanks. Please patch the Changelog and create a cvs diff next time: http://www.gtkmm.org/bugs.shtml#CreatingPatches