Turn on strict checking. Use local file handles. Use three argument open. Signed-off-by: Stephen Hemminger <shemminger@vyatta.com> Cc: Cong Wang <amwang@redhat.com> Cc: Michal Marek <mmarek@suse.cz> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Michal Marek <mmarek@suse.cz>
		
			
				
	
	
		
			90 lines
		
	
	
		
			1.8 KiB
		
	
	
	
		
			Perl
		
	
	
		
			Executable File
		
	
	
	
	
			
		
		
	
	
			90 lines
		
	
	
		
			1.8 KiB
		
	
	
	
		
			Perl
		
	
	
		
			Executable File
		
	
	
	
	
| #!/usr/bin/perl
 | |
| #
 | |
| # checkincludes: find/remove files included more than once
 | |
| #
 | |
| # Copyright abandoned, 2000, Niels Kristian Bech Jensen <nkbj@image.dk>.
 | |
| # Copyright 2009 Luis R. Rodriguez <mcgrof@gmail.com>
 | |
| #
 | |
| # This script checks for duplicate includes. It also has support
 | |
| # to remove them in place. Note that this will not take into
 | |
| # consideration macros so you should run this only if you know
 | |
| # you do have real dups and do not have them under #ifdef's. You
 | |
| # could also just review the results.
 | |
| 
 | |
| use strict;
 | |
| 
 | |
| sub usage {
 | |
| 	print "Usage: checkincludes.pl [-r]\n";
 | |
| 	print "By default we just warn of duplicates\n";
 | |
| 	print "To remove duplicated includes in place use -r\n";
 | |
| 	exit 1;
 | |
| }
 | |
| 
 | |
| my $remove = 0;
 | |
| 
 | |
| if ($#ARGV < 0) {
 | |
| 	usage();
 | |
| }
 | |
| 
 | |
| if ($#ARGV >= 1) {
 | |
| 	if ($ARGV[0] =~ /^-/) {
 | |
| 		if ($ARGV[0] eq "-r") {
 | |
| 			$remove = 1;
 | |
| 			shift;
 | |
| 		} else {
 | |
| 			usage();
 | |
| 		}
 | |
| 	}
 | |
| }
 | |
| 
 | |
| foreach my $file (@ARGV) {
 | |
| 	open(my $f, '<', $file)
 | |
| 	    or die "Cannot open $file: $!.\n";
 | |
| 
 | |
| 	my %includedfiles = ();
 | |
| 	my @file_lines = ();
 | |
| 
 | |
| 	while (<$f>) {
 | |
| 		if (m/^\s*#\s*include\s*[<"](\S*)[>"]/o) {
 | |
| 			++$includedfiles{$1};
 | |
| 		}
 | |
| 		push(@file_lines, $_);
 | |
| 	}
 | |
| 
 | |
| 	close($f);
 | |
| 
 | |
| 	if (!$remove) {
 | |
| 		foreach my $filename (keys %includedfiles) {
 | |
| 			if ($includedfiles{$filename} > 1) {
 | |
| 				print "$file: $filename is included more than once.\n";
 | |
| 			}
 | |
| 		}
 | |
| 		next;
 | |
| 	}
 | |
| 
 | |
| 	open($f, '>', $file)
 | |
| 	    or die("Cannot write to $file: $!");
 | |
| 
 | |
| 	my $dups = 0;
 | |
| 	foreach (@file_lines) {
 | |
| 		if (m/^\s*#\s*include\s*[<"](\S*)[>"]/o) {
 | |
| 			foreach my $filename (keys %includedfiles) {
 | |
| 				if ($1 eq $filename) {
 | |
| 					if ($includedfiles{$filename} > 1) {
 | |
| 						$includedfiles{$filename}--;
 | |
| 						$dups++;
 | |
| 					} else {
 | |
| 						print {$f} $_;
 | |
| 					}
 | |
| 				}
 | |
| 			}
 | |
| 		} else {
 | |
| 			print {$f} $_;
 | |
| 		}
 | |
| 	}
 | |
| 	if ($dups > 0) {
 | |
| 		print "$file: removed $dups duplicate includes\n";
 | |
| 	}
 | |
| 	close($f);
 | |
| }
 |