Re: svn commit: rev 2803 - trunk/subversion/include trunk/subversion/libsvn_wc trunk/subversion/libsvn_client trunk/subversion/tests/clients/cmdline
From: Philip Martin <philip_at_codematters.co.uk>
Date: 2002-07-30 22:40:49 CEST
Karl Fogel <kfogel@newton.ch.collab.net> writes:
> > +/* Return an access baton in ADM_ACCESS associated with PATH. PATH must be
In the case above it's a pointer-to-pointer, so I could make it
"Return a pointer to a new access baton in *ADM_ACCESS"
Lots of functions take a parameter "svn_wc_adm_access_t *adm_access"
"Ensure ADM_ACCESS has a write lock".
Do you think that should be changed as well? svn_wc_adm_access_t is
Do people prefer "working copy context" to "access baton"?
>
I like the idea of the type being opaque to libsvn_wc as well.
>
Yes.
>
OK.
>
As I remember, the use of pool cleanup handlers means that at least
-- Philip Martin --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org For additional commands, e-mail: dev-help@subversion.tigris.orgReceived on Tue Jul 30 22:41:39 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.