vgeorgescu_at_tigris.org writes:
> --- trunk/subversion/libsvn_wc/update_editor.c
> +++ trunk/subversion/libsvn_wc/update_editor.c
> @@ -207,6 +208,9 @@ struct dir_baton
> used for filtering; we have a separate wrapping editor for that. */
> svn_depth_t ambient_depth;
>
> + /* Was the directory marked as incomplete before the update? */
> + svn_boolean_t was_incomplete;
> +
> /* The pool in which this baton itself is allocated. */
> apr_pool_t *pool;
> };
Would it be correct if I changed this new comment to say:
/* Was the directory marked as incomplete before the update?
(In other words, are we resuming an interrupted update or
checkout?) */
I thought your comment was fine; I'd just like to add a little more
information to save future readers some speculation time. But before I
do, I want to make sure what I say is right :-).
> @@ -1579,12 +1588,29 @@ externals_prop_changed(apr_array_header_
> }
>
> static svn_error_t *
> +add_prop_deletion(void *baton, const void *key,
> + apr_ssize_t klen, void *val,
> + apr_pool_t *pool)
> +{
> + apr_array_header_t *propchanges = baton;
> + const char *name = key;
> + svn_prop_t *prop = apr_array_push(propchanges);
> +
> + /* Add the deletion of NAME to PROPCHANGES. */
> + prop->name = name;
> + prop->value = NULL;
> +
> + return SVN_NO_ERROR;
> +}
This needs a doc string; I gave it one in r32589.
Thanks for the fix!
-K
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-08-20 20:57:04 CEST