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

Re: svn commit: rev 5677 - in trunk/subversion: include libsvn_wc

From: <cmpilato_at_collab.net>
Date: 2003-04-19 00:19:46 CEST

kfogel@tigris.org writes:

> Author: kfogel
> Date: Fri Apr 18 16:45:54 2003
> New Revision: 5677
>
> Modified:
> trunk/subversion/include/svn_wc.h
> trunk/subversion/libsvn_wc/adm_ops.c
> trunk/subversion/libsvn_wc/lock.c
> Log:
> Factorize the access baton retrieval changes from revision 5663, and
> make the code smarter about when to error and when not. Thanks to
> Greg Stein and Mike Pilato for review and discussion:
>
> * subversion/include/svn_wc.h, subversion/libsvn_wc/lock.c
> (svn_wc_adm_probe_try): New function, tries various ways to get an
> access baton while still playing nice with a pre-existing
> associated set, and gives up only if it must.
>
> * subversion/libsvn_wc/adm_ops.c
> (svn_wc_add, svn_wc_delete): Use above new function instead of
> inline code.

Mmm... watch the code shrink, shrink, shrink away...

> Modified: trunk/subversion/include/svn_wc.h
> ==============================================================================
> --- trunk/subversion/include/svn_wc.h (original)
> +++ trunk/subversion/include/svn_wc.h Fri Apr 18 16:45:54 2003
> @@ -150,6 +150,30 @@
> const char *path,
> apr_pool_t *pool);
>
> +/* Try to obtain @a *adm_access via @c svn_wc_adm_probe_retrieve(),

Does doxygen require that comments start with '/**' instead of just
'/*' ? I've noticed that all of our public API docstrings look like
that now.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Apr 19 00:23:07 2003

This is an archived mail posted to the Subversion Dev mailing list.