Stop using validcount for sprites

src/swrenderer/scene/r_opaque_pass.cpp

@@ -773,6 +773,9 @@ namespace swrenderer
 
 	void RenderOpaquePass::RenderScene()
 	{
+		SeenSpriteSectors.clear();
+		SeenActors.clear();
+
 		InSubsector = nullptr;
 		RenderBSPNode(nodes + numnodes - 1);	// The head node is the last node output.
 	}
@@ -824,11 +827,12 @@ namespace swrenderer
 		// A sector might have been split into several
 		//	subsectors during BSP building.
 		// Thus we check whether it was already added.
-		if (sec->touching_renderthings == nullptr || sec->validcount == validcount)
+		if (sec->touching_renderthings == nullptr || SeenSpriteSectors.find(sec) != SeenSpriteSectors.end()/*|| sec->validcount == validcount*/)
 			return;
 
 		// Well, now it will be done.
-		sec->validcount = validcount;
+		//sec->validcount = validcount;
+		SeenSpriteSectors.insert(sec);
 
 		int spriteshade = LIGHT2SHADE(lightlevel + R_ActualExtraLight(foggy));
 
@@ -836,8 +840,10 @@ namespace swrenderer
 		for (auto p = sec->touching_renderthings; p != nullptr; p = p->m_snext)
 		{
 			auto thing = p->m_thing;
-			if (thing->validcount == validcount) continue;
-			thing->validcount = validcount;
+			if (SeenActors.find(thing) != SeenActors.end()) continue;
+			SeenActors.insert(thing);
+			//if (thing->validcount == validcount) continue;
+			//thing->validcount = validcount;
 
 			FIntCVar *cvar = thing->GetClass()->distancecheck;
 			if (cvar != nullptr && *cvar >= 0)

src/swrenderer/scene/r_opaque_pass.h

@@ -18,6 +18,7 @@
 #include "r_defs.h"
 #include "swrenderer/line/r_line.h"
 #include "swrenderer/scene/r_3dfloors.h"
+#include <set>
 
 struct FVoxelDef;
 
@@ -84,5 +85,7 @@ namespace swrenderer
 		bool r_fakingunderwater = false;
 
 		SWRenderLine renderline;
+		std::set<sector_t*> SeenSpriteSectors;
+		std::set<AActor*> SeenActors;
 	};
 }

src/swrenderer/scene/r_scene.cpp

@@ -106,10 +106,9 @@ namespace swrenderer
 		if (CameraLight::Instance()->ShaderColormap() && viewport->RenderTarget->IsBgra() && !(r_shadercolormaps && screen->Accel2D))
 		{
 			MainThread()->DrawQueue->Push<ApplySpecialColormapRGBACommand>(CameraLight::Instance()->ShaderColormap(), screen);
-		}
-
 			RenderDrawQueues();
 		}
+	}
 
 	void RenderScene::RenderDrawQueues()
 	{
@@ -120,6 +119,9 @@ namespace swrenderer
 			queues.push_back((*it)->DrawQueue);
 		}
 		DrawerThreads::Execute(queues);
+
+		//using namespace std::chrono_literals;
+		//std::this_thread::sleep_for(0.5s);
 	}
 
 	void RenderScene::RenderActorView(AActor *actor, bool dontmaplines)
@@ -151,6 +153,7 @@ namespace swrenderer
 		}
 
 		RenderThreadSlices();
+		RenderDrawQueues();
 
 		camera->renderflags = savedflags;
 		interpolator.RestoreInterpolations();
@@ -165,7 +168,13 @@ namespace swrenderer
 
 	void RenderScene::RenderThreadSlices()
 	{
-		int numThreads = r_scene_multithreaded ? 8 : 1;
+		int numThreads = std::thread::hardware_concurrency();
+		if (numThreads == 0)
+			numThreads = 4;
+
+		if (!r_scene_multithreaded)
+			numThreads = 1;
+
 		if (numThreads != Threads.size())
 		{
 			StopThreads();
@@ -327,7 +336,6 @@ namespace swrenderer
 		viewport->SetViewport(width, height, WidescreenRatio);
 
 		RenderActorView(actor, dontmaplines);
-		RenderDrawQueues();
 		
 		viewport->RenderTarget = screen;
 

src/swrenderer/things/r_visiblesprite.cpp

@@ -77,6 +77,10 @@ namespace swrenderer
 		if (x1 >= x2)
 			return;
 
+		// Reject sprites outside the slice rendered by the thread
+		if (x2 < thread->X1 || x1 > thread->X2)
+			return;
+
 		// [RH] Sprites split behind a one-sided line can also be discarded.
 		if (spr->sector == nullptr)
 			return;