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

Re: [PROPOSAL] Change of convention for multiline string literals

From: Jonathan Gilbert <o2w9gs702_at_sneakemail.com>
Date: 2007-04-22 09:10:56 CEST

At 01:26 AM 4/22/2007 -0500, Peter Samuelson wrote:
>[Jonathan Gilbert]
>> +/**
>> + * A shorter version of APR_EOL_STR; holds the native linefeed for the
>> + * current platform. E.g., on UNIX systems, this will be "\n", while on
>> + * Windows, it will be "\r\n". This is defined privately because the
>> + * macro lacks a namespacing prefix. Used to terminate lines in large
>> + * multi-line string literals.
>> + */
>> +#define NL APR_EOL_STR
>IMHO there is no need to use /** here; that's for extracting API
>documentation from source files, and NL is not going to become part of
>any API.

Ah, I wasn't aware that /** had special meaning.

>Indeed, I'm not sure I would document it at all: if anyone doesn't know
>what APR_EOL_STR is, they can look it up easily enough. And, speaking
>only for myself, the first time I saw APR_EOL_STR in use it was pretty
>obvious what it was for anyway.

Perhaps a more brief comment, then?:

/* Used to terminate lines in large multi-line string literals. */

>> + "This directory holds run-time configuration information for
>> Subversion" NL
>> + "clients. The configuration files all share the same syntax, but you"
>> NL
>> + "should examine a particular file to learn what configuration"
>> NL
>And you'll want to tell your email client not to wrap lines when you
>paste patches....

I *did* mention at the head of the inline patch that the attached gzipped
copy should be used instead for precisely this reason. I don't think my
e-mail client has such an option.

Jonathan Gilbert

To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sun Apr 22 09:11:51 2007

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.