Re: [PATCH] USB: remove remaining instances of USB_SUSPEND

From: Alan Stern
Date: Thu May 02 2013 - 09:59:07 EST


On Wed, 1 May 2013, Linus Torvalds wrote:

> On Wed, May 1, 2013 at 9:13 AM, Alan Stern <stern@xxxxxxxxxxxxxxxxxxx> wrote:
> >
> > This patch (as1677) removes the remaining instances of that symbol.
>
> Btw, what are these "asXXXX" ID's, and what does the noise buy us?
>
> Doing a
>
> git log | egrep -w 'as[0-9]{3,}'
>
> shows that this has been going on forever, but it still doesn't make
> any *sense*. It adds nothing worthwhile. In fact, doing
>
> git log | grep 'This patch ('
>
> shows that you're pretty the only one doing it, but there's *one*
> other one jidong xiao: jx001. I really don't see the advantage of
> having this kind of random noise in the kernel logs.

I put those indexes in the patch descriptions to help with my own file
organization. Without them, the patch files are just a big mess of
stuff with more or less arbitrary names. The indexes have no meaning
to anybody else.

Originally I put the indexes in the email subject lines. Greg didn't
like this, so at his request I moved them into the patch descriptions.

I suppose they could be moved below the tear line, so that they
wouldn't show up in the final commits but only in the email messages.
That would retain most of their utility for me without bothering anyone
else.

Jidong Xiao put that label in his patch because he was imitating me.

Alan Stern

--
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/