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

Re: [PATCH] Move up declaration of svn_wc_adm_access_t?

From: Greg Stein <gstein_at_lyra.org>
Date: 2002-07-11 22:40:10 CEST

On Thu, Jul 11, 2002 at 07:20:03PM +0100, Philip Martin wrote:
> Kevin Pilch-Bisson <kevin@pilch-bisson.net> writes:
> > On Thu, Jul 11, 2002 at 10:58:54AM -0700, Justin Erenkrantz wrote:
> > > Would anyone have a problem if I move the svn_wc_adm_access_t
> > > declaration and functions to the top of svn_wc.h? In order to use
> > > the baton in svn_wc calls, it has to be declared first. -- justin
> >
> > I'd rather changed the functions that need it to take a void * (or maybe an
> > incomplete type, and keep the details out of the public header personally.
> I'm working on next step in issue 749. As a side effect it moves them
> up, and makes svn_wc_adm_access_t incomplete. I plan to commit it
> later today.

Ah... perfect. +1 -- definitely hide the definition, yet leave it as an
incomplete type for the safety.


Greg Stein, http://www.lyra.org/
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Jul 11 22:38:47 2002

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.