From 03708b636acaa8ba343c3c7ecbe1a694572e29cb Mon Sep 17 00:00:00 2001
From: Glenn Maynard <gmaynard@users.sourceforge.net>
Date: Sat, 10 Dec 2005 02:11:34 +0000
Subject: [PATCH] firming up this API: USBStorageDevicesChanged is "calling
 GetUSBStorageDevices may return different devices"; calling it twice in a row
 should have the same result

---
 .../MemoryCardDriverThreaded_Windows.cpp          | 15 ++++-----------
 1 file changed, 4 insertions(+), 11 deletions(-)

diff --git a/stepmania/src/arch/MemoryCard/MemoryCardDriverThreaded_Windows.cpp b/stepmania/src/arch/MemoryCard/MemoryCardDriverThreaded_Windows.cpp
index 0d4ed1e231..3dcfbf9102 100644
--- a/stepmania/src/arch/MemoryCard/MemoryCardDriverThreaded_Windows.cpp
+++ b/stepmania/src/arch/MemoryCard/MemoryCardDriverThreaded_Windows.cpp
@@ -73,6 +73,9 @@ static bool IsFloppyDrive( char c )
 
 void MemoryCardDriverThreaded_Windows::GetUSBStorageDevices( vector<UsbStorageDevice>& vDevicesOut )
 {
+	DWORD dwLogicalDrives = ::GetLogicalDrives();
+	m_dwLastLogicalDrives = dwLogicalDrives;
+
 	const int MAX_DRIVES = 26;
 	for( int i=0; i<MAX_DRIVES; ++i )
 	{
@@ -115,17 +118,7 @@ void MemoryCardDriverThreaded_Windows::GetUSBStorageDevices( vector<UsbStorageDe
 
 bool MemoryCardDriverThreaded_Windows::USBStorageDevicesChanged()
 {
-	/* Nothing needs a write test (or we can't do it right now).  If no devices
-	 * have changed, either, we have nothing to do. */
-	DWORD dwLogicalDrives = ::GetLogicalDrives();
-	if( dwLogicalDrives != m_dwLastLogicalDrives )
-	{
-		m_dwLastLogicalDrives = dwLogicalDrives;
-		return true;
-	}
-
-	/* Nothing to do. */
-	return false;
+	return ::GetLogicalDrives() != m_dwLastLogicalDrives;
 }
 
 bool MemoryCardDriverThreaded_Windows::Mount( UsbStorageDevice* pDevice )