Multithreaded see casting

Author: Causeless

Source/Entities/Actor.cpp

@@ -1090,18 +1090,9 @@ void Actor::Update() {
 	// Update the viewpoint to be at least what the position is
 	m_ViewPoint = m_Pos;
 
-	// "See" the location and surroundings of this actor on the unseen map
-	// Todo - split MT safe and potentially expensive stuff like this in a seperate ThreadedUpdate for the C++ side
-	if (m_Status != Actor::INACTIVE) {
-		const int lookIterations = 6; // How many see rays to cast per frame
-		for (int i = 0; i < lookIterations; ++i) {
-			Look(45 * m_Perceptiveness, g_FrameMan.GetPlayerScreenWidth() * 0.51 * m_Perceptiveness);
-		}
-	}
-
 	// Check if the MO we're following still exists, and if not, then clear the destination
 	if (m_pMOMoveTarget && !g_MovableMan.ValidMO(m_pMOMoveTarget)) {
-		m_pMOMoveTarget = 0;
+		m_pMOMoveTarget = nullptr;
 	}
 
 	///////////////////////////////////////////////////////////////////////////////
@@ -1275,6 +1266,16 @@ void Actor::Update() {
 	}
 }
 
+void RTE::Actor::CastSeeRays() {
+	// "See" the location and surroundings of this actor on the unseen map
+	if (m_Status != Actor::INACTIVE) {
+		const int lookIterations = 6; // How many see rays to cast per frame
+		for (int i = 0; i < lookIterations; ++i) {
+			Look(45 * m_Perceptiveness, g_FrameMan.GetPlayerScreenWidth() * 0.51 * m_Perceptiveness);
+		}
+	}
+}
+
 void Actor::FullUpdate() {
 	PreControllerUpdate();
 	m_Controller.Update();

Source/Entities/Actor.h

@@ -664,6 +664,9 @@ namespace RTE {
 		/// Updates this MovableObject. Supposed to be done every frame.
 		void Update() override;
 
+		/// Cast see rays for this actor.
+		void CastSeeRays();
+
 		/// Updates the full state of this object in one call. (PreControllerUpdate(), Controller::Update(), and Update())
 		virtual void FullUpdate() override;
 

Source/Managers/MovableMan.cpp

@@ -1367,6 +1367,14 @@ void MovableMan::Update() {
 	g_PerformanceMan.StopPerformanceMeasurement(PerformanceMan::ScriptsUpdate);
 
 	{
+		auto actorsSeeFuture = g_ThreadMan.GetPriorityThreadPool().parallelize_loop(m_Actors.size(),
+																					[&](int start, int end) {
+																						ZoneScopedN("Actors See");
+																						for (int i = start; i < end; ++i) {
+																							m_Actors[i]->CastSeeRays();
+																						}
+																					});
+
 		{
 			ZoneScopedN("Actors Update");
 
@@ -1438,6 +1446,8 @@ void MovableMan::Update() {
 				particle->PostUpdate();
 			}
 		}
+
+		actorsSeeFuture.wait();
 	} // namespace RTE
 
 	//////////////////////////////////////////////////////////////////////