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