diff options
author | Peter Wu <peter@lekensteyn.nl> | 2015-11-26 11:26:13 +0100 |
---|---|---|
committer | Anders Broman <a.broman58@gmail.com> | 2015-11-27 08:56:56 +0000 |
commit | 20bfc3d5a9a44132e5cd1ffc3d9364dba7824574 (patch) | |
tree | eb16997cd54eb659bc68ad6b376151147ab7bbd3 /caputils | |
parent | c986188daed7cf1acf754243b7eebe80ee430bda (diff) | |
download | wireshark-20bfc3d5a9a44132e5cd1ffc3d9364dba7824574.tar.gz |
autotools: do not apply CFLAGS in reverse order
Fixes false "-Wvariadic-macros" because "-Wpedantic" came after
"-Wno-variadic-macros". While at it, avoid (unintentionally?) adding
-fPIE to all C++ programs (via CXXFLAGS).
Availability of flags is checked by appending a flag (in case the user
has something like CFLAGS=-Wno-error).
This removes a -fPIE check for CXXFLAGS and removes 5
-Woverlength-strings and 9 -Wvariadic-macros warnings from make with
Clang 3.7.0. configure and compile times were equal. (A diff between the
configure outputs showed no other changes.)
This reverts commit cf0d762d7304aa569ea25faf999c74bbe94f9023 and applies
a different approach.
Ping-Bug: 10791
Change-Id: Ic7b4137e2d98b06bc7625091be9bc7dd69182586
Reviewed-on: https://code.wireshark.org/review/12175
Petri-Dish: Peter Wu <peter@lekensteyn.nl>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
(cherry picked from commit 470c745a7a0a5b930bdfb0edd8874436cb9beea0)
Reviewed-on: https://code.wireshark.org/review/12186
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'caputils')
0 files changed, 0 insertions, 0 deletions