ChunkFile has allowed me to accidentally "Do" a non-POD for the last time!

Source/Core/Common/Src/ChunkFile.h

@@ -32,6 +32,7 @@
 #include <list>
 #include <deque>
 #include <string>
+#include <type_traits>
 
 #include "Common.h"
 #include "FileUtil.h"
@@ -139,12 +140,18 @@ public:
 	template <typename T>
 	void Do(T& x)
 	{
-		// TODO: Bad, Do(some_non_POD) will compile and fail at runtime
+		// Ideally this would be std::is_trivially_copyable, but not enough support yet
-		// type_traits are not fully supported everywhere yet
+		static_assert(std::is_pod<T>::value, "Only sane for POD types");
 		
 		DoVoid((void*)&x, sizeof(x));
 	}
 	
+	template <typename T>
+	void DoPOD(T& x)
+	{
+		DoVoid((void*)&x, sizeof(x));
+	}
+
 	template <typename T>
 	void DoPointer(T*& x, T* const base)
 	{

Source/Core/Core/Src/HW/AudioInterface.cpp

@@ -131,8 +131,8 @@ static unsigned int g_AIDSampleRate = 32000;
 
 void DoState(PointerWrap &p)
 {
-	p.Do(m_Control);
+	p.DoPOD(m_Control);
-	p.Do(m_Volume);
+	p.DoPOD(m_Volume);
 	p.Do(m_SampleCounter);
 	p.Do(m_InterruptTiming);
 	p.Do(g_LastCPUTime);

Source/Core/Core/Src/HW/DSP.cpp

@@ -223,9 +223,9 @@ void DoState(PointerWrap &p)
 {
 	if (!g_ARAM.wii_mode)
 		p.DoArray(g_ARAM.ptr, g_ARAM.size);
-	p.Do(g_dspState);
+	p.DoPOD(g_dspState);
-	p.Do(g_audioDMA);
+	p.DoPOD(g_audioDMA);
-	p.Do(g_arDMA);
+	p.DoPOD(g_arDMA);
 	p.Do(g_ARAM_Info);
 	p.Do(g_AR_MODE);
 	p.Do(g_AR_REFRESH);

Source/Core/Core/Src/HW/DSPHLE/DSPHLE.cpp

@@ -157,8 +157,8 @@ void DSPHLE::DoState(PointerWrap &p)
 		}
 	}
 
-	p.Do(m_DSPControl);
+	p.DoPOD(m_DSPControl);
-	p.Do(m_dspState);
+	p.DoPOD(m_dspState);
 
 	int ucode_crc = IUCode::GetCRC(m_pUCode);
 	int ucode_crc_beforeLoad = ucode_crc;

Source/Core/Core/Src/HW/DVDInterface.cpp

@@ -229,14 +229,14 @@ void ExecuteCommand(UDICR& _DICR);
 
 void DoState(PointerWrap &p)
 {
-	p.Do(m_DISR);
+	p.DoPOD(m_DISR);
-	p.Do(m_DICVR);
+	p.DoPOD(m_DICVR);
 	p.DoArray(m_DICMDBUF, 3);
 	p.Do(m_DIMAR);
 	p.Do(m_DILENGTH);
 	p.Do(m_DICR);
 	p.Do(m_DIIMMBUF);
-	p.Do(m_DICFG);
+	p.DoPOD(m_DICFG);
 
 	p.Do(LoopStart);
 	p.Do(AudioPos);

Source/Core/Core/Src/HW/EXI_Channel.cpp

@@ -282,7 +282,7 @@ void CEXIChannel::Write32(const u32 _iValue, const u32 _iRegister)
 
 void CEXIChannel::DoState(PointerWrap &p)
 {
-	p.Do(m_Status);
+	p.DoPOD(m_Status);
 	p.Do(m_DMAMemoryAddress);
 	p.Do(m_DMALength);
 	p.Do(m_Control);

Source/Core/Core/Src/HW/SI.cpp

@@ -257,8 +257,8 @@ void DoState(PointerWrap &p)
 		}
 	}
 	p.Do(g_Poll);
-	p.Do(g_ComCSR);
+	p.DoPOD(g_ComCSR);
-	p.Do(g_StatusReg);
+	p.DoPOD(g_StatusReg);
 	p.Do(g_EXIClockCount);
 	p.Do(g_SIBuffer);
 }	

Source/Core/Core/Src/HW/VideoInterface.cpp

