(svn r18781) -Codechange: pass the CommandCost to the callback instead of whether it succeeded or not.

-Fix: AIs did update their last cost incorrectly in network games if the cost of the DC_EXEC phase differed from the ~DC_EXEC phase.

1 files changed, 6 insertions, 6 deletions

diff --git a/src/signs_cmd.cpp b/src/signs_cmd.cpp
index dff599b65..90ccb89f8 100644
--- a/src/signs_cmd.cpp
+++ b/src/signs_cmd.cpp
@@ -105,17 +105,17 @@ CommandCost CmdRenameSign(TileIndex tile, DoCommandFlag flags, uint32 p1, uint32
 
 /**
  * Callback function that is called after a sign is placed
- * @param success of the operation
+ * @param result of the operation
  * @param tile unused
  * @param p1 unused
  * @param p2 unused
  */
-void CcPlaceSign(bool success, TileIndex tile, uint32 p1, uint32 p2)
+void CcPlaceSign(const CommandCost &result, TileIndex tile, uint32 p1, uint32 p2)
 {
-	if (success) {
-		ShowRenameSignWindow(Sign::Get(_new_sign_id));
-		ResetObjectToPlace();
-	}
+	if (result.Failed()) return;
+
+	ShowRenameSignWindow(Sign::Get(_new_sign_id));
+	ResetObjectToPlace();
 }
 
 /**