Wireshark-commits: [Wireshark-commits] master 866ddb1: Add -Wextra-semi to (clang) C and C++ flags

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: Fri, 15 Jun 2018 11:17:47 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=866ddb1b1808ec24610bd493b3ca57e47a7f2148
Submitter: Jörg Mayer (jmayer@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

866ddb1 by Joerg Mayer (jmayer@xxxxxxxxx):

    Add -Wextra-semi to (clang) C and C++ flags to make sure the buildbots will find this.
    
    Fix all warnings that come up.
    
    Change-Id: Ib426e79f7c75152589a3c2af153de35ca5d63783
    Reviewed-on: https://code.wireshark.org/review/28268
    Petri-Dish: Jörg Mayer <jmayer@xxxxxxxxx>
    Petri-Dish: Anders Broman <a.broman58@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Jörg Mayer <jmayer@xxxxxxxxx>
    

Actions performed:

    from  8b87a86   NAS 5GS: Add dissection of Security header.
    adds  866ddb1   Add -Wextra-semi to (clang) C and C++ flags to make sure the buildbots will find this.


Summary of changes:
 CMakeLists.txt                         |  1 +
 ui/qt/manager/wireshark_preference.cpp | 30 +++++++++++++++---------------
 2 files changed, 16 insertions(+), 15 deletions(-)