Re: dialog in patch for Bug 342137 (f-spot)

[Bengt Thuree <bengt thuree com>]

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