> -----Original Message-----
> From: Daniel Rall [mailto:dlr@collab.net]
> Sent: Monday, April 23, 2007 1:31 PM
> To: Paul Burba
> Cc: dev@subversion.tigris.org
> Subject: Re: svn commit: r24693 - trunk/subversion/libsvn_client
>
> On Fri, 20 Apr 2007, Paul Burba wrote:
>
> > Hi Dan,
> >
> > I don't understand this, we now have svn_client__elide_mergeinfo()
> > declared twice:
> >
> > subversion\libsvn_client\client.h
> >
> > and
> >
> > include\private\svn_client_mergeinfo_private.h (r24653)
> >
> > Should I get rid of the latter now?
>
> I was trying to fix a compiler warning about there not being
> a previous declaration of the svn_client__elide_mergeinfo()
> API, and didn't notice that you had defined it.
>
> It should be defined in either client.h (since it is a client
> library-internal API, not shared across libraries), or in a
> new mergeinfo.h. I'll take care of the latter, since there
> are other merge info-specific APIs which would do well
> factored into their own header file.
To close this thread: dlr moved svn_client__elide_mergeinfo() into
libsvn_client\mergeinfo.h in r24717.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Apr 24 00:26:45 2007