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

Re: CVS update: MODIFIED: include ...

From: <kfogel_at_collab.net>
Date: 2001-08-14 18:01:35 CEST

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
Received on Sat Oct 21 14:36:36 2006

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.