[CORE] Refactor XferCRC to make it branchless and to remove winsock dependency

Author: Mauller

Core/GameEngine/Include/Common/XferCRC.h

@@ -64,7 +64,7 @@ class XferCRC : public Xfer
 
 	virtual void xferImplementation( void *data, Int dataSize );
 
-	void addCRC( UnsignedInt val );								///< CRC a 4-byte block
+	inline void addCRC( UnsignedInt val );								///< CRC a 4-byte block
 
 	UnsignedInt m_crc;
 

Core/GameEngine/Source/Common/System/XferCRC.cpp

@@ -34,7 +34,7 @@
 #include "Common/XferDeepCRC.h"
 #include "Common/crc.h"
 #include "Common/Snapshot.h"
-#include "winsock2.h" // for htonl
+#include "utility/endian_compat.h"
 
 //-------------------------------------------------------------------------------------------------
 //-------------------------------------------------------------------------------------------------
@@ -97,22 +97,8 @@ void XferCRC::endBlock( void )
 //-------------------------------------------------------------------------------------------------
 void XferCRC::addCRC( UnsignedInt val )
 {
-	int hibit;
 
-	val = htonl(val);
-
-	if (m_crc & 0x80000000)
-	{
-		hibit = 1;
-	}
-	else
-	{
-		hibit = 0;
-	}
-
-	m_crc <<= 1;
-	m_crc += val;
-	m_crc += hibit;
+	m_crc = (m_crc << 1) + htobe(val) +((m_crc >> 31) & 0x01);
 
 }  // end addCRC
 
@@ -140,30 +126,35 @@ void XferCRC::xferSnapshot( Snapshot *snapshot )
 void XferCRC::xferImplementation( void *data, Int dataSize )
 {
 
-	if (!data || dataSize < 1)
-	{
-		return;
-	}
-
 	const UnsignedInt *uintPtr = (const UnsignedInt *) (data);
+	int validData = (data != NULL) & (dataSize > 0);
+	int dataBytes = (dataSize / 4) * validData;
 
-	for (Int i=0 ; i<dataSize/4 ; i++)
+	Int i = 0;
+	for (i=0 ; i<dataBytes; i++)
 	{
 		addCRC (*uintPtr++);
 	}
 
 	int leftover = dataSize & 3;
-	if (leftover)
+	validData = (data != NULL) & (leftover > 0);
+
+	UnsignedInt val = 0;
+	const unsigned char *c = (const unsigned char *)uintPtr;
+
+	switch(leftover * validData)
 	{
-		UnsignedInt val = 0;
-		const unsigned char *c = (const unsigned char *)uintPtr;
-		for (Int i=0; i<leftover; i++)
-		{
-			val += (c[i] << (i*8));
-		}
-		val = htonl(val);
-		addCRC (val);
+	case 3:
+		val += (c[2] << (2 * 8));
+	case 2:
+		val += (c[1] << (1 * 8));
+	case 1:
+		val += (c[0] << (0 * 8));
+	default:
+		break;
 	}
+
+	m_crc = (m_crc << (1*validData)) + (val +((m_crc >> 31) & 0x01)) * validData;
 
 }  // end xferImplementation
 
@@ -179,7 +170,7 @@ void XferCRC::skip( Int dataSize )
 UnsignedInt XferCRC::getCRC( void )
 {
 
-	return htonl(m_crc);
+	return htobe(m_crc);
 
 }  // end skip