From: maximilian attems <janitor@sternwelten.at>
To: kernel-janitors@vger.kernel.org
Subject: Re: [Kernel-janitors] [PATCH] remove check_region specialix
Date: Mon, 12 Jul 2004 16:42:16 +0000 [thread overview]
Message-ID: <20040712164216.GE1821@sputnik.stro.at> (raw)
In-Reply-To: <200407072024.i67KOdQM017410@sirius.cs.pdx.edu>
[-- Attachment #1: Type: text/plain, Size: 878 bytes --]
On Mon, 12 Jul 2004, Kristen Carlson wrote:
> Thanks! For my benefit, what's the easiest way to check for trailing spaces?
> I thought I had gotten all of them by just looking at the patch and noting
> which lines shouldn't have needed modifications but I obviously missed some,
> especially on the lines that I was modifying anyway. Do you have a more
> foolproof method?
> Kristen
Domen posted that one on kj (bonus detects also spaces instead of tabs):
egrep -n "^\+(.* $| )"
in vim you may want to :set list for viewing those spaces
and afterwards :set nolist.
another possibility is to use the patch-scripts from Andrew Morton,
they warn you when trailing space is introduced.
that's the place, where i see most of them ;)
-> http://developer.osdl.org/rddunlap/patch-scripts/
or latest http://www.zip.com.au/~akpm/linux/patches/patch-scripts-0.18/
a++ maks
[-- Attachment #2: Type: text/plain, Size: 167 bytes --]
_______________________________________________
Kernel-janitors mailing list
Kernel-janitors@lists.osdl.org
http://lists.osdl.org/mailman/listinfo/kernel-janitors
next prev parent reply other threads:[~2004-07-12 16:42 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-07-07 20:24 [Kernel-janitors] [PATCH] remove check_region specialix Kristen Carlson
2004-07-07 20:40 ` Matthew Wilcox
2004-07-07 21:05 ` Kristen Carlson
2004-07-07 21:28 ` Michael Veeck
2004-07-07 21:59 ` Kristen Carlson
2004-07-11 13:15 ` maximilian attems
2004-07-12 16:11 ` Kristen Carlson
2004-07-12 16:30 ` Roger Luethi
2004-07-12 16:42 ` maximilian attems [this message]
2004-07-12 21:14 ` [Kernel-janitors] [PATCH] remove check_region from eepro.c Kristen Carlson
2004-07-13 1:55 ` [Kernel-janitors] [PATCH] remove check_region specialix Randy.Dunlap
2004-07-13 10:53 ` Roger Luethi
2004-08-14 14:15 ` [Kernel-janitors] [PATCH] remove check_region from eepro.c maximilian attems
2004-08-14 20:06 ` [Kernel-janitors] [PATCH] remove check_region specialix maximilian attems
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20040712164216.GE1821@sputnik.stro.at \
--to=janitor@sternwelten.at \
--cc=kernel-janitors@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.