You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by kf...@collab.net on 2001/08/14 16:01:35 UTC

Re: CVS update: MODIFIED: include ...

gstein@tigris.org writes:
>   Index: update.c
>   ===================================================================
>   RCS file: /usr/local/tigris/data/helm/cvs/repository/subversion/subversion/mod_dav_svn/update.c,v
>   retrieving revision 1.7
>   retrieving revision 1.8
>   diff -u -b -r1.7 -r1.8
>   --- update.c	2001/08/09 11:00:27	1.7
>   +++ update.c	2001/08/14 03:41:21	1.8
>   @@ -353,9 +353,9 @@
>      if (ns == -1)
>        {
>          return dav_new_error(resource->pool, HTTP_BAD_REQUEST, 0,
>   -                           "The request does not contain the 'SVN:' "
>   +                           "The request does not contain the 'svn:' "
>                               "namespace, so it is not going to have an "
>   -                           "SVN:target-revision element. That element "
>   +                           "svn:target-revision element. That element "
>                               "is required.");
>        }

Some reason you preferred not to do the usual

   "literal string"  DEFINED_CONSTANT  "more literal string"

concatenation trick here?  Just curious -- you seem to have used the
defined constant for most of the rest of this change.

-Karl

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org