From e3156546fa2e1b3d0cb0710edc883d43c08b5b4e Mon Sep 17 00:00:00 2001 From: Tyler Trahan Date: Sun, 16 Jun 2024 16:07:57 -0400 Subject: [PATCH] Codefix: Don't claim to allocate new waypoint when just checking (#12792) --- src/waypoint_cmd.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/waypoint_cmd.cpp b/src/waypoint_cmd.cpp index 4fa43cb0b9..e28cbe691b 100644 --- a/src/waypoint_cmd.cpp +++ b/src/waypoint_cmd.cpp @@ -225,7 +225,7 @@ CommandCost CmdBuildRailWaypoint(DoCommandFlag flags, TileIndex start_tile, Axis /* Reuse an existing waypoint. */ if (wp->owner != _current_company) return_cmd_error(STR_ERROR_TOO_CLOSE_TO_ANOTHER_WAYPOINT); - /* check if we want to expand an already existing waypoint? */ + /* Check if we want to expand an already existing waypoint. */ if (wp->train_station.tile != INVALID_TILE) { ret = CanExpandRailStation(wp, new_location); if (ret.Failed()) return ret; @@ -234,7 +234,7 @@ CommandCost CmdBuildRailWaypoint(DoCommandFlag flags, TileIndex start_tile, Axis ret = wp->rect.BeforeAddRect(start_tile, width, height, StationRect::ADD_TEST); if (ret.Failed()) return ret; } else { - /* allocate and initialize new waypoint */ + /* Check if we can create a new waypoint. */ if (!Waypoint::CanAllocateItem()) return_cmd_error(STR_ERROR_TOO_MANY_STATIONS_LOADING); }