The patch titled
checkincludes.pl: close file as soon as we're done with it
has been added to the -mm tree. Its filename is
checkincludespl-close-file-as-soon-as-were-done-with-it.patch
Before you just go and hit "reply", please:
a) Consider who else should be cc'ed
b) Prefer to cc a suitable mailing list as well
c) Ideally: find the original patch on the mailing list and do a
reply-to-all to that, adding suitable additional cc's
*** Remember to use Documentation/SubmitChecklist when testing your code ***
See http://userweb.kernel.org/~akpm/stuff/added-to-mm.txt to find
out what to do about this
The current -mm tree may be found at http://userweb.kernel.org/~akpm/mmotm/
------------------------------------------------------
Subject: checkincludes.pl: close file as soon as we're done with it
From: "Luis R. Rodriguez" <lrodriguez@xxxxxxxxxxx>
Signed-off-by: Luis R. Rodriguez <lrodriguez@xxxxxxxxxxx>
Cc: Sam Ravnborg <sam@xxxxxxxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
---
scripts/checkincludes.pl | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff -puN scripts/checkincludes.pl~checkincludespl-close-file-as-soon-as-were-done-with-it scripts/checkincludes.pl
--- a/scripts/checkincludes.pl~checkincludespl-close-file-as-soon-as-were-done-with-it
+++ a/scripts/checkincludes.pl
@@ -13,12 +13,12 @@ foreach $file (@ARGV) {
++$includedfiles{$1};
}
}
+
+ close(FILE);
foreach $filename (keys %includedfiles) {
if ($includedfiles{$filename} > 1) {
print "$file: $filename is included more than once.\n";
}
}
-
- close(FILE);
}
_
Patches currently in -mm which might be from lrodriguez@xxxxxxxxxxx are
checkincludespl-close-file-as-soon-as-were-done-with-it.patch
checkincludespl-provide-usage-helper.patch
checkincludespl-add-option-to-remove-duplicates-in-place.patch
checkincludespl-add-option-to-remove-duplicates-in-place-fix.patch
--
To unsubscribe from this list: send the line "unsubscribe mm-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
