From 15f873c018e1550c22eed38b1ea5ab7970edd1f0 Mon Sep 17 00:00:00 2001
From: Tulio Leao <tupaschoal@gmail.com>
Date: Tue, 24 Dec 2019 17:21:26 -0300
Subject: [PATCH] Remove LocationXY usage on
 openrct2-ui\windows\ViewClipping.cpp

---
 src/openrct2-ui/windows/ViewClipping.cpp | 24 ++++++++++++------------
 1 file changed, 12 insertions(+), 12 deletions(-)

diff --git a/src/openrct2-ui/windows/ViewClipping.cpp b/src/openrct2-ui/windows/ViewClipping.cpp
index bc58ca2533..f16891a3d9 100644
--- a/src/openrct2-ui/windows/ViewClipping.cpp
+++ b/src/openrct2-ui/windows/ViewClipping.cpp
@@ -66,9 +66,9 @@ static rct_widget window_view_clipping_widgets[] = {
 
 #pragma region Members
 
-static LocationXY16 _selectionStart;
-static LocationXY8 _previousClipSelectionA;
-static LocationXY8 _previousClipSelectionB;
+static CoordsXY _selectionStart;
+static TileCoordsXY _previousClipSelectionA;
+static TileCoordsXY _previousClipSelectionB;
 static bool _toolActive;
 static bool _dragging;
 
@@ -239,8 +239,8 @@ static void window_view_clipping_mouseup(rct_window* w, rct_widgetindex widgetIn
             _dragging = false;
 
             // Reset clip selection to show all tiles
-            _previousClipSelectionA = gClipSelectionA;
-            _previousClipSelectionB = gClipSelectionB;
+            _previousClipSelectionA = TileCoordsXY{ gClipSelectionA.x, gClipSelectionA.y };
+            _previousClipSelectionB = TileCoordsXY{ gClipSelectionB.x, gClipSelectionB.y };
             gClipSelectionA = { 0, 0 };
             gClipSelectionB = { MAXIMUM_MAP_SIZE_TECHNICAL - 1, MAXIMUM_MAP_SIZE_TECHNICAL - 1 };
             gfx_invalidate_screen();
@@ -303,8 +303,8 @@ static void window_view_clipping_update(rct_window* w)
     if (_toolActive && !window_view_clipping_tool_is_active())
     {
         _toolActive = false;
-        gClipSelectionA = _previousClipSelectionA;
-        gClipSelectionB = _previousClipSelectionB;
+        gClipSelectionA = { static_cast<uint8_t>(_previousClipSelectionA.x), static_cast<uint8_t>(_previousClipSelectionA.y) };
+        gClipSelectionB = { static_cast<uint8_t>(_previousClipSelectionB.x), static_cast<uint8_t>(_previousClipSelectionB.y) };
     }
 
     widget_invalidate(w, WIDX_CLIP_HEIGHT_SLIDER);
@@ -336,7 +336,7 @@ static void window_view_clipping_tool_down(rct_window* w, rct_widgetindex widget
     if (mapCoords)
     {
         _dragging = true;
-        _selectionStart = { static_cast<int16_t>(mapCoords->x), static_cast<int16_t>(mapCoords->y) };
+        _selectionStart = *mapCoords;
     }
 }
 
@@ -353,10 +353,10 @@ static void window_view_clipping_tool_drag(rct_window* w, rct_widgetindex widget
     {
         map_invalidate_selection_rect();
         gMapSelectFlags |= MAP_SELECT_FLAG_ENABLE;
-        gMapSelectPositionA.x = std::min(_selectionStart.x, static_cast<int16_t>(mapCoords->x));
-        gMapSelectPositionB.x = std::max(_selectionStart.x, static_cast<int16_t>(mapCoords->x));
-        gMapSelectPositionA.y = std::min(_selectionStart.y, static_cast<int16_t>(mapCoords->y));
-        gMapSelectPositionB.y = std::max(_selectionStart.y, static_cast<int16_t>(mapCoords->y));
+        gMapSelectPositionA.x = std::min(_selectionStart.x, mapCoords->x);
+        gMapSelectPositionB.x = std::max(_selectionStart.x, mapCoords->x);
+        gMapSelectPositionA.y = std::min(_selectionStart.y, mapCoords->y);
+        gMapSelectPositionB.y = std::max(_selectionStart.y, mapCoords->y);
         gMapSelectType = MAP_SELECT_TYPE_FULL;
         map_invalidate_selection_rect();
     }