Re: [PATCH 0/4] scripts/checkpatch.pl: A small step for Linux...

[Cruz Julian Bishop <cruzjbishop@gmail.com>]

Ah, the wonders of having no coffee. I only just realized that
I wrote checkpatch instead of checkincludes!

Should I resubmit this patch series, or just leave it since it's
already on the mailing list?

On 31/08/12 23:18, Cruz Julian Bishop wrote:
> ...and a big step for learning more Perl.
>
> Hi!
>
> This patch set started off as an experiment, but as I continued, I
> realised that I should submit this and see if it's suitable for merging
> into Linux.
>
> To start off with, here's some patches for checkincludes.pl
>
> This set was created for the following purposes:
>
>     1: Making it slightly easier to read
>     2: Making it easier to extend and add new arguments
>     3: Changing the usage output to look more "correct"
>     4: Introducing a 'quiet' mode like checkpatch.pl, and printing
>         messages for files without duplicated includes by default
>        
> Sorry if I did anything wrong - It's my first time really
> experimenting with Perl. 
>
> Thanks in advance.
>
> ~Cruz
>
> Cruz Julian Bishop (4):
>   scripts/checkincludes.pl: Print usage with heredoc
>   scripts/checkincludes.pl: Simplify and shorten argument logic
>   scripts/checkincludes.pl: Fix a bug introduced in the last commit
>   scripts/checkincludes.pl: Introduce 'quiet' mode
>
>  scripts/checkincludes.pl |  113 ++++++++++++++++++++++++++--------------------
>  1 file changed, 65 insertions(+), 48 deletions(-)
>