- Add polyobj support to r_scene_multithreaded

src/swrenderer/r_renderthread.cpp

@@ -103,23 +103,28 @@ namespace swrenderer
 		// calls to GetPixels for this to work.
 
 		static std::mutex loadmutex;
-		loadmutex.lock();
+
-		try
+		std::unique_lock<std::mutex> lock(loadmutex);
+
+		texture->GetPixels();
+		const FTexture::Span *spans;
+		texture->GetColumn(0, &spans);
+		if (Viewport->RenderTarget->IsBgra())
 		{
-			texture->GetPixels();
+			texture->GetPixelsBgra();
-			const FTexture::Span *spans;
+			texture->GetColumnBgra(0, &spans);
-			texture->GetColumn(0, &spans);
-			if (Viewport->RenderTarget->IsBgra())
-			{
-				texture->GetPixelsBgra();
-				texture->GetColumnBgra(0, &spans);
-			}
-			loadmutex.unlock();
 		}
-		catch (...)
+	}
+
+	void RenderThread::PreparePolyObject(subsector_t *sub)
+	{
+		static std::mutex polyobjmutex;
+
+		std::unique_lock<std::mutex> lock(polyobjmutex);
+
+		if (sub->BSP == nullptr || sub->BSP->bDirty)
 		{
-			loadmutex.unlock();
+			sub->BuildPolyBSP();
-			throw;
 		}
 	}
 }

src/swrenderer/r_renderthread.h

@@ -83,6 +83,9 @@ namespace swrenderer
 
 		// Make sure texture can accessed safely
 		void PrepareTexture(FTexture *texture);
+
+		// Setup poly object in a threadsafe manner
+		void PreparePolyObject(subsector_t *sub);
 		
 	private:
 		std::unique_ptr<SWTruecolorDrawers> tc_drawers;

src/swrenderer/scene/r_opaque_pass.cpp

@@ -390,10 +390,8 @@ namespace swrenderer
 
 	void RenderOpaquePass::AddPolyobjs(subsector_t *sub)
 	{
-		if (sub->BSP == nullptr || sub->BSP->bDirty)
+		Thread->PreparePolyObject(sub);
-		{
+
-			sub->BuildPolyBSP();
-		}
 		if (sub->BSP->Nodes.Size() == 0)
 		{
 			RenderSubsector(&sub->BSP->Subsectors[0]);