cpp/openttd-patchpack/source Changeset - r7806:a2c6f8da57ac

Changeset - r7806:a2c6f8da57ac

Parent rev.

Child rev.

[Not reviewed]

master

0 2 0

glx - 17 years ago 2007-10-29 23:29:06
glx@openttd.org

(svn r11356) -Fix (r11305): funding industries in MP game was causing desync

2 files changed with 3 insertions and 3 deletions:

src/industry_cmd.cpp

src/industry_gui.cpp

0 comments (0 inline, 0 general)

src/industry_cmd.cpp

➞

Show inline comments

@@ @@ -1567,13 +1567,13 @@ static Industry *CreateNewIndustryHelper @@
+}
 /** Build/Fund an industry
  * @param tile tile where industry is built
  * @param flags of operations to conduct
  * @param p1 industry type see build_industry.h and see industry.h
- * @param p2 unused
+ * @param p2 first layout to try
  * @return index of the newly create industry, or CMD_ERROR if it failed
  */
 CommandCost CmdBuildIndustry(TileIndex tile, uint32 flags, uint32 p1, uint32 p2)
+{
 	const IndustrySpec *indspec;
@@ @@ -1615,13 +1615,13 @@ CommandCost CmdBuildIndustry(TileIndex t @@
+				}
+			}
+		}
 	} else {
 		int count = indspec->num_table;
 		const IndustryTileTable * const *itt = indspec->table;
-		int num = RandomRange(count);
+		int num = clamp(p2, 0, count - 1);
 		_error_message = STR_0239_SITE_UNSUITABLE;
 		do {
 			if (--count < 0) return CMD_ERROR;
 			if (--num < 0) num = indspec->num_table - 1;
 		} while (!CheckIfIndustryTilesAreFree(tile, itt[num], num, p1));

src/industry_gui.cpp

➞

Show inline comments

@@ @@ -312,13 +312,13 @@ static void BuildDynamicIndustryWndProc( @@
 					ShowErrorMessage(_error_message, STR_0285_CAN_T_BUILD_HERE, e->we.place.pt.x, e->we.place.pt.y);
+				}
 				_ignore_restrictions = false;
 				_generating_world = false;
 			} else {
-				success = DoCommandP(e->we.place.tile, WP(w, fnd_d).select, 0, NULL, CMD_BUILD_INDUSTRY | CMD_MSG(STR_4830_CAN_T_CONSTRUCT_THIS_INDUSTRY));
+				success = DoCommandP(e->we.place.tile, WP(w, fnd_d).select, InteractiveRandomRange(indsp->num_table), NULL, CMD_BUILD_INDUSTRY | CMD_MSG(STR_4830_CAN_T_CONSTRUCT_THIS_INDUSTRY));
+			}
 			/* If an industry has been built, just reset the cursor and the system */
 			if (success) ResetObjectToPlace();
 		} break;

0 comments (0 inline, 0 general)