Merge lp:~klaus-halfmann/widelands/feature-NoClangWarn into lp:widelands

Proposed by Klaus Halfmann
Status: Merged
Merged at revision: 7724
Proposed branch: lp:~klaus-halfmann/widelands/feature-NoClangWarn
Merge into: lp:widelands
Diff against target: 10 lines (+1/-0)
1 file modified
CMakeLists.txt (+1/-0)
To merge this branch: bzr merge lp:~klaus-halfmann/widelands/feature-NoClangWarn
Reviewer Review Type Date Requested Status
SirVer Approve
Review via email: mp+283521@code.launchpad.net

Description of the change

Added -Wno-documentation for clang to suppress a lot of warnings

To post a comment you must log in.
Revision history for this message
bunnybot (widelandsofficial) wrote :

Hi, I am bunnybot (https://github.com/widelands/bunnybot).

I am keeping the source branch lp:~klaus-halfmann/widelands/feature-NoClangWarn mirrored to https://github.com/widelands/widelands/tree/_klaus_halfmann_widelands_feature_NoClangWarn

You can give me commands by starting a line with @bunnybot <command>. I understand:
 merge: Merges the source branch into the target branch, closing the merge proposal. I will use the proposed commit message if it is set.

Revision history for this message
SirVer (sirver) wrote :

Just a nit (see inline comment), please merge once addressed.

review: Approve
Revision history for this message
bunnybot (widelandsofficial) wrote :

Travis build 328 has changed state to: passed. Details: https://travis-ci.org/widelands/widelands/builds/103907202.

Revision history for this message
GunChleoc (gunchleoc) wrote :

@bunnybot merge

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'CMakeLists.txt'
2--- CMakeLists.txt 2016-01-11 20:25:48 +0000
3+++ CMakeLists.txt 2016-01-23 08:27:49 +0000
4@@ -119,6 +119,7 @@
5 # they always come in pairs in these cases.
6 wl_add_flag(WL_COMPILE_DIAGNOSTICS "-Wno-covered-switch-default")
7 wl_add_flag(WL_COMPILE_DIAGNOSTICS "-Wno-unreachable-code")
8+ wl_add_flag(WL_COMPILE_DIAGNOSTICS "-Wno-documentation")
9
10 wl_add_flag(WL_COMPILE_DIAGNOSTICS "-Werror=non-pod-varargs")
11

Subscribers

People subscribed via source and target branches

to status/vote changes: