linux-kernel - Re: [PATCH] checkpatch.pl: show how to read from stdin

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <1200073192.29897.95.camel@imap.mvista.com>
Date:	Fri, 11 Jan 2008 09:39:51 -0800
From:	Daniel Walker <dwalker@...sta.com>
To:	Stefan Richter <stefanr@...6.in-berlin.de>
Cc:	Andy Whitcroft <apw@...dowen.org>,
	Bernd Petrovitsch <bernd@...mix.at>,
	linux-kernel@...r.kernel.org, Randy Dunlap <rdunlap@...otime.net>,
	Joel Schopp <jschopp@...tin.ibm.com>,
	Jiri Slaby <jirislaby@...il.com>
Subject: Re: [PATCH] checkpatch.pl: show how to read from stdin


On Fri, 2008-01-11 at 18:29 +0100, Stefan Richter wrote:
> Daniel Walker wrote:
> > On Fri, 2008-01-11 at 18:06 +0100, Stefan Richter wrote:
> >> Signed-off-by: Stefan Richter <stefanr@...6.in-berlin.de>
> >> +	print "When patchfile is -, read standard input.\n";
> 
> > Naww .. Why add documentation when you can just make it do the right
> > thing ..
> 
> Why add code for a feature which already exists?

To make it simpler to use .. A good percentage (if not all) command line
unix data processing utils accept piped data directly, without the need
for the "-" .. That's the expect usage .. checkpatch.pl doesn't need to
deviate from the norm, there is no size constraint, there is no line
limit .. So we're not duplicating functionality, we're making the tool
conform ..

Daniel

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