On Tue, 10 Oct 2006, Kamesh Jayachandran wrote:
>
> >[[[
> >Abstract construction of a mergeinfo of a single path with single
> >merge-range
> >into a separate function.
> >
> >On the merge-tracking branch:
> >Patch By: dlr
> > madanus
> >
> >* subversion/include/svn_mergeinfo.h
> > (svn_mergeinfo__make): New.
> Somehow I feel not having '__', if it is '__' the function should be on
> a private header file.
> May be 'svn_mergeinfo__make' -> 'svn_mergeinfo_make'?
We already have double-underscore (private) functions in our private
header files. If there were an appropriate private header, I would
prefer to add private APIs there. Without an appropriate spot, I'm
okay adding them to the usual headers. *shrug*
- application/pgp-signature attachment: stored
Received on Wed Oct 11 02:49:07 2006