Re: svn commit: rev 3964 - in trunk: . subversion subversion/libsvn_ra subversion/include subversion/tests/clients/cmdline/getopt_tests_data subversion/libsvn_ra_svn subversion/svnserve

Hello.

I glimpsed over ra_svn patch and noticed a minor thing:

On Mon 2002-12-02 at 21:55:50 -0600, ghudson@tigris.org wrote:
[...]
> +The Subversion protocol is specified in terms of the following
> +syntactic elements, specified using ABNF [RFC 2234]:
> +
> + item = word / number / string / list
> + word = ALPHA *(ALPHA / DIGIT / "-") space
> + number = 1*DIGIT space
> + string = number ":" *OCTET space
> + ; number gives the byte count of the *OCTET portion
> + list = "(" space *item ")" space
> + space = 1*(SP / LF)
> +
> +Here is an example item showing each of the syntactic elements:
> +
> + ( word 22 6:string ( sublist ) )

Correct me, if I am wrong, but doesn't the ABNF say that there must be
(at least one) space after each number? If so, it must be

  ( word 22 6 :string ( sublist ) )

as it is represented by

  ( word number string list )

which is (only expanded as necessary to make my point)

  ( word 1*DIGIT space 1*DIGIT space ":" *OCTET space list )

HTH,

        Benjamin.