Finn Behrens | 2d1f7c9 | 2020-11-23 15:15:33 +0100 | [diff] [blame] | 1 | #!/usr/bin/env perl |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 2 | # |
Luis R. Rodriguez | 92f3f19 | 2009-09-18 12:49:27 -0700 | [diff] [blame] | 3 | # checkincludes: find/remove files included more than once |
| 4 | # |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 5 | # Copyright abandoned, 2000, Niels Kristian Bech Jensen <nkbj@image.dk>. |
Luis R. Rodriguez | 92f3f19 | 2009-09-18 12:49:27 -0700 | [diff] [blame] | 6 | # Copyright 2009 Luis R. Rodriguez <mcgrof@gmail.com> |
| 7 | # |
| 8 | # This script checks for duplicate includes. It also has support |
| 9 | # to remove them in place. Note that this will not take into |
| 10 | # consideration macros so you should run this only if you know |
| 11 | # you do have real dups and do not have them under #ifdef's. You |
| 12 | # could also just review the results. |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 13 | |
Stephen Hemminger | 3da2715 | 2010-02-22 15:17:12 -0800 | [diff] [blame] | 14 | use strict; |
| 15 | |
Luis R. Rodriguez | f9d490a | 2009-09-18 12:49:26 -0700 | [diff] [blame] | 16 | sub usage { |
Luis R. Rodriguez | 92f3f19 | 2009-09-18 12:49:27 -0700 | [diff] [blame] | 17 | print "Usage: checkincludes.pl [-r]\n"; |
| 18 | print "By default we just warn of duplicates\n"; |
| 19 | print "To remove duplicated includes in place use -r\n"; |
Luis R. Rodriguez | f9d490a | 2009-09-18 12:49:26 -0700 | [diff] [blame] | 20 | exit 1; |
| 21 | } |
| 22 | |
Luis R. Rodriguez | 92f3f19 | 2009-09-18 12:49:27 -0700 | [diff] [blame] | 23 | my $remove = 0; |
| 24 | |
Luis R. Rodriguez | f9d490a | 2009-09-18 12:49:26 -0700 | [diff] [blame] | 25 | if ($#ARGV < 0) { |
Luis R. Rodriguez | 92f3f19 | 2009-09-18 12:49:27 -0700 | [diff] [blame] | 26 | usage(); |
| 27 | } |
| 28 | |
| 29 | if ($#ARGV >= 1) { |
| 30 | if ($ARGV[0] =~ /^-/) { |
| 31 | if ($ARGV[0] eq "-r") { |
| 32 | $remove = 1; |
| 33 | shift; |
| 34 | } else { |
| 35 | usage(); |
| 36 | } |
| 37 | } |
Luis R. Rodriguez | f9d490a | 2009-09-18 12:49:26 -0700 | [diff] [blame] | 38 | } |
| 39 | |
Stephen Hemminger | 3da2715 | 2010-02-22 15:17:12 -0800 | [diff] [blame] | 40 | foreach my $file (@ARGV) { |
| 41 | open(my $f, '<', $file) |
| 42 | or die "Cannot open $file: $!.\n"; |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 43 | |
| 44 | my %includedfiles = (); |
Luis R. Rodriguez | 92f3f19 | 2009-09-18 12:49:27 -0700 | [diff] [blame] | 45 | my @file_lines = (); |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 46 | |
Stephen Hemminger | 3da2715 | 2010-02-22 15:17:12 -0800 | [diff] [blame] | 47 | while (<$f>) { |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 48 | if (m/^\s*#\s*include\s*[<"](\S*)[>"]/o) { |
| 49 | ++$includedfiles{$1}; |
| 50 | } |
Luis R. Rodriguez | 92f3f19 | 2009-09-18 12:49:27 -0700 | [diff] [blame] | 51 | push(@file_lines, $_); |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 52 | } |
Luis R. Rodriguez | d9a7a2b | 2009-09-18 12:49:25 -0700 | [diff] [blame] | 53 | |
Stephen Hemminger | 3da2715 | 2010-02-22 15:17:12 -0800 | [diff] [blame] | 54 | close($f); |
Luis R. Rodriguez | 92f3f19 | 2009-09-18 12:49:27 -0700 | [diff] [blame] | 55 | |
| 56 | if (!$remove) { |
Stephen Hemminger | 3da2715 | 2010-02-22 15:17:12 -0800 | [diff] [blame] | 57 | foreach my $filename (keys %includedfiles) { |
Luis R. Rodriguez | 92f3f19 | 2009-09-18 12:49:27 -0700 | [diff] [blame] | 58 | if ($includedfiles{$filename} > 1) { |
| 59 | print "$file: $filename is included more than once.\n"; |
| 60 | } |
| 61 | } |
| 62 | next; |
| 63 | } |
| 64 | |
Stephen Hemminger | 3da2715 | 2010-02-22 15:17:12 -0800 | [diff] [blame] | 65 | open($f, '>', $file) |
| 66 | or die("Cannot write to $file: $!"); |
Luis R. Rodriguez | 92f3f19 | 2009-09-18 12:49:27 -0700 | [diff] [blame] | 67 | |
| 68 | my $dups = 0; |
| 69 | foreach (@file_lines) { |
| 70 | if (m/^\s*#\s*include\s*[<"](\S*)[>"]/o) { |
Stephen Hemminger | 3da2715 | 2010-02-22 15:17:12 -0800 | [diff] [blame] | 71 | foreach my $filename (keys %includedfiles) { |
Luis R. Rodriguez | 92f3f19 | 2009-09-18 12:49:27 -0700 | [diff] [blame] | 72 | if ($1 eq $filename) { |
| 73 | if ($includedfiles{$filename} > 1) { |
| 74 | $includedfiles{$filename}--; |
| 75 | $dups++; |
| 76 | } else { |
Stephen Hemminger | 3da2715 | 2010-02-22 15:17:12 -0800 | [diff] [blame] | 77 | print {$f} $_; |
Luis R. Rodriguez | 92f3f19 | 2009-09-18 12:49:27 -0700 | [diff] [blame] | 78 | } |
| 79 | } |
| 80 | } |
| 81 | } else { |
Stephen Hemminger | 3da2715 | 2010-02-22 15:17:12 -0800 | [diff] [blame] | 82 | print {$f} $_; |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 83 | } |
| 84 | } |
Luis R. Rodriguez | 92f3f19 | 2009-09-18 12:49:27 -0700 | [diff] [blame] | 85 | if ($dups > 0) { |
| 86 | print "$file: removed $dups duplicate includes\n"; |
| 87 | } |
Stephen Hemminger | 3da2715 | 2010-02-22 15:17:12 -0800 | [diff] [blame] | 88 | close($f); |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 89 | } |