Re: tny-merge-folder patch

[Philip Van Hoof <spam pvanhoof be>]

On Wed, 2007-07-04 at 15:30 +0200, Murray Cumming wrote:
> On Wed, 2007-07-04 at 15:24 +0200, Johannes Schmid wrote:
> > +2007-07-04  Johannes Schmid <johannes schmid openismus com>
> > +
> > +       * libtinymail/tny-merge-folder.[ch]:
> > +       Added tny_merge_folder_set_folder_type() API to be able
> > +       to assing
> 
> "assing"?
> 
> [snip]

I guess he meant "assign the folder type to the TnyMergeFolder type."

> > +void
> > +tny_merge_folder_set_folder_type (TnyMergeFolder* self, TnyFolderType
> > folder_type) 
> 
> I'd like to see some documentation for this function, please, mentioning
> what it might be used for.

Right, since this is a public method, gtk-doc API documentation is
indeed necessary. Please indeed add the documentation.


-- 
Philip Van Hoof, software developer
home: me at pvanhoof dot be 
gnome: pvanhoof at gnome dot org 
http://www.pvanhoof.be/blog