osmith
/
wireshark
forked from osmocom/wireshark
1
0
Fork 0
Code Pull Requests Releases Wiki Activity

Fix setting COMMON_WARN_FLAGS. 

If we want to add -Wframe-larger-than=16384, *append* it, don't
*overwrite* it.

Indent that code properly while we're at it.

Change-Id: Iae8894e6663500620a07589fd6ba3d00f9eac7ba
Reviewed-on: https://code.wireshark.org/review/22977
Reviewed-by: Guy Harris <guy@alum.mit.edu>
This commit is contained in:
Guy Harris 2017-08-06 15:49:13 -07:00
parent b0b874815d
commit cd7c47e638
1 changed files with 9 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
CMakeLists.txt
View File

@ -451,14 +451,15 @@ else()
-Wno-long-long
-Wheader-guard
)
if((NOT ENABLE_ASAN) AND (NOT ENABLE_UBSAN))
#
# Code that may be worth looking into (coding practices)
#
set(COMMON_WARN_FLAGS
-Wframe-larger-than=16384
)
endif()
if((NOT ENABLE_ASAN) AND (NOT ENABLE_UBSAN))
#
# Code that may be worth looking into (coding practices)
#
set(COMMON_WARN_FLAGS ${COMMON_WARN_FLAGS}
-Wframe-larger-than=16384
)
endif()
set(C_WARN_FLAGS
# The following are C only, not C++