Re: [PATCH 2.6.20 1/1] usb/input updated aiptek tablet driver

Pete Zaitcev <[email protected]>
Newsgroups gmane.linux.usb.devel
Organization Red Hat, Inc.
Message-ID <[email protected]>
On Thu, 8 Mar 2007 00:02:48 +0100, "Rene van Paassen" <[email protected]> wrote:

I don't mind this:

> Signed-off-by: René van Paassen <[email protected]>

This non-ASCII character is in the changelog, so it won't cause any
spurious patch rejections. But this is bad:

> + *  Extra bits    2004      René van Paassen <[email protected]>
> +#define DRIVER_AUTHOR  "Bryan W. Headley/Chris Atenasio/Cédric
> Brun/René van Paassen"

Please understand, patches are commonly sent naked in the mail, even
in this enlightened day of git and MIME. So, even if you send it right,
eventually someone else will patch your code, diff context will
catch the name, and...

Your patch was line-wrapped, so you'll need to resend it anyway.
While addressing that, please consider my request too.

Yours,
-- Pete

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
[email protected]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.