Re: tny-merge-folder observer notification



I agree, and the patch looks good too.

Please commit and thanks!

On Thu, 2007-07-05 at 18:58 +0200, Armin Burgmeier wrote:
> Hi tinymail developers,
> 
> Currently, TnyMergeFolder observes its underlaying folders and
> propagates any TnyFolderChanges to its own observers without
> modification. However, since those observers actually observe the
> TnyMergeFolder (and not necessarily the underlaying folders) it seems
> more correct when the changed folder for the TnyFolderChange is set to
> the TnyMergeFolder before propagating. Also, if an underlaying folder is
> renamed it does not affect the merge folder itself.
> 
> I prepared a patch to change the behaviour as described. This also fixes
> projects.maemo.org bug NB#59369.
> 
> Thanks,
> Armin
> _______________________________________________
> tinymail-devel-list mailing list
> tinymail-devel-list gnome org
> http://mail.gnome.org/mailman/listinfo/tinymail-devel-list
-- 
Philip Van Hoof, software developer
home: me at pvanhoof dot be 
gnome: pvanhoof at gnome dot org 
http://www.pvanhoof.be/blog







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