Wireshark-commits: [Wireshark-commits] master ceef87c: CMake: limit -Wextra-semi to C++ only

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: Thu, 06 Dec 2018 23:22:58 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=ceef87c5507388a789af18366b896a88c4a9861b
Submitter: "Jaap Keuter <jaap.keuter@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

ceef87c by Peter Wu (peter@xxxxxxxxxxxxx):

    CMake: limit -Wextra-semi to C++ only
    
    Saves a check and removes a GCC warning from CMakeError.log:
    
        cc1: warning: command line option ‘-Wextra-semi’ is valid for C++/ObjC++ but not for C
    
    Change-Id: I76d027e7b5703b06237ac70eb340c4a09322e09e
    Fixes: v2.9.0rc0-958-g866ddb1b18 ("Add -Wextra-semi to (clang) C and C++ flags to make sure the buildbots will find this.")
    Reviewed-on: https://code.wireshark.org/review/30948
    Petri-Dish: Peter Wu <peter@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: João Valverde <j@xxxxxx>
    Reviewed-by: Jaap Keuter <jaap.keuter@xxxxxxxxx>
    

Actions performed:

    from  247e729   DOCSIS: Added Partial Service Downstream Channels TLV 49.6
     add  ceef87c   CMake: limit -Wextra-semi to C++ only


Summary of changes:
 CMakeLists.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)