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 <cmpil...@collab.net> CollabNet <> www.collab.net <> Enterprise Cloud Development
signature.asc
Description: OpenPGP digital signature
