widelands

Merge lp://qastaging/~widelands-dev/widelands/compiler-warnings-201908 into lp://qastaging/widelands

  1. compiler-warnings-201908
  2. Merge into trunk
Proposed by GunChleoc
Status: Merged
Merged at revision: 9191
Proposed branch: lp://qastaging/~widelands-dev/widelands/compiler-warnings-201908
Merge into: lp://qastaging/widelands
Diff against target: 27 lines (+1/-5)
2 files modified
src/wlapplication_options.cc (+1/-1)
src/wui/constructionsitewindow.cc (+0/-4)
To merge this branch: bzr merge lp://qastaging/~widelands-dev/widelands/compiler-warnings-201908
Reviewer Review Type Date Requested Status
Klaus Halfmann Approve
Review via email: mp+372109@code.qastaging.launchpad.net

Commit message

Fix compiler warnings.

To post a comment you must log in.
Revision history for this message
Klaus Halfmann (klaus-halfmann) wrote :

Nothing to really check here.
I will it compile it anyway

review: Approve (code review)
Revision history for this message
Klaus Halfmann (klaus-halfmann) wrote :

Compiled withouth any noteworthy complaints

@bunnybot merge

review: Approve
Revision history for this message
GunChleoc (gunchleoc) wrote :

Thanks for the review!

I'll be going through the Travis/Appveyor logs to see if I can find more stuff to fix.

Revision history for this message
bunnybot (widelandsofficial) wrote :

Continuous integration builds have changed state:

Travis build 5365. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/579080055.
Appveyor build 5135. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_compiler_warnings_201908-5135.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to status/vote changes: