cpp/openttd-patchpack/source Changeset - r15889:68e9eadf2026

Changeset - r15889:68e9eadf2026

Parent rev.

Child rev.

[Not reviewed]

master

0 1 0

rubidium - 14 years ago 2010-08-20 17:35:10
rubidium@openttd.org

(svn r20582) -Fix [FS#4044]: Autoreplace can trigger an assertion when at the vehicle limit

1 file changed with 5 insertions and 4 deletions:

src/train_cmd.cpp

0 comments (0 inline, 0 general)

src/train_cmd.cpp

➞

Show inline comments

@@ @@ -1053,9 +1053,10 @@ static CommandCost CheckTrainAttachment( @@
  * @param dst          The destination chain after constructing the train.
  * @param original_dst The original source chain.
  * @param dst          The source chain after constructing the train.
  * @param check_limit  Whether to check the vehicle limit.
  * @return possible error of this command.
  */
 static CommandCost ValidateTrains(Train *original_dst, Train *dst, Train *original_src, Train *src)
+static CommandCost ValidateTrains(Train *original_dst, Train *dst, Train *original_src, Train *src, bool check_limit)
+{
 	/* Check whether we may actually construct the trains. */
 	CommandCost ret = CheckTrainAttachment(src);
@@ @@ -1064,7 +1065,7 @@ static CommandCost ValidateTrains(Train @@
 	if (ret.Failed()) return ret;
 	/* Check whether we need to build a new train. */
 	return CheckNewTrain(original_dst, dst, original_src, src);
+	return check_limit ? CheckNewTrain(original_dst, dst, original_src, src) : CommandCost();
+}
 /**
@@ @@ -1235,7 +1236,7 @@ CommandCost CmdMoveRailVehicle(TileIndex @@
 		/* If the autoreplace flag is set we do not need to test for the validity
 		 * because we are going to revert the train to its original state. As we
 		 * assume the original state was correct autoreplace can skip this. */
 		CommandCost ret = ValidateTrains(original_dst_head, dst_head, original_src_head, src_head);
+		CommandCost ret = ValidateTrains(original_dst_head, dst_head, original_src_head, src_head, true);
 		if (ret.Failed()) {
 			/* Restore the train we had. */
 			RestoreTrainBackup(original_src);
@@ @@ -1355,7 +1356,7 @@ CommandCost CmdSellRailWagon(DoCommandFl @@
 	ArrangeTrains(&sell_head, NULL, &new_head, v, sell_chain);
 	/* We don't need to validate the second train; it's going to be sold. */
 	CommandCost ret = ValidateTrains(NULL, NULL, first, new_head);
+	CommandCost ret = ValidateTrains(NULL, NULL, first, new_head, (flags & DC_AUTOREPLACE) == 0);
 	if (ret.Failed()) {
 		/* Restore the train we had. */
 		RestoreTrainBackup(original);

0 comments (0 inline, 0 general)