[coreboot] New patch to review for coreboot: ce4abe3 lint: Add test for whitespace issues in the code

Patrick Georgi (patrick@georgi-clan.de) gerrit at coreboot.org
Fri Feb 17 10:49:26 CET 2012


Patrick Georgi (patrick at georgi-clan.de) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/645

-gerrit

commit ce4abe3fa98283a400b63760750e4df04e0ae204
Author: Patrick Georgi <patrick at georgi-clan.de>
Date:   Thu Feb 16 20:28:42 2012 +0100

    lint: Add test for whitespace issues in the code
    
    So far it tests for trailing whitespace.
    "Upstream" files (bison/flex's .?_shipped, kconfig, vendorcode) are ignored.
    
    Change-Id: I7af1954d537fd05f06cd210ac130dac87892159b
    Signed-off-by: Patrick Georgi <patrick at georgi-clan.de>
---
 util/lint/lint-003-whitespace |   39 +++++++++++++++++++++++++++++++++++++++
 1 files changed, 39 insertions(+), 0 deletions(-)

diff --git a/util/lint/lint-003-whitespace b/util/lint/lint-003-whitespace
new file mode 100755
index 0000000..fe305bf
--- /dev/null
+++ b/util/lint/lint-003-whitespace
@@ -0,0 +1,39 @@
+#!/bin/sh
+# This file is part of the coreboot project.
+#
+# Copyright (C) 2011 Patrick Georgi <patrick at georgi-clan.de>
+#
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; version 2 of the License.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program; if not, write to the Free Software
+# Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
+#
+# DESCR: Check for superfluous whitespace in the tree
+
+LC_ALL=C export LC_ALL
+find src util -name .svn -type d -prune -o \
+	-name .git -type d -prune -o \
+	-name README -prune -o \
+	-name LICENSE -prune -o \
+	-name TODO -prune -o \
+	-name COPYING -prune -o \
+	-name \*.txt -prune -o \
+	-name microcode-\*.h -prune -o \
+	-name \*.?_shipped -prune -o \
+	-name \*.[18] -prune -o \
+	-name kconfig -type d -prune -o \
+	-name romcc -type d -prune -o \
+	-name crossgcc -type d -prune -o \
+	-name vendorcode -type d -prune -o \
+	-type f -exec \
+		grep -l "[[:space:]][[:space:]]*$" {} + | \
+	sed -e "s,^.*$,File & has lines ending with whitespace.,"
+




More information about the coreboot mailing list