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.