public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Cruz Julian Bishop <cruzjbishop@gmail.com>
To: mcgrof@gmail.com
Cc: linux-kernel@vger.kernel.org
Subject: Re: [PATCH 5/5] scripts/checkincludes.pl: Replace tabs with spaces
Date: Tue, 04 Sep 2012 09:12:52 +1000	[thread overview]
Message-ID: <1346713972.2312.2.camel@Cruz-PC> (raw)
In-Reply-To: <1346710853-19166-6-git-send-email-cruzjbishop@gmail.com>

My bad, misread the documentation files.

I'll resubmit without this

On Tue, 2012-09-04 at 08:20 +1000, Cruz Julian Bishop wrote:
> This is required by the CodingStyle kernel documentation file
> 
> Signed-off-by: Cruz Julian Bishop <cruzjbishop@gmail.com>
> ---
>  scripts/checkincludes.pl | 122 +++++++++++++++++++++++------------------------
>  1 file changed, 61 insertions(+), 61 deletions(-)
> 
> diff --git a/scripts/checkincludes.pl b/scripts/checkincludes.pl
> index bff7316..7e3df9e 100755
> --- a/scripts/checkincludes.pl
> +++ b/scripts/checkincludes.pl
> @@ -14,7 +14,7 @@
>  use strict;
>  
>  sub usage {
> -	print <<EOM;
> +        print <<EOM;
>  Usage: checkincludes.pl [OPTIONS]... FILE...
>  By default, we just warn of duplicates.
>  
> @@ -23,84 +23,84 @@ Options:
>    -q                Do not show messages for files that do not
>                      have duplicated includes
>  EOM
> -	exit 1;
> +        exit 1;
>  }
>  
>  my $remove = 0;
>  my $quiet = 0;
>  
>  if ($#ARGV < 0) {
> -	usage();
> +        usage();
>  }
>  
>  my $argc = 0;
>  
>  foreach my $arg (@ARGV) {
> -	if ($arg eq "-r") {
> -		$remove = 1;
> -	} elsif($arg eq "-q") {
> -		$quiet = 1;
> -	} else {
> -		open(my $f, '<', $arg)
> -		    or die "Cannot open $arg: $!.\n";
> -		    
> -		$argc++;
> +        if ($arg eq "-r") {
> +                $remove = 1;
> +        } elsif($arg eq "-q") {
> +                $quiet = 1;
> +        } else {
> +                open(my $f, '<', $arg)
> +                        or die "Cannot open $arg: $!.\n";
> +                    
> +                $argc++;
>  
> -		my %includedfiles = ();
> -		my @file_lines = ();
> +                my %includedfiles = ();
> +                my @file_lines = ();
>  
> -		while (<$f>) {
> -			if (m/^\s*#\s*include\s*[<"](\S*)[>"]/o) {
> -				++$includedfiles{$1};
> -			}
> -			push(@file_lines, $_);
> -		}
> +                while (<$f>) {
> +                        if (m/^\s*#\s*include\s*[<"](\S*)[>"]/o) {
> +                                ++$includedfiles{$1};
> +                        }
> +                        push(@file_lines, $_);
> +                }
>  
> -		close($f);
> +                close($f);
>  
> -		if (!$remove) {
> -			my $detected = 0;
> -			foreach my $filename (keys %includedfiles) {
> -				if ($includedfiles{$filename} > 1) {
> -					$detected++;
> -					print "$arg: $filename is included more than once.\n";
> -				}
> -			}
> -			if (!$detected && !$quiet) {
> -				print "$arg: No duplicated includes detected.\n";
> -			}
> -			next;
> -		}
> +                if (!$remove) {
> +                        my $detected = 0;
> +                        foreach my $filename (keys %includedfiles) {
> +                                if ($includedfiles{$filename} > 1) {
> +                                        $detected++;
> +                                print "$arg: $filename is included more than once.\n";
> +                                }
> +                        }
> +                        if (!$detected && !$quiet) {
> +                                print "$arg: No duplicated includes detected.\n";
> +                        }
> +                        next;
> +                }
>  
> -		open($f, '>', $arg)
> -			or die("Cannot write to $arg: $!");
> +                open($f, '>', $arg)
> +                        or die("Cannot write to $arg: $!");
>  
> -		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 "$arg: removed $dups duplicate includes \n";
> -		} elsif (!$quiet) {
> -			print "$arg: no duplicated includes to remove \n";
> -		}
> -		close($f);
> -	}
> +                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 "$arg: removed $dups duplicate includes \n";
> +                } elsif (!$quiet) {
> +                        print "$arg: no duplicated includes to remove \n";
> +                }
> +                close($f);
> +        }
>  }
>  
>  if ($argc == 0) {
> -	usage();
> +        usage();
>  }



      parent reply	other threads:[~2012-09-03 23:13 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1346710853-19166-1-git-send-email-cruzjbishop@gmail.com>
     [not found] ` <1346710853-19166-3-git-send-email-cruzjbishop@gmail.com>
2012-09-03 22:29   ` [PATCH 2/5] scripts/checkincludes.pl: Simplify and shorten argument logic Cruz Julian Bishop
     [not found] ` <1346710853-19166-6-git-send-email-cruzjbishop@gmail.com>
2012-09-03 23:12   ` Cruz Julian Bishop [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1346713972.2312.2.camel@Cruz-PC \
    --to=cruzjbishop@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mcgrof@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox