lkml.org 
[lkml]   [2012]   [Apr]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 03/13] tools: Cleanup EXTRA_WARNINGS
    Date
    From: Borislav Petkov <borislav.petkov@amd.com>

    Use += instead of the bash syntax, as Sam Ravnborg suggests. Also, sort
    the -W options alphabetically and (... keep them sorted).

    Suggested-by: Sam Ravnborg <sam@ravnborg.org>
    Signed-off-by: Borislav Petkov <borislav.petkov@amd.com>
    Cc: Ingo Molnar <mingo@kernel.org>
    Cc: Michal Marek <mmarek@suse.cz>
    Cc: Sam Ravnborg <sam@ravnborg.org>
    Link: http://lkml.kernel.org/r/1334162178-17152-3-git-send-email-bp@amd64.org
    Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
    ---
    tools/scripts/Makefile.include | 40 ++++++++++++++++++++--------------------
    1 file changed, 20 insertions(+), 20 deletions(-)

    diff --git a/tools/scripts/Makefile.include b/tools/scripts/Makefile.include
    index 52348d3..87b55a7 100644
    --- a/tools/scripts/Makefile.include
    +++ b/tools/scripts/Makefile.include
    @@ -11,26 +11,26 @@ endif
    #
    # Include saner warnings here, which can catch bugs:
    #
    -EXTRA_WARNINGS := -Wformat
    -EXTRA_WARNINGS := $(EXTRA_WARNINGS) -Wformat-security
    -EXTRA_WARNINGS := $(EXTRA_WARNINGS) -Wformat-y2k
    -EXTRA_WARNINGS := $(EXTRA_WARNINGS) -Wshadow
    -EXTRA_WARNINGS := $(EXTRA_WARNINGS) -Winit-self
    -EXTRA_WARNINGS := $(EXTRA_WARNINGS) -Wpacked
    -EXTRA_WARNINGS := $(EXTRA_WARNINGS) -Wredundant-decls
    -EXTRA_WARNINGS := $(EXTRA_WARNINGS) -Wstrict-aliasing=3
    -EXTRA_WARNINGS := $(EXTRA_WARNINGS) -Wswitch-default
    -EXTRA_WARNINGS := $(EXTRA_WARNINGS) -Wswitch-enum
    -EXTRA_WARNINGS := $(EXTRA_WARNINGS) -Wno-system-headers
    -EXTRA_WARNINGS := $(EXTRA_WARNINGS) -Wundef
    -EXTRA_WARNINGS := $(EXTRA_WARNINGS) -Wwrite-strings
    -EXTRA_WARNINGS := $(EXTRA_WARNINGS) -Wbad-function-cast
    -EXTRA_WARNINGS := $(EXTRA_WARNINGS) -Wmissing-declarations
    -EXTRA_WARNINGS := $(EXTRA_WARNINGS) -Wmissing-prototypes
    -EXTRA_WARNINGS := $(EXTRA_WARNINGS) -Wnested-externs
    -EXTRA_WARNINGS := $(EXTRA_WARNINGS) -Wold-style-definition
    -EXTRA_WARNINGS := $(EXTRA_WARNINGS) -Wstrict-prototypes
    -EXTRA_WARNINGS := $(EXTRA_WARNINGS) -Wdeclaration-after-statement
    +EXTRA_WARNINGS := -Wbad-function-cast
    +EXTRA_WARNINGS += -Wdeclaration-after-statement
    +EXTRA_WARNINGS += -Wformat-security
    +EXTRA_WARNINGS += -Wformat-y2k
    +EXTRA_WARNINGS += -Winit-self
    +EXTRA_WARNINGS += -Wmissing-declarations
    +EXTRA_WARNINGS += -Wmissing-prototypes
    +EXTRA_WARNINGS += -Wnested-externs
    +EXTRA_WARNINGS += -Wno-system-headers
    +EXTRA_WARNINGS += -Wold-style-definition
    +EXTRA_WARNINGS += -Wpacked
    +EXTRA_WARNINGS += -Wredundant-decls
    +EXTRA_WARNINGS += -Wshadow
    +EXTRA_WARNINGS += -Wstrict-aliasing=3
    +EXTRA_WARNINGS += -Wstrict-prototypes
    +EXTRA_WARNINGS += -Wswitch-default
    +EXTRA_WARNINGS += -Wswitch-enum
    +EXTRA_WARNINGS += -Wundef
    +EXTRA_WARNINGS += -Wwrite-strings
    +EXTRA_WARNINGS += -Wformat

    ifneq ($(findstring $(MAKEFLAGS), w),w)
    PRINT_DIR = --no-print-directory
    --
    1.7.9.2.358.g22243


    \
     
     \ /
      Last update: 2012-04-11 23:09    [W:0.023 / U:59.156 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site