Re: svn commit: r965892 - in /subversion/trunk: notes/dump-load-format.txt subversion/include/svn_repos.h subversion/libsvn_repos/dump.c

C. Michael Pilato wrote on Thu, Jul 22, 2010 at 09:00:25 -0400:
> Now for some nit-picking. This format:
>
> Header-Name: (PROPNAME) CHECKSUM
>
> is unnecessarily complex and, I bet, will catch at least person offguard
> when their dumpstream contains a property name with a ')' character in it.
> Checksum strings have a constant length and a fixed character set, so you
> can simply do:
>
> Header-Name: CHECKSUM PROPNAME
>
> (which also has the nice property of looking similar to the output of
> 'md5sum' and 'sha1sum').
>

Are parentheses valid in property names? (svn_prop_name_is_valid() doesn't allow them.)

Daniel
(that's the first thing I did when I saw the commit: make sure the (propname)
syntax would be unambiguously parsable)

> --
> C. Michael Pilato <cmpilato_at_collab.net>
> CollabNet <> www.collab.net <> Distributed Development On Demand
>
Received on 2010-07-22 15:42:55 CEST