Re: [PATCH] Move up declaration of svn_wc_adm_access_t?

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.

Cheers,
-g

-- 
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