Re: [PATCH] Don't validate properties during deletion.

On 06/21/2013 06:43 AM, Daniel Shahaf wrote:
> I still think the logic better belongs inside svn_repos__validate_prop().
> (Not the least because it has three other callers which also need to
> accept NULL values.)
>
> --- subversion/libsvn_repos/fs-wrap.c (revision 1495373)
> +++ subversion/libsvn_repos/fs-wrap.c (working copy)
> @@ -172,6 +172,10 @@ svn_repos__validate_prop(const char *name,
> {
> svn_prop_kind_t kind = svn_property_kind2(name);
>
> + /* No property is mandatory. */
> + if (value == NULL)
> + return SVN_NO_ERROR;
> +
> /* Disallow setting non-regular properties. */
> if (kind != svn_prop_regular_kind)
> return svn_error_createf
>

+1. This is precisely what I was suggesting as well.

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Enterprise Cloud Development