cpp/openttd-patchpack/source Changeset - r11370:6df9331ca497

Changeset - r11370:6df9331ca497

Parent rev.

Child rev.

[Not reviewed]

master

0 8 0

smatz - 15 years ago 2009-03-15 15:25:18
smatz@openttd.org

(svn r15725) -Fix: centering on a vehicle didn't respect its z coordinate

8 files changed with 25 insertions and 15 deletions:

src/main_gui.cpp

src/news_gui.cpp

src/smallmap_gui.cpp

src/vehicle_gui.cpp

src/viewport.cpp

src/viewport_func.h

src/waypoint_gui.cpp

src/window.cpp

0 comments (0 inline, 0 general)

src/main_gui.cpp

➞

Show inline comments

@@ @@ -176,7 +176,7 @@ void ZoomInOrOutToCursorWindow(bool in, @@
 		Point pt = GetTileZoomCenterWindow(in, w);
 		if (pt.x != -1) {
 			ScrollWindowTo(pt.x, pt.y, w, true);
+			ScrollWindowTo(pt.x, pt.y, -1, w, true);
 			DoZoomInOutWindow(in ? ZOOM_IN : ZOOM_OUT, w);
+		}

src/news_gui.cpp

➞

Show inline comments

@@ @@ -267,8 +267,8 @@ struct NewsWindow : Window { @@
 			case 0:
 				if (this->ni->flags & NF_VEHICLE) {
 					Vehicle *v = GetVehicle(this->ni->data_a);
 					ScrollMainWindowTo(v->x_pos, v->y_pos);
 					const Vehicle *v = GetVehicle(this->ni->data_a);
 					ScrollMainWindowTo(v->x_pos, v->y_pos, v->z_pos);
 				} else if (this->ni->flags & NF_TILE) {
 					if (_ctrl_pressed) {
 						ShowExtraViewPortWindow(this->ni->data_a);

src/smallmap_gui.cpp

➞

Show inline comments

@@ @@ -1214,9 +1214,17 @@ void ShowExtraViewPortWindow(TileIndex t @@
 	new ExtraViewportWindow(&_extra_view_port_desc, i, tile);
+}
 bool ScrollMainWindowTo(int x, int y, bool instant)
 /**
  * Scrolls the main window to given coordinates.
  * @param x x coordinate
  * @param y y coordinate
  * @param z z coordinate; -1 to scroll to terrain height
  * @param instant scroll instantly (meaningful only when smooth_scrolling is active)
  * @return did the viewport position change?
  */
 bool ScrollMainWindowTo(int x, int y, int z, bool instant)
+{
 	bool res = ScrollWindowTo(x, y, FindWindowById(WC_MAIN_WINDOW, 0), instant);
+	bool res = ScrollWindowTo(x, y, z, FindWindowById(WC_MAIN_WINDOW, 0), instant);
 	/* If a user scrolls to a tile (via what way what so ever) and already is on
 	 *  that tile (e.g.: pressed twice), move the smallmap to that location,

src/vehicle_gui.cpp

➞

Show inline comments

@@ @@ -1957,7 +1957,7 @@ struct VehicleViewWindow : Window { @@
 				if (_ctrl_pressed && mainwindow->viewport->zoom == ZOOM_LVL_NORMAL) {
 					mainwindow->viewport->follow_vehicle = v->index;
 				} else {
 					ScrollMainWindowTo(v->x_pos, v->y_pos);
+					ScrollMainWindowTo(v->x_pos, v->y_pos, v->z_pos);
+				}
 			} break;
@@ @@ -2037,7 +2037,7 @@ void StopGlobalFollowVehicle(const Vehic @@
+{
 	Window *w = FindWindowById(WC_MAIN_WINDOW, 0);
 	if (w != NULL && w->viewport->follow_vehicle == v->index) {
 		ScrollMainWindowTo(v->x_pos, v->y_pos, true); // lock the main view on the vehicle's last position
+		ScrollMainWindowTo(v->x_pos, v->y_pos, v->z_pos, true); // lock the main view on the vehicle's last position
 		w->viewport->follow_vehicle = INVALID_VEHICLE;
+	}
+}

src/viewport.cpp

➞

Show inline comments

@@ @@ -137,7 +137,7 @@ TileHighlightData _thd; @@
 static TileInfo *_cur_ti;
 bool _draw_bounding_boxes = false;
-static Point MapXYZToViewport(const ViewPort *vp, uint x, uint y, uint z)
+static Point MapXYZToViewport(const ViewPort *vp, int x, int y, int z)
+{
 	Point p = RemapCoords(x, y, z);
 	p.x -= vp->virtual_width / 2;
@@ @@ -2065,10 +2065,12 @@ void PlaceObject() @@
 /* scrolls the viewport in a window to a given location */
-bool ScrollWindowTo(int x , int y, Window *w, bool instant)
+bool ScrollWindowTo(int x, int y, int z, Window *w, bool instant)
+{
 	/* The slope cannot be acquired outside of the map, so make sure we are always within the map. */
 	Point pt = MapXYZToViewport(w->viewport, x, y, GetSlopeZ(Clamp(x, 0, MapSizeX() * TILE_SIZE - 1), Clamp(y, 0, MapSizeY() * TILE_SIZE - 1)));
 	if (z == -1) z = GetSlopeZ(Clamp(x, 0, MapSizeX() * TILE_SIZE - 1), Clamp(y, 0, MapSizeY() * TILE_SIZE - 1));
 	Point pt = MapXYZToViewport(w->viewport, x, y, z);
 	w->viewport->follow_vehicle = INVALID_VEHICLE;
 	if (w->viewport->dest_scrollpos_x == pt.x && w->viewport->dest_scrollpos_y == pt.y)
@@ @@ -2086,7 +2088,7 @@ bool ScrollWindowTo(int x , int y, Windo @@
 bool ScrollMainWindowToTile(TileIndex tile, bool instant)
+{
 	return ScrollMainWindowTo(TileX(tile) * TILE_SIZE + TILE_SIZE / 2, TileY(tile) * TILE_SIZE + TILE_SIZE / 2, instant);
+	return ScrollMainWindowTo(TileX(tile) * TILE_SIZE + TILE_SIZE / 2, TileY(tile) * TILE_SIZE + TILE_SIZE / 2, -1, instant);
+}
 void SetRedErrorSquare(TileIndex tile)

src/viewport_func.h

➞

Show inline comments

@@ @@ -53,10 +53,10 @@ Vehicle *CheckMouseOverVehicle(); @@
 void ViewportDoDraw(const ViewPort *vp, int left, int top, int right, int bottom);
 bool ScrollWindowTo(int x, int y, Window *w, bool instant = false);
+bool ScrollWindowTo(int x, int y, int z, Window *w, bool instant = false);
 bool ScrollMainWindowToTile(TileIndex tile, bool instant = false);
 bool ScrollMainWindowTo(int x, int y, bool instant = false);
+bool ScrollMainWindowTo(int x, int y, int z = -1, bool instant = false);
 extern Point _tile_fract_coords;

src/waypoint_gui.cpp

➞

Show inline comments

@@ @@ -87,7 +87,7 @@ public: @@
+	{
 		int x = TileX(this->wp->xy) * TILE_SIZE;
 		int y = TileY(this->wp->xy) * TILE_SIZE;
 		ScrollWindowTo(x, y, this);
+		ScrollWindowTo(x, y, -1, this);
+	}
 	virtual void OnQueryTextFinished(char *str)

src/window.cpp

➞

Show inline comments

@@ @@ -1651,7 +1651,7 @@ static bool HandleViewportScroll() @@
 	if (w == FindWindowById(WC_MAIN_WINDOW, 0) && w->viewport->follow_vehicle != INVALID_VEHICLE) {
 		/* If the main window is following a vehicle, then first let go of it! */
 		const Vehicle *veh = GetVehicle(w->viewport->follow_vehicle);
 		ScrollMainWindowTo(veh->x_pos, veh->y_pos, true); // This also resets follow_vehicle
+		ScrollMainWindowTo(veh->x_pos, veh->y_pos, veh->z_pos, true); // This also resets follow_vehicle
 		return true;
+	}

0 comments (0 inline, 0 general)