From 008386fc7fc7182f145e7129c5ed671bd5c83e39 Mon Sep 17 00:00:00 2001
From: Scott Mansell <phiren@gmail.com>
Date: Wed, 4 Nov 2015 22:44:19 +1300
Subject: [PATCH] Just a minor cleanup. Totally not the cause of random FifoCi
 failures.

This 'absolutely not' a read of uninitialized memory is '100% not'
the cause of our non-deterministic behavior on the Intel NUC.

If there was a such an error, it would show up on all FifoCi
backends equally. This is 'probably' unrelated to the fact that
the Intel NUC is the only fifoci runner not running under virtualization.
---
 Source/Core/Core/FifoPlayer/FifoDataFile.cpp | 3 ++-
 Source/Core/Core/FifoPlayer/FifoDataFile.h   | 2 +-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/Source/Core/Core/FifoPlayer/FifoDataFile.cpp b/Source/Core/Core/FifoPlayer/FifoDataFile.cpp
index 0168303bf2..2ce5cf9e66 100644
--- a/Source/Core/Core/FifoPlayer/FifoDataFile.cpp
+++ b/Source/Core/Core/FifoPlayer/FifoDataFile.cpp
@@ -30,7 +30,7 @@ FifoDataFile::~FifoDataFile()
 
 bool FifoDataFile::HasBrokenEFBCopies() const
 {
-	return version < 2;
+	return m_Version < 2;
 }
 
 void FifoDataFile::SetIsWii(bool isWii)
@@ -150,6 +150,7 @@ FifoDataFile* FifoDataFile::Load(const std::string &filename, bool flagsOnly)
 	FifoDataFile* dataFile = new FifoDataFile;
 
 	dataFile->m_Flags = header.flags;
+	dataFile->m_Version = header.min_loader_version;
 
 	if (flagsOnly)
 	{
diff --git a/Source/Core/Core/FifoPlayer/FifoDataFile.h b/Source/Core/Core/FifoPlayer/FifoDataFile.h
index d45e58f024..8db380033e 100644
--- a/Source/Core/Core/FifoPlayer/FifoDataFile.h
+++ b/Source/Core/Core/FifoPlayer/FifoDataFile.h
@@ -94,7 +94,7 @@ private:
 	u32 m_XFRegs[XF_REGS_SIZE];
 
 	u32 m_Flags;
-	u32 version;
+	u32 m_Version;
 
 	std::vector<FifoFrameInfo> m_Frames;
 };