Re: tny-merge-folder observer notification



On Thu, 2007-07-05 at 23:24 +0200, Philip Van Hoof wrote:
> I agree, and the patch looks good too.
> 
> Please commit and thanks!
Sorry, I don't think I have commit access to the tinymail SVN.

> 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

Armin




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