Merge lp:~hjd/widelands/unused-value into lp:widelands

Proposed by Hans Joachim Desserud
Status: Merged
Merged at revision: 7159
Proposed branch: lp:~hjd/widelands/unused-value
Merge into: lp:widelands
Diff against target: 13 lines (+0/-3)
1 file modified
src/ai/defaultai.cc (+0/-3)
To merge this branch: bzr merge lp:~hjd/widelands/unused-value
Reviewer Review Type Date Requested Status
SirVer Approve
Review via email: mp+228921@code.launchpad.net

Description of the change

Removed variable assignment happening just before returning (found by scan-build). At the same time I removed what I assume is an older version of this check.

To post a comment you must log in.
Revision history for this message
SirVer (sirver) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/ai/defaultai.cc'
--- src/ai/defaultai.cc 2014-07-28 18:03:51 +0000
+++ src/ai/defaultai.cc 2014-07-30 17:51:29 +0000
@@ -1419,10 +1419,7 @@
1419 } // end of mines_ section1419 } // end of mines_ section
14201420
1421 // if there is no winner:1421 // if there is no winner:
1422 // if (proposed_building == INVALID_INDEX) {
1423 if (best_building == nullptr) {1422 if (best_building == nullptr) {
1424
1425 mine = false;
1426 return false;1423 return false;
1427 }1424 }
14281425

Subscribers

People subscribed via source and target branches

to status/vote changes: