← Back to team overview

widelands-dev team mailing list archive

[Merge] lp:~majcherlk/widelands/bug-1370136 into lp:widelands

 

Lukasz has proposed merging lp:~majcherlk/widelands/bug-1370136 into lp:widelands.

Requested reviews:
  Widelands Developers (widelands-dev)

For more details, see:
https://code.launchpad.net/~majcherlk/widelands/bug-1370136/+merge/246085

Changing condition in handle_mousemove() function - now anchor equals to zero is valid anchor (index of Barbarian ax is equal to zero and before the change zero wasn't considered as valid anchor so dragging for axe wasn't working).
-- 
Your team Widelands Developers is requested to review the proposed merge of lp:~majcherlk/widelands/bug-1370136 into lp:widelands.
=== modified file 'src/wui/waresdisplay.cc'
--- src/wui/waresdisplay.cc	2014-11-27 12:02:08 +0000
+++ src/wui/waresdisplay.cc	2015-01-11 23:35:54 +0000
@@ -99,7 +99,7 @@
 	                          m_tribe.get_worker_descr(index)->descname() :
 	                          m_tribe.get_ware_descr(index)->descname()) :
 	                      "");
-	if (m_selection_anchor) {
+	if (m_selection_anchor != Widelands::INVALID_INDEX) {
 		// Ensure mouse button is still pressed as some
 		// mouse release events do not reach us
 		if (state ^ SDL_BUTTON_LMASK) {


Follow ups