"Garrett Rooney" <rooneg@electricjellyfish.net> writes:
> On 8/22/06, kfogel@tigris.org <kfogel@tigris.org> wrote:
>
>> +/* Return string representation of DEPTH */
>> +static const char *
>> +depth_str(svn_depth_t depth)
>> +{
>> + switch (depth)
>> + {
>> + case svn_depth_unknown:
>> + return "unknown";
>> + case svn_depth_zero:
>> + return "zero";
>> + case svn_depth_one:
>> + return "one";
>> + case svn_depth_infinity:
>> + return "infinity";
>> + default:
>> + /* Remember that svn_depth_exclude should never happen here. */
>> + return "INVALID";
>> + }
>> +}
>
> It seems like this function should be used in the following printfs,
> instead of hardcoding the names in two places.
I'd like to, yes, but localization gets a bit trickier then...
-K
>> + switch (info->depth)
>> + {
>> + case svn_depth_unknown:
>> + /* Unknown depth is the norm for remote directories anyway
>> + (although infinity would be equally appropriate). Let's
>> + not bother to print it. */
>> + break;
>> +
>> + case svn_depth_zero:
>> + SVN_ERR(svn_cmdline_printf(pool, _("Depth: zero\n")));
>> + break;
>> +
>> + case svn_depth_one:
>> + SVN_ERR(svn_cmdline_printf(pool, _("Depth: one\n")));
>> + break;
>> +
>> + case svn_depth_infinity:
>> + /* Infinity is the default depth for working copy
>> + directories. Let's not print it, it's not special enough
>> + to be worth mentioning. */
>> + break;
>> +
>> + default:
>> + /* Remember that svn_depth_exclude should never happen here. */
>> + SVN_ERR(svn_cmdline_printf(pool, _("Depth: INVALID\n")));
>> + }
>> +
>> if (info->copyfrom_url)
>> SVN_ERR(svn_cmdline_printf(pool, _("Copied From URL: %s\n"),
>> info->copyfrom_url));
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Aug 23 02:40:29 2006