cpp/openttd-patchpack/source Changeset - r23750:62d6ca31f51e

Changeset - r23750:62d6ca31f51e

Parent rev.

Child rev.

[Not reviewed]

master

0 1 0

stormcone - 5 years ago 2019-05-26 22:52:53
48624099+stormcone@users.noreply.github.com

Fix #7609: In the sprite aligner window the 'Reset relative' button crashes the game.

The 'offs_start_map' is a 'SmallMap', so its own 'Erase' function should be called instead of the underlying vector's 'erase' function.
And fix a "typo". :)

1 file changed with 2 insertions and 2 deletions:

src/newgrf_debug_gui.cpp

0 comments (0 inline, 0 general)

src/newgrf_debug_gui.cpp

➞

Show inline comments

@@ @@ -972,7 +972,7 @@ struct SpriteAlignerWindow : Window { @@
 					this->offs_start_map.Insert(this->current_sprite, XyOffs(spr->x_offs, spr->y_offs));
+				}
 				switch (widget) {
-					/* Move ten units at a time if ctrl is pressed. */
+					/* Move eight units at a time if ctrl is pressed. */
 					case WID_SA_UP:    spr->y_offs -= _ctrl_pressed ? 8 : 1; break;
 					case WID_SA_DOWN:  spr->y_offs += _ctrl_pressed ? 8 : 1; break;
 					case WID_SA_LEFT:  spr->x_offs -= _ctrl_pressed ? 8 : 1; break;
@@ @@ -986,7 +986,7 @@ struct SpriteAlignerWindow : Window { @@
 			case WID_SA_RESET_REL:
 				/* Reset the starting offsets for the current sprite. */
-				this->offs_start_map.erase(this->offs_start_map.begin() + this->current_sprite);
+				this->offs_start_map.Erase(this->current_sprite);
 				this->SetDirty();
 				break;
+		}

0 comments (0 inline, 0 general)