cpp/openttd-patchpack/source Changeset - r6510:68b182bc0b88

Changeset - r6510:68b182bc0b88

Parent rev.

Child rev.

[Not reviewed]

master

0 4 0

KUDr - 17 years ago 2007-04-20 19:13:35
kudr@openttd.org

(svn r9693) -Codechange [YAPF]: GetBestNode() now returns pointer to node instead of reference

4 files changed with 14 insertions and 13 deletions:

src/yapf/yapf_base.hpp

src/yapf/yapf_rail.cpp

src/yapf/yapf_road.cpp

src/yapf/yapf_ship.cpp

0 comments (0 inline, 0 general)

src/yapf/yapf_base.hpp

➞

Show inline comments

@@ @@ -117,7 +117,8 @@ public: @@
 		while (true) {
 			m_num_steps++;
 			Node *n = m_nodes.GetBestOpenNode();
 			if (n == NULL) break;
 			if (n == NULL)
 				break;
 			// if the best open node was worse than the best path found, we can finish
 			if (m_pBestDestNode != NULL && m_pBestDestNode->GetCost() < n->GetCostEstimate())
@@ @@ -160,9 +161,9 @@ public: @@
 	/** If path was found return the best node that has reached the destination. Otherwise
 	 *  return the best visited node (which was nearest to the destination).
 	 */
-	FORCEINLINE Node& GetBestNode()
+	FORCEINLINE Node* GetBestNode()
+	{
-		return (m_pBestDestNode != NULL) ? *m_pBestDestNode : *m_pBestIntermediateNode;
 		return (m_pBestDestNode != NULL) ? m_pBestDestNode : m_pBestIntermediateNode;
+	}
 	/** Calls NodeList::CreateNewNode() - allocates new node that can be filled and used

src/yapf/yapf_rail.cpp

➞

Show inline comments

@@ @@ -61,11 +61,11 @@ public: @@
 		// some path found
 		// get found depot tile
 		Node& n = Yapf().GetBestNode();
 		*depot_tile = n.GetLastTile();
 		Node *n = Yapf().GetBestNode();
 		*depot_tile = n->GetLastTile();
 		// walk through the path back to the origin
-		Node* pNode = &n;
+		Node *pNode = n;
 		while (pNode->m_parent != NULL) {
 			pNode = pNode->m_parent;
+		}
@@ @@ -128,7 +128,7 @@ public: @@
 		// if path not found - return INVALID_TRACKDIR
 		Trackdir next_trackdir = INVALID_TRACKDIR;
-		Node* pNode = &Yapf().GetBestNode();
+		Node *pNode = Yapf().GetBestNode();
 		if (pNode != NULL) {
 			// path was found or at least suggested
 			// walk through the path back to the origin
@@ @@ -165,7 +165,7 @@ public: @@
 		// path was found
 		// walk through the path back to the origin
-		Node* pNode = &Yapf().GetBestNode();
+		Node *pNode = Yapf().GetBestNode();
 		while (pNode->m_parent != NULL) {
 			pNode = pNode->m_parent;
+		}

src/yapf/yapf_road.cpp

➞

Show inline comments

@@ @@ -287,7 +287,7 @@ public: @@
 		// if path not found - return INVALID_TRACKDIR
 		Trackdir next_trackdir = INVALID_TRACKDIR;
-		Node* pNode = &Yapf().GetBestNode();
+		Node *pNode = Yapf().GetBestNode();
 		if (pNode != NULL) {
 			// path was found or at least suggested
 			// walk through the path back to its origin
@@ @@ -329,7 +329,7 @@ public: @@
 		// if path not found - return distance = UINT_MAX
 		uint dist = UINT_MAX;
-		Node* pNode = &Yapf().GetBestNode();
+		Node *pNode = Yapf().GetBestNode();
 		if (pNode != NULL) {
 			// path was found or at least suggested
 			// get the path cost estimate
@@ @@ -371,8 +371,8 @@ public: @@
 		// some path found
 		// get found depot tile
 		Node& n = Yapf().GetBestNode();
 		TileIndex depot_tile = n.m_segment_last_tile;
 		Node *n = Yapf().GetBestNode();
 		TileIndex depot_tile = n->m_segment_last_tile;
 		assert(IsTileDepotType(depot_tile, TRANSPORT_ROAD));
 		Depot* ret = GetDepotByTile(depot_tile);
 		return ret;

src/yapf/yapf_ship.cpp

➞

Show inline comments

@@ @@ -67,7 +67,7 @@ public: @@
 		if (bFound) {
 			// path was found
 			// walk through the path back to the origin
-			Node* pNode = &pf.GetBestNode();
 			Node* pNode = pf.GetBestNode();
 			Node* pPrevNode = NULL;
 			while (pNode->m_parent != NULL) {
 				pPrevNode = pNode;

0 comments (0 inline, 0 general)