Make UpdateList a separate entity

Author: lhog

rts/Rendering/CMakeLists.txt

@@ -123,6 +123,7 @@ set(sources_engine_Rendering
 		"${CMAKE_CURRENT_SOURCE_DIR}/Common/ModelDrawerData.cpp"
 		"${CMAKE_CURRENT_SOURCE_DIR}/Common/ModelDrawerState.cpp"
 		"${CMAKE_CURRENT_SOURCE_DIR}/Common/ModelDrawerHelpers.cpp"
+		"${CMAKE_CURRENT_SOURCE_DIR}/Common/UpdateList.cpp"
 		"${CMAKE_CURRENT_SOURCE_DIR}/Features/FeatureDrawerData.cpp"
 		"${CMAKE_CURRENT_SOURCE_DIR}/Features/FeatureDrawer.cpp"
 		"${CMAKE_CURRENT_SOURCE_DIR}/Units/UnitDrawerData.cpp"

rts/Rendering/Env/Decals/GroundDecalHandler.cpp

@@ -69,13 +69,6 @@ CR_REG_METADATA_SUB(CGroundDecalHandler, UnitMinMaxHeight,
 	CR_MEMBER(max)
 ))
 
-CR_BIND(CGroundDecalHandler::DecalUpdateList, )
-CR_REG_METADATA_SUB(CGroundDecalHandler, DecalUpdateList,
-(
-	CR_MEMBER(updateList),
-	CR_MEMBER(changed)
-))
-
 CR_BIND_DERIVED(CGroundDecalHandler, IGroundDecalDrawer, )
 CR_REG_METADATA(CGroundDecalHandler, (
 	CR_MEMBER_UN(maxUniqueScars),
@@ -1673,64 +1666,4 @@ void CGroundDecalHandler::FeatureMoved(const CFeature* feature, const float3& ol
 void CGroundDecalHandler::UnitLoaded(const CUnit* unit, const CUnit* transport) { ForceRemoveSolidObject(unit); }
 void CGroundDecalHandler::UnitUnloaded(const CUnit* unit, const CUnit* transport) { AddSolidObject(unit); }
 
-void CGroundDecalHandler::UnitMoved(const CUnit* unit) { AddTrack(unit, unit->pos); }
-
-///////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
-
-void CGroundDecalHandler::DecalUpdateList::SetNeedUpdateAll()
-{
-	RECOIL_DETAILED_TRACY_ZONE;
-	std::fill(updateList.begin(), updateList.end(), true);
-	changed = true;
-}
-
-void CGroundDecalHandler::DecalUpdateList::ResetNeedUpdateAll()
-{
-	RECOIL_DETAILED_TRACY_ZONE;
-	std::fill(updateList.begin(), updateList.end(), false);
-	changed = false;
-}
-
-void CGroundDecalHandler::DecalUpdateList::SetUpdate(const CGroundDecalHandler::DecalUpdateList::IteratorPair& it)
-{
-	RECOIL_DETAILED_TRACY_ZONE;
-	std::fill(it.first, it.second, true);
-	changed = true;
-}
-
-void CGroundDecalHandler::DecalUpdateList::SetUpdate(size_t offset)
-{
-	RECOIL_DETAILED_TRACY_ZONE;
-	assert(offset < updateList.size());
-	updateList[offset] = true;
-	changed = true;
-}
-
-void CGroundDecalHandler::DecalUpdateList::EmplaceBackUpdate()
-{
-	RECOIL_DETAILED_TRACY_ZONE;
-	updateList.emplace_back(true);
-	changed = true;
-}
-
-std::optional<CGroundDecalHandler::DecalUpdateList::IteratorPair> CGroundDecalHandler::DecalUpdateList::GetNext(const std::optional<CGroundDecalHandler::DecalUpdateList::IteratorPair>& prev)
-{
-	RECOIL_DETAILED_TRACY_ZONE;
-	auto beg = prev.has_value() ? prev.value().second : updateList.begin();
-	     beg = std::find(beg, updateList.end(),  true);
-	auto end = std::find(beg, updateList.end(), false);
-
-	if (beg == end)
-		return std::nullopt;
-
-	return std::make_optional(std::make_pair(beg, end));
-}
-
-std::pair<size_t, size_t> CGroundDecalHandler::DecalUpdateList::GetOffsetAndSize(const CGroundDecalHandler::DecalUpdateList::IteratorPair& it)
-{
-	RECOIL_DETAILED_TRACY_ZONE;
-	return std::make_pair(
-		std::distance(updateList.begin(), it.first ),
-		std::distance(it.first          , it.second)
-	);
-}
+void CGroundDecalHandler::UnitMoved(const CUnit* unit) { AddTrack(unit, unit->pos); }

rts/Rendering/Env/Decals/GroundDecalHandler.h

@@ -14,6 +14,7 @@
 #include "Rendering/GL/VAO.h"
 #include "Rendering/DepthBufferCopy.h"
 #include "Rendering/Textures/TextureRenderAtlas.h"
+#include "Rendering/Common/UpdateList.h"
 #include "System/EventClient.h"
 #include "System/UnorderedMap.hpp"
 #include "System/creg/creg.h"
@@ -36,37 +37,6 @@ class CGroundDecalHandler: public IGroundDecalDrawer, public CEventClient, publi
 {
 	CR_DECLARE_DERIVED(CGroundDecalHandler)
 	CR_DECLARE_SUB(UnitMinMaxHeight)
-	CR_DECLARE_SUB(DecalUpdateList)
-public:
-	class DecalUpdateList {
-		CR_DECLARE_STRUCT(DecalUpdateList)
-	public:
-		using IteratorPair = std::pair<std::vector<bool>::iterator, std::vector<bool>::iterator>;
-	public:
-		DecalUpdateList()
-			: updateList()
-			, changed(true)
-		{}
-
-		void Resize(size_t newSize) { updateList.resize(newSize); SetNeedUpdateAll(); }
-		void Reserve(size_t reservedSize) { updateList.reserve(reservedSize); }
-
-		void SetUpdate(const IteratorPair& it);
-		void SetUpdate(size_t offset);
-
-		void SetNeedUpdateAll();
-		void ResetNeedUpdateAll();
-
-		void EmplaceBackUpdate();
-
-		bool NeedUpdate() const { return changed; }
-
-		std::optional<IteratorPair> GetNext(const std::optional<IteratorPair>& prev = std::nullopt);
-		std::pair<size_t, size_t> GetOffsetAndSize(const IteratorPair& it);
-	private:
-		std::vector<bool> updateList;
-		bool changed;
-	};
 public:
 	CGroundDecalHandler();
 	~CGroundDecalHandler() override;
@@ -195,7 +165,7 @@ class CGroundDecalHandler: public IGroundDecalDrawer, public CEventClient, publi
 	spring::unordered_map<uint32_t, size_t> idToPos;
 	spring::unordered_map<uint32_t, std::tuple<const CColorMap*, std::pair<size_t, size_t>>> idToCmInfo;
 
-	DecalUpdateList decalsUpdateList;
+	UpdateList decalsUpdateList;
 
 	uint32_t nextId;
 	std::vector<uint32_t> freeIds;