qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] checkpatch: Don't WARN about missing spaces in


From: malc
Subject: Re: [Qemu-devel] [PATCH] checkpatch: Don't WARN about missing spaces in audio files
Date: Thu, 9 Feb 2012 20:43:05 +0400 (MSK)
User-agent: Alpine 2.00 (LNX 1167 2008-08-23)

On Thu, 9 Feb 2012, Andreas F?rber wrote:

> Disable warnings for spaces before opening parenthesis in
> hw/{ac97,adlib,cs4231a,es1370,gus,sb16}.c.
> 
> Signed-off-by: Andreas F?rber <address@hidden>
> Cc: Blue Swirl <address@hidden>
> Cc: malc <address@hidden>

Thanks.

> ---
>  scripts/checkpatch.pl |    5 +++++
>  1 files changed, 5 insertions(+), 0 deletions(-)
> 
> diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
> index 8850a5f..5433736 100755
> --- a/scripts/checkpatch.pl
> +++ b/scripts/checkpatch.pl
> @@ -246,6 +246,8 @@ our $allowed_asm_includes = qr{(?x:
>  )};
>  # memory.h: ARM has a custom one
>  
> +our $audio_files = 
> qr{hw/ac97.c|hw/adlib.c|hw/cs4231a.c|hw/es1370.c|hw/gus.c|hw/sb16.c};

My perl is not up to snuff so i wouldn't know how to add audio/* to the
mix.

> +
>  sub build_types {
>       my $mods = "(?x:  \n" . join("|\n  ", @modifierList) . "\n)";
>       my $all = "(?x:  \n" . join("|\n  ", @typeList) . "\n)";
> @@ -1966,6 +1968,9 @@ sub process {
>                               asm|__asm__)$/x)
>                       {
>  
> +                     # malc wants to keep spacing consistent in the audio 
> files.
> +                     } elsif ($realfile =~ /($audio_files)/) {
> +
>                       # cpp #define statements have non-optional spaces, ie
>                       # if there is a space between the name and the open
>                       # parenthesis it is simply not a parameter group.
> 

-- 
mailto:address@hidden



reply via email to

[Prev in Thread] Current Thread [Next in Thread]