Compilation adjustments for GCC/Clang

These compilers are more strict than MSVC, so fix the compilation errors coming from the linux build. Signed-off-by: Pavel Solodovnikov <[email protected]>
Warzone2100 · Jan 23, 2024 · 12585b9 · 12585b9
1 parent 2cd2eb2
commit 12585b9
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 5 deletions.
diff --git a/lib/framework/frameresource.cpp b/lib/framework/frameresource.cpp
@@ -33,6 +33,7 @@
 #include "resly.h"
 
 #include <list>
+#include <algorithm>
 
 // Local prototypes
 static std::list<RES_TYPE *> psResTypes;

diff --git a/lib/framework/object_list_iteration.h b/lib/framework/object_list_iteration.h
@@ -24,6 +24,7 @@
 #include <list>
 #include <type_traits>
 #include <iterator>
+#include <functional>
 
 enum class IterationResult
 {
@@ -67,7 +68,7 @@ struct LoopBodyHandlerCallStrategy
 	template <typename ObjectType>
 	static constexpr bool handler_accepts_iter = std::is_convertible<
 		Callable,
-		std::function<IterationResult(std::list<ObjectType*>::iterator)>>::value;
+		std::function<IterationResult(typename std::list<ObjectType*>::iterator)>>::value;
 
 	// `Invoke` overload for Callable taking a list iterator as the argument
 	template <typename ObjectType>
@@ -95,8 +96,8 @@ void mutating_list_iterate(std::list<ObjectType*>& list, MaybeErasingLoopBodyHan
 	using HandlerCallStrategy = LoopBodyHandlerCallStrategy<MaybeErasingLoopBodyHandler>;
 
 	static_assert(
-		   HandlerCallStrategy::handler_accepts_ptr<ObjectType>
-		|| HandlerCallStrategy::handler_accepts_iter<ObjectType>,
+		   HandlerCallStrategy::template handler_accepts_ptr<ObjectType>
+		|| HandlerCallStrategy::template handler_accepts_iter<ObjectType>,
 		"Unsupported loop body handler signature: "
 		"should return IterationResult and take either an ObjectType* or an iterator");
 
@@ -110,7 +111,7 @@ void mutating_list_iterate(std::list<ObjectType*>& list, MaybeErasingLoopBodyHan
 	{
 		itNext = std::next(it);
 		// Can possibly invalidate `it` and anything before it.
-		const auto res = HandlerCallStrategy::Invoke<ObjectType>(handler, it);
+		const auto res = HandlerCallStrategy::template Invoke<ObjectType>(handler, it);
 		if (res == IterationResult::BREAK_ITERATION)
 		{
 			break;

diff --git a/src/objmem.h b/src/objmem.h
@@ -28,7 +28,6 @@
 
 #include <array>
 #include <list>
-#include <functional>
 
 /* The lists of objects allocated */
 template <typename ObjectType, unsigned PlayerCount>