21053: Adds additional debugging capabilities related to AMALGAM_FAST_MEMORY_INTEGRITY and catching asserts

src/Amalgam/PlatformSpecific.h

@@ -223,23 +223,19 @@ inline void Platform_Assert(bool expr, const char *file, int line)
 		std::cerr << "Runtime Exception: Debug Assertion Failed at line " << line << " of " << file << "\n";
 
 	//platform dependent assertion function
-	#ifdef _DEBUG
-
-		#ifdef OS_WINDOWS
-			_ASSERT(expr);
-		#else
-			raise(SIGTRAP);
-		#endif
-			exit(-1);
-
+	#ifdef OS_WINDOWS
+		_ASSERT(expr);
 	#else
-		if(Platform_IsDebuggerPresent())
-		{
-			//wait for user input
-			std::string temp;
-			std::getline(std::cin, temp);
-		}
-		exit(-1);
+		raise(SIGTRAP);
 	#endif
+
+	if(Platform_IsDebuggerPresent())
+	{
+		//wait for user input in case the _ASSERT above was optimized out
+		std::string temp;
+		std::getline(std::cin, temp);
+	}
+
+	exit(-1);
 	}
 }

src/Amalgam/evaluablenode/EvaluableNode.cpp

@@ -1565,8 +1565,7 @@ void EvaluableNode::DestructValue()
 void EvaluableNode::Invalidate()
 {
 #ifdef AMALGAM_FAST_MEMORY_INTEGRITY
-	if(IsNodeDeallocated())
-		assert(false);
+	assert(!IsNodeDeallocated());
 #endif
 
 	if(!HasExtendedValue())

src/Amalgam/evaluablenode/EvaluableNode.h

@@ -239,35 +239,35 @@ class EvaluableNode
 	}
 
 	//Returns true if the node is some form of associative array
-	static constexpr bool IsAssociativeArray(EvaluableNode *n)
+	static __forceinline bool IsAssociativeArray(EvaluableNode *n)
 	{
 		if(n == nullptr)
 			return false;
 		return n->IsAssociativeArray();
 	}
 
 	//returns true if the type is immediate
-	constexpr bool IsImmediate()
+	__forceinline bool IsImmediate()
 	{
 		return IsEvaluableNodeTypeImmediate(GetType());
 	}
 
 	//Returns true if the node is some form of ordered array
-	constexpr bool IsOrderedArray()
+	__forceinline bool IsOrderedArray()
 	{
 		return DoesEvaluableNodeTypeUseOrderedData(GetType());
 	}
 
 	//Returns true if the node is some form of ordered array
-	static constexpr bool IsOrderedArray(EvaluableNode *n)
+	static __forceinline bool IsOrderedArray(EvaluableNode *n)
 	{
 		if(n == nullptr)
 			return false;
 		return n->IsOrderedArray();
 	}
 
 	//returns true if the EvaluableNode is of a query type
-	static constexpr bool IsQuery(EvaluableNode *n)
+	static __forceinline bool IsQuery(EvaluableNode *n)
 	{
 		return (n != nullptr && IsEvaluableNodeTypeQuery(n->GetType()));
 	}
@@ -419,13 +419,10 @@ class EvaluableNode
 	static size_t GetEstimatedNodeSizeInBytes(EvaluableNode *n);
 
 	//gets current type
-	constexpr EvaluableNodeType &GetType()
+	__forceinline EvaluableNodeType &GetType()
 	{
 	#ifdef AMALGAM_FAST_MEMORY_INTEGRITY
-		if(type == ENT_DEALLOCATED)
-		{
-			assert(false);
-		}
+		assert(type != ENT_DEALLOCATED);
 	#endif
 		return type;
 	}

src/Amalgam/evaluablenode/EvaluableNodeManagement.h

@@ -204,6 +204,10 @@ class EvaluableNodeStackStateSaver
 		stack = _stack;
 		originalStackSize = stack->size();
 
+	#ifdef AMALGAM_FAST_MEMORY_INTEGRITY
+		assert(initial_element == nullptr || !initial_element->IsNodeDeallocated());
+	#endif
+
 		stack->push_back(initial_element);
 	}
 
@@ -214,6 +218,9 @@ class EvaluableNodeStackStateSaver
 
 	__forceinline void PushEvaluableNode(EvaluableNode *n)
 	{
+	#ifdef AMALGAM_FAST_MEMORY_INTEGRITY
+		assert(n == nullptr || !n->IsNodeDeallocated());
+	#endif
 		stack->push_back(n);
 	}
 
@@ -231,6 +238,9 @@ class EvaluableNodeStackStateSaver
 	//replaces the position of the stack with new_value
 	__forceinline void SetStackLocation(size_t location, EvaluableNode *new_value)
 	{
+	#ifdef AMALGAM_FAST_MEMORY_INTEGRITY
+		assert(new_value == nullptr || !new_value->IsNodeDeallocated());
+	#endif
 		(*stack)[location] = new_value;
 	}