On Tue, 2006-07-04 at 13:44 -0400, Bradford Powell wrote: > Hello, > > Looking over your patch for 342137, I see that you modified the > constructor for FileImportBackend so that it takes a GTK.Dialog as an > argument to pass along to the XmpTagsImporter. However, once > XmpTagsImporter sets its 'dialog' property, I don't see that it is ever > accessed. This may be either a vestige of earlier usage or planning for > future use, but either way, I think it may be better to remove the dialog > variable from XmpTagsImporter and revert the FileImportBackend back to how > it was for now, in the interest of keeping patches simpler. > > I'm not sure of proper protocol, so let me know if I should have just > brought this up on the f-spot mailing list or added a comment to the > bugzilla bug. >
Hi I have updated the patch, and removed the dialog parameter. Thanks for the feedback /Bengt _______________________________________________ F-spot-list mailing list [email protected] http://mail.gnome.org/mailman/listinfo/f-spot-list