@@ -72,8 +72,8 @@ static int fields = 2;
 
 void DoState(PointerWrap &p)
 {
-	p.Do(m_VerticalTimingRegister);
+	p.DoPOD(m_VerticalTimingRegister);
-	p.Do(m_DisplayControlRegister);
+	p.DoPOD(m_DisplayControlRegister);
 	p.Do(m_HTiming0);
 	p.Do(m_HTiming1);
 	p.Do(m_VBlankTimingOdd);
@@ -89,7 +89,7 @@ void DoState(PointerWrap &p)
 	p.DoArray(m_InterruptRegister, 4);
 	p.DoArray(m_LatchRegister, 2);
 	p.Do(m_HorizontalStepping);
-	p.Do(m_HorizontalScaling);
+	p.DoPOD(m_HorizontalScaling);
 	p.Do(m_FilterCoefTables);
 	p.Do(m_UnkAARegister);
 	p.Do(m_Clock);

Source/Core/Core/Src/HW/WII_IPC.cpp

@@ -111,7 +111,7 @@ void DoState(PointerWrap &p)
 {
 	p.Do(ppc_msg);
 	p.Do(arm_msg);
-	p.Do(ctrl);
+	p.DoPOD(ctrl);
 	p.Do(ppc_irq_flags);
 	p.Do(ppc_irq_masks);
 	p.Do(arm_irq_flags);

Source/Core/Core/Src/IPC_HLE/WII_IPC_HLE_Device_usb.cpp

@@ -109,8 +109,8 @@ void CWII_IPC_HLE_Device_usb_oh1_57e_305::DoState(PointerWrap &p)
 	p.Do(m_ControllerBD);
 	p.Do(m_CtrlSetup);
 	p.Do(m_ACLSetup);
-	p.Do(m_HCIEndpoint);
+	p.DoPOD(m_HCIEndpoint);
-	p.Do(m_ACLEndpoint);
+	p.DoPOD(m_ACLEndpoint);
 	p.Do(m_last_ticks);
 	p.DoArray(m_PacketCount,4);
 	p.Do(m_ScanEnable);

Source/Core/Core/Src/IPC_HLE/WII_IPC_HLE_Device_usb.h

@@ -51,6 +51,12 @@ struct SQueuedEvent
 	}
 };
 
+// Hacks for ChunkFile to accept SQueuedEvent as POD
+namespace std
+{
+template <>
+struct is_pod<SQueuedEvent> : std::true_type {};
+}
 
 // Important to remember that this class is for /dev/usb/oh1/57e/305 ONLY
 // /dev/usb/oh1 -> internal usb bus

Source/Core/Core/Src/PowerPC/PowerPC.cpp

@@ -80,7 +80,7 @@ void DoState(PointerWrap &p)
 //	rSPR(SPR_DEC) = SystemTimers::GetFakeDecrementer();
 //	*((u64 *)&TL) = SystemTimers::GetFakeTimeBase(); //works since we are little endian and TL comes first :)
 
-	p.Do(ppcState);
+	p.DoPOD(ppcState);
 
 //	SystemTimers::DecrementerSet();
 //	SystemTimers::TimeBaseSet();

Source/Core/DiscIO/Src/WbfsBlob.cpp

@@ -133,7 +133,7 @@ bool WbfsFileReader::Read(u64 offset, u64 nbytes, u8* out_ptr)
 {
 	while(nbytes)
 	{
-		u64 read_size;
+		u64 read_size = 0;
 		File::IOFile& data_file = SeekToCluster(offset, &read_size);
 		read_size = (read_size > nbytes) ? nbytes : read_size;
 

Source/Core/VideoCommon/Src/CommandProcessor.cpp

@@ -80,9 +80,9 @@ void UpdateInterrupts_Wrapper(u64 userdata, int cyclesLate)
 
 void DoState(PointerWrap &p)
 {
-	p.Do(m_CPStatusReg);
+	p.DoPOD(m_CPStatusReg);
-	p.Do(m_CPCtrlReg);
+	p.DoPOD(m_CPCtrlReg);
-	p.Do(m_CPClearReg);
+	p.DoPOD(m_CPClearReg);
 	p.Do(m_bboxleft);
 	p.Do(m_bboxtop);
 	p.Do(m_bboxright);

Source/Core/VideoCommon/Src/PixelEngine.cpp

@@ -137,7 +137,7 @@ void DoState(PointerWrap &p)
 	p.Do(m_DstAlphaConf);
 	p.Do(m_AlphaModeConf);
 	p.Do(m_AlphaRead);
-	p.Do(m_Control);
+	p.DoPOD(m_Control);
 
 	p.Do(g_bSignalTokenInterrupt);
 	p.Do(g_bSignalFinishInterrupt);

Source/Plugins/Plugin_VideoSoftware/Src/SWCommandProcessor.cpp

@@ -56,7 +56,7 @@ CPReg cpreg; // shared between gfx and emulator thread
 
 void DoState(PointerWrap &p)
 {
-	p.Do(cpreg);
+	p.DoPOD(cpreg);
 	p.DoArray(commandBuffer, commandBufferSize);
 	p.Do(readPos);
 	p.Do(writePos);

Source/Plugins/Plugin_VideoSoftware/Src/SWPixelEngine.cpp

@@ -50,7 +50,7 @@ static int et_SetFinishOnMainThread;
 
 void DoState(PointerWrap &p)
 {
-    p.Do(pereg);
+    p.DoPOD(pereg);
 	p.Do(g_bSignalTokenInterrupt);
 	p.Do(g_bSignalFinishInterrupt);
 	p.Do(et_SetTokenOnMainThread);

Source/Plugins/Plugin_VideoSoftware/Src/SWmain.cpp

@@ -112,7 +112,7 @@ void VideoSoftware::DoState(PointerWrap& p)
 	Clipper::DoState(p);
 	p.Do(swxfregs);
     p.Do(bpmem);
-	p.Do(swstats);
+	p.DoPOD(swstats);
 
 	// CP Memory
     p.DoArray(arraybases, 16);