cpp/openttd-patchpack/source Changeset - r24316:fdd14cd985bb

Changeset - r24316:fdd14cd985bb

Parent rev.

Child rev.

[Not reviewed]

master

0 1 0

TrevorShelton - 4 years ago 2020-06-28 08:55:57
54145769+TrevorShelton@users.noreply.github.com

Change #8162: Conversion of Town Road, Impr. Error

This changes the error when you attempt to convert a road owned by a town to another road, specifying that it's owned by the town rather than simply being incompatible. As the original poster of the issue pointed out, these seemed to be the only occurences of the incompatible road string, so now it's unused, but they would be left untouched in case of future use or since changing it to a different error would do the work of removing it then. If requested, it likely wouldn't be too difficult to remove the string entirely.

1 file changed with 4 insertions and 2 deletions:

src/road_cmd.cpp

0 comments (0 inline, 0 general)

src/road_cmd.cpp

➞

Show inline comments

@@ @@ -2396,13 +2396,14 @@ CommandCost CmdConvertRoad(TileIndex til @@
 				if (ret.Failed()) {
 					error = ret;
 					continue;
+				}
 				if (rtt == RTT_ROAD && owner == OWNER_TOWN) {
 					error.MakeError(STR_ERROR_INCOMPATIBLE_ROAD);
 					error.MakeError(STR_ERROR_OWNED_BY);
 					GetNameOfOwner(OWNER_TOWN, tile);
 					continue;
+				}
+			}
 			uint num_pieces = CountBits(GetAnyRoadBits(tile, rtt));;
 			found_convertible_road = true;
@@ @@ -2442,13 +2443,14 @@ CommandCost CmdConvertRoad(TileIndex til @@
 				if (ret.Failed()) {
 					error = ret;
 					continue;
+				}
 				if (rtt == RTT_ROAD && owner == OWNER_TOWN) {
 					error.MakeError(STR_ERROR_INCOMPATIBLE_ROAD);
 					error.MakeError(STR_ERROR_OWNED_BY);
 					GetNameOfOwner(OWNER_TOWN, tile);
 					continue;
+				}
+			}
 			/* There are 2 pieces on *every* tile of the bridge or tunnel */
 			uint num_pieces = (GetTunnelBridgeLength(tile, endtile) + 2) * 2;

0 comments (0 inline, 0 general)