Wireshark-commits: [Wireshark-commits] master 5f0d506: Use -Werror except for explicitly listed dir

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sat, 17 Feb 2018 01:52:10 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=5f0d50690ae01427ae310e15989cbe3780a241ad
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

5f0d506 by Guy Harris (guy@xxxxxxxxxxxx):

    Use -Werror except for explicitly listed dirty dissectors.
    
    Use AM_CFLAGS for everything except for libdirtydissectors in
    epan/dissectors.  Rename GENERATED_CFLAGS/GENERATED_CXXFLAGS to
    DIRTY_CFLAGS/DIRTY_CXXFLAGS, as it doesn't apply to all generated files.
    
    Change-Id: I702b53e185d6972c08d68ef31c05df7b03669daa
    Reviewed-on: https://code.wireshark.org/review/25829
    Petri-Dish: Guy Harris <guy@xxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  0ac5c79   Don't use GENERATED_CFLAGS on generated files, use AM_CFLAGS.
    adds  5f0d506   Use -Werror except for explicitly listed dirty dissectors.


Summary of changes:
 Makefile.am.inc                     | 4 ++--
 epan/dfilter/Makefile.am            | 2 +-
 epan/dissectors/Makefile.am         | 2 +-
 plugins/epan/mate/Makefile.am       | 2 +-
 plugins/epan/wimaxasncp/Makefile.am | 2 +-
 ui/Makefile.am                      | 2 +-
 wiretap/Makefile.am                 | 2 +-
 7 files changed, 8 insertions(+), 8 deletions(-)