Fix PlayerSAO deletion warning (0eede97)

minetest · Feb 4, 2017 · de664b1 · de664b1
1 parent 047168c
commit de664b1
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/src/serverenvironment.cpp b/src/serverenvironment.cpp
@@ -2087,9 +2087,7 @@ void ServerEnvironment::deactivateFarObjects(bool force_delete)
 
 			if(block)
 			{
-				// Force delete object if mapblock is full, but ignore players
-				if (obj->getType() != ACTIVEOBJECT_TYPE_PLAYER &&
-					block->m_static_objects.m_stored.size() >= g_settings->getU16("max_objects_per_block")) {
+				if (block->m_static_objects.m_stored.size() >= g_settings->getU16("max_objects_per_block")) {
 					warningstream << "ServerEnv: Trying to store id = " << obj->getId()
 						<< " statically but block " << PP(blockpos)
 						<< " already contains "
@@ -2149,6 +2147,13 @@ void ServerEnvironment::deactivateFarObjects(bool force_delete)
 			continue;
 		}
 
+		if (!force_delete && obj->getType() == ACTIVEOBJECT_TYPE_PLAYER) {
+			warningstream << "ServerEnvironment::deactivateFarObjects(): "
+				<< "Trying to delete player object, THIS SHOULD NEVER HAPPEN!"
+				<< std::endl;
+			continue;
+		}
+
 		verbosestream<<"ServerEnvironment::deactivateFarObjects(): "
 			<<"object id="<<id<<" is not known by clients"
 			<<"; deleting"<<std::endl;