[Jonathan Gilbert]
> >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. */
*nod*
> >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.
Fair enough. The most common workaround for that problem is to attach
patches uncompressed, as regular text files (MIME type text/plain).
That way you don't need to paste a separate copy, as we will see the
attachment inline. To encourage an email client to label the file as
text/plain, if you don't have direct control over this, give it a name
of "something.txt".
Received on Sun Apr 22 09:39:57 2007