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

Re: svn commit: r1144316 - in /subversion/trunk/subversion/libsvn_repos: commit.c fs-wrap.c

From: Philip Martin <philip.martin_at_wandisco.com>
Date: Fri, 08 Jul 2011 20:39:02 +0100

Daniel Shahaf <d.s_at_daniel.shahaf.name> writes:

> My point wasn't the validation of svn:mergeinfo revprops, it was whether
> the validation of svn:mergeinfo nodeprops should happen in
> svn_repos__validate_prop().
>
> I'm not against telling that function whether it's validating a nodeprop
> or a revprop, and applying different logic in either case.

Is there any common validation between node props and rev props? If the
validation is completely different it should be two functions rather
than one function with a boolean flag.

-- 
uberSVN: Apache Subversion Made Easy
http://www.uberSVN.com
Received on 2011-07-08 21:39:39 CEST

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.