From 20adcecb1d480724ebccc9448d2bef562c3ba59f Mon Sep 17 00:00:00 2001
From: "alexey.lysiuk" <alexey.lysiuk@gmail.com>
Date: Mon, 2 Jun 2014 11:37:40 +0300
Subject: [PATCH] Remove redundant saving of GL nodes if they were loaded from
 cache

---
 src/p_glnodes.cpp | 28 ++++++++++++++++------------
 1 file changed, 16 insertions(+), 12 deletions(-)

diff --git a/src/p_glnodes.cpp b/src/p_glnodes.cpp
index e59f3859a..899005ffa 100644
--- a/src/p_glnodes.cpp
+++ b/src/p_glnodes.cpp
@@ -959,6 +959,7 @@ bool P_LoadGLNodes(MapData * map)
 bool P_CheckNodes(MapData * map, bool rebuilt, int buildtime)
 {
 	bool ret = false;
+	bool loaded = false;
 
 	// If the map loading code has performed a node rebuild we don't need to check for it again.
 	if (!rebuilt && !P_CheckForGLNodes())
@@ -978,7 +979,8 @@ bool P_CheckNodes(MapData * map, bool rebuilt, int buildtime)
 		numsegs = 0;
 
 		// Try to load GL nodes (cached or GWA)
-		if (!P_LoadGLNodes(map))
+		loaded = P_LoadGLNodes(map);
+		if (!loaded)
 		{
 			// none found - we have to build new ones!
 			unsigned int startTime, endTime;
@@ -1006,20 +1008,22 @@ bool P_CheckNodes(MapData * map, bool rebuilt, int buildtime)
 		}
 	}
 
+	if (!loaded)
+	{
 #ifdef DEBUG
-	// Building nodes in debug is much slower so let's cache them only if cachetime is 0
-	buildtime = 0;
+		// Building nodes in debug is much slower so let's cache them only if cachetime is 0
+		buildtime = 0;
 #endif
-	if (gl_cachenodes && buildtime/1000.f >= gl_cachetime)
-	{
-		DPrintf("Caching nodes\n");
-		CreateCachedNodes(map);
+		if (gl_cachenodes && buildtime/1000.f >= gl_cachetime)
+		{
+			DPrintf("Caching nodes\n");
+			CreateCachedNodes(map);
+		}
+		else
+		{
+			DPrintf("Not caching nodes (time = %f)\n", buildtime/1000.f);
+		}
 	}
-	else
-	{
-		DPrintf("Not caching nodes (time = %f)\n", buildtime/1000.f);
-	}
-
 
 	if (!gamenodes)
 	{