Merge lp:~hjd/widelands/assert-to-exception into lp:widelands
Proposed by
Hans Joachim Desserud
Status: | Merged |
---|---|
Merged at revision: | 7224 |
Proposed branch: | lp:~hjd/widelands/assert-to-exception |
Merge into: | lp:widelands |
Diff against target: |
22 lines (+2/-3) 1 file modified
src/wui/interactive_base.cc (+2/-3) |
To merge this branch: | bzr merge lp:~hjd/widelands/assert-to-exception |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
SirVer | Approve | ||
Review via email: mp+238824@code.launchpad.net |
Description of the change
As mentioned, turn this assert into an exception so that it will also work with release builds and resolve unused assignment issue.
Btw, this seems rather small and independent so I believe it can be refactored into a separate method getWareInfoInde
To post a comment you must log in.