cpp/openttd-patchpack/source Changeset - r7298:f65dbf822953

Changeset - r7298:f65dbf822953

Parent rev.

Child rev.

[Not reviewed]

master

0 2 0

KUDr - 17 years ago 2007-07-21 13:45:15
kudr@openttd.org

(svn r10646) -Cleanup: AutoPtrT<>::Release() renamed to Detach()
-better describes what it does
-was inconsistent with CCountedPtr<>::Release()

2 files changed with 7 insertions and 7 deletions:

src/misc/autoptr.hpp

src/station_cmd.cpp

0 comments (0 inline, 0 general)

src/misc/autoptr.hpp

➞

Show inline comments

@@ @@ -45,7 +45,7 @@ public: @@
+	}
 	/** give-up ownership and NULLify the raw pointer */
-	FORCEINLINE T* Release()
+	FORCEINLINE T* Detach()
+	{
 		T* p = m_p;
 		m_p = NULL;

src/station_cmd.cpp

➞

Show inline comments

@@ @@ -982,7 +982,7 @@ CommandCost CmdBuildRailroadStation(Tile @@
 		RebuildStationLists();
 		InvalidateWindow(WC_STATION_LIST, st->owner);
 		/* success, so don't delete the new station */
-		st_auto_delete.Release();
+		st_auto_delete.Detach();
+	}
 	return cost;
@@ @@ -1361,8 +1361,8 @@ CommandCost CmdBuildRoadStop(TileIndex t @@
 		RebuildStationLists();
 		InvalidateWindow(WC_STATION_LIST, st->owner);
 		/* success, so don't delete the new station and the new road stop */
 		st_auto_delete.Release();
 		rs_auto_delete.Release();
 		st_auto_delete.Detach();
 		rs_auto_delete.Detach();
+	}
 	return cost;
+}
@@ @@ -1668,7 +1668,7 @@ CommandCost CmdBuildAirport(TileIndex ti @@
 		RebuildStationLists();
 		InvalidateWindow(WC_STATION_LIST, st->owner);
 		/* success, so don't delete the new station */
-		st_auto_delete.Release();
+		st_auto_delete.Detach();
+	}
 	return cost;
@@ @@ -1764,7 +1764,7 @@ CommandCost CmdBuildBuoy(TileIndex tile, @@
 		RebuildStationLists();
 		InvalidateWindow(WC_STATION_LIST, st->owner);
 		/* success, so don't delete the new station */
-		st_auto_delete.Release();
+		st_auto_delete.Detach();
+	}
 	return CommandCost(_price.build_dock);
@@ @@ -1931,7 +1931,7 @@ CommandCost CmdBuildDock(TileIndex tile, @@
 		RebuildStationLists();
 		InvalidateWindow(WC_STATION_LIST, st->owner);
 		/* success, so don't delete the new station */
-		st_auto_delete.Release();
+		st_auto_delete.Detach();
+	}
 	return CommandCost(_price.build_dock);
+}

0 comments (0 inline, 0 general)