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

Re: svn commit: r21361 - in branches/incomplete-directories/subversion: libsvn_ra_dav libsvn_ra_serf libsvn_ra_svn mod_dav_svn/reports svnserve

From: Peter Samuelson <peter_at_p12n.org>
Date: 2006-09-06 10:44:59 CEST

[kfogel@tigris.org]
> + if (depth != svn_depth_infinity)
> + depthstring = apr_psprintf(pool, "depth=\"%d\"", depth);
> +
> if (lock_token)
> tokenstring = apr_psprintf(pool, "lock-token=\"%s\"", lock_token);
>
> svn_xml_escape_cdata_cstring(&qpath, path, pool);
> - /* ### TODO: send depth here */
> if (start_empty)
> entry = apr_psprintf(pool,
> - "<S:entry rev=\"%ld\" %s"
> + "<S:entry rev=\"%ld\" %s %s"
> " start-empty=\"true\">%s</S:entry>" DEBUG_CR,

Is it worth the trouble to put a leading space into 'depthstring' and
'tokenstring' above, instead of in this format string, so that the wire
data looks a bit neater (only one space)? The code is no more complex
or anything, so I guess it's just a question of whether it would be
considered equally readable.

Received on Wed Sep 6 10:45:37 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.