[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH][MERGE-TRACKING] New function to create a mergeinfo hash out of a single path and single merge-range

From: Daniel Rall <dlr_at_collab.net>
Date: 2006-10-11 02:50:36 CEST

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

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.