Wireshark-commits: [Wireshark-commits] master 8dabe00: We don't need -Qunused-arguments with Clang
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Wed, 18 Jun 2014 18:58:59 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=8dabe0027a36aa5e2d95ec6ad724faa973cd1ee7
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

8dabe00 by Guy Harris (guy@xxxxxxxxxxxx):

    We don't need -Qunused-arguments with Clang
    
    check_c_compiler_flag now checks for the message Clang prints for -f and
    -m flags it doesn't handle ("argument unused during compilation:
    '-{flag}'"), so the checks for it now should fail properly during
    testing, causing us not to use the flag in question.  This means we
    don't need to suppress that warning, as we shouldn't be getting it.
    
    Change-Id: Ieb9657f9e2cee2f357acd52725199d78d2dad80f
    Reviewed-on: https://code.wireshark.org/review/2401
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  c6007e8   We don't need -Werror=unknown-warning-option with Clang.
    adds  8dabe00   We don't need -Qunused-arguments with Clang


Summary of changes:
 CMakeLists.txt |    4 ----
 1 file changed, 4 deletions(-)