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

Re: svn commit: r24693 - trunk/subversion/libsvn_client

From: Daniel Rall <dlr_at_collab.net>
Date: 2007-04-23 19:31:15 CEST

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.

  • application/pgp-signature attachment: stored
Received on Mon Apr 23 19:31:56 2007

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.