Re: Posix compliant CLOCK_PROCESS/THREAD_CPUTIME_ID V4

From: Jesper Juhl
Date: Wed Sep 29 2004 - 14:49:30 EST



Unrelated to the CLOCK_PROCESS/THREAD_CPUTIME_ID discussion, just wanted
to comment on the 'patches inline vs attached' bit.

On Wed, 29 Sep 2004, George Anzinger wrote:

> Christoph Lameter wrote:
> > On Wed, 29 Sep 2004, George Anzinger wrote:
> >
> > > Christoph Lameter wrote:
> > >
> > > Please, when sending patches, attach them. This avoids problems with
> > > mailers,
> > > on both ends, messing with white space. They still appear in line, at
> > > least in
> > > some mailers (mozilla in my case).
> >
> >
> > The custom on lkml, for Linus and Andrew is to send them inline. I also
> > prefer them inline. Will try to remember sending attachments when sending a
> > patch to you.
>
> I think they WILL be inline as well as attached if you attach them. The
> difference is that in both presentations neither mailer will mess with white
> space. This means that long lines will not be wrapped and tabs vs space will
> not be changed.
>
Not all mailers show attachments inline. Mailers that do usually depend on
the mimetype of the attachment when choosing to show inline or not. pine
(my personal favorite) show attachments with a text/plain and similar
mime-type inline, but a not all mailers use that (I see a lot of attached
patches on lkml that don't show inline, and that's somewhat annoying).

It's also harder to reply and comment on bits of a patch when your mailer
does not include attachments inline in a reply (even if it did show them
inline while reading the mail).
Having to save the patch, open it in a text editor and then cut'n'paste
bits of it into the reply mail is a pain. Same goes for having to save &
open it in order to read it in the first place.

--
Jesper Juhl

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/