From e892ecc0274096f9c8edb3aaed6c2b491a6bc6a1 Mon Sep 17 00:00:00 2001
From: Chris Danford <chrisdanford@users.sourceforge.net>
Date: Sat, 2 Aug 2003 20:05:46 +0000
Subject: [PATCH] use ARRAYSIZE macro

---
 stepmania/src/InputMapper.cpp     | 3 +--
 stepmania/src/NotesLoader.cpp     | 4 ++--
 stepmania/src/OptionIcon.cpp      | 3 +--
 stepmania/src/OptionIconRow.cpp   | 3 +--
 stepmania/src/RageUtil.h          | 2 +-
 stepmania/src/ScreenNameEntry.cpp | 2 +-
 6 files changed, 7 insertions(+), 10 deletions(-)

diff --git a/stepmania/src/InputMapper.cpp b/stepmania/src/InputMapper.cpp
index b9078617b9..a154fc7116 100644
--- a/stepmania/src/InputMapper.cpp
+++ b/stepmania/src/InputMapper.cpp
@@ -225,7 +225,6 @@ const AutoJoyMapping g_AutoJoyMappings[] =
 		}
 	},
 };
-const int NUM_AUTO_JOY_MAPPINGS = ARRAYSIZE(g_AutoJoyMappings);
 
 void InputMapper::AutoMapJoysticksForCurrentGame()
 {
@@ -240,7 +239,7 @@ void InputMapper::AutoMapJoysticksForCurrentGame()
 	{
 		InputDevice device = vDevices[i];
 		CString sDescription = vDescriptions[i];
-		for( int j=0; j<NUM_AUTO_JOY_MAPPINGS; j++ )
+		for( int j=0; j<ARRAYSIZE(g_AutoJoyMappings); j++ )
 		{
 			const AutoJoyMapping& mapping = g_AutoJoyMappings[j];
 
diff --git a/stepmania/src/NotesLoader.cpp b/stepmania/src/NotesLoader.cpp
index 88eaaa445e..773436cce0 100644
--- a/stepmania/src/NotesLoader.cpp
+++ b/stepmania/src/NotesLoader.cpp
@@ -13,8 +13,8 @@ bool NotesLoader::Loadable( CString sPath )
 void NotesLoader::GetMainAndSubTitlesFromFullTitle( const CString sFullTitle, CString &sMainTitleOut, CString &sSubTitleOut )
 {
 	const CString sLeftSeps[]  = { " -", " ~", " (", " [" };
-	int iNumSeps = sizeof(sLeftSeps)/sizeof(CString);
-	for( int i=0; i<iNumSeps; i++ )
+
+	for( int i=0; i<ARRAYSIZE(sLeftSeps); i++ )
 	{
 		int iBeginIndex = sFullTitle.Find( sLeftSeps[i] );
 		if( iBeginIndex == -1 )
diff --git a/stepmania/src/OptionIcon.cpp b/stepmania/src/OptionIcon.cpp
index bb9678cec1..626beae1dd 100644
--- a/stepmania/src/OptionIcon.cpp
+++ b/stepmania/src/OptionIcon.cpp
@@ -41,9 +41,8 @@ OptionIcon::OptionIcon()
 void OptionIcon::Load( PlayerNumber pn, CString sText, bool bHeader )
 {
 	static CString sStopWords[] = { "OFF", "VISIBLE", "VIVID", "STANDARD", "X1", "HOLDS", "DEFAULT", "OVERHEAD" };
-	const int iNumStopWords = sizeof(sStopWords)/sizeof(sStopWords[0]);
 	
-	for( int i=0; i<iNumStopWords; i++ )
+	for( int i=0; i<ARRAYSIZE(sStopWords); i++ )
 		if( 0==stricmp(sText,sStopWords[i]) )
 			sText = "";
 
diff --git a/stepmania/src/OptionIconRow.cpp b/stepmania/src/OptionIconRow.cpp
index 619870d9a1..c85c7dae59 100644
--- a/stepmania/src/OptionIconRow.cpp
+++ b/stepmania/src/OptionIconRow.cpp
@@ -68,7 +68,6 @@ const OptionColumnEntry g_OptionColumnEntries[] =
 
 	{"TimingAssist",-1},
 };
-const int NUM_OPTION_COL_ENTRIES = sizeof(g_OptionColumnEntries)/sizeof(OptionColumnEntry);
 
 int OptionToPreferredColumn( CString sOptionText )
 {
@@ -79,7 +78,7 @@ int OptionToPreferredColumn( CString sOptionText )
 			return 0;
 	}
 
-	for( int i=0; i<NUM_OPTION_COL_ENTRIES; i++ )
+	for( int i=0; i<ARRAYSIZE(g_OptionColumnEntries); i++ )
 		if( g_OptionColumnEntries[i].szString == sOptionText )
 			return g_OptionColumnEntries[i].iSlotIndex;
 
diff --git a/stepmania/src/RageUtil.h b/stepmania/src/RageUtil.h
index 84a51d74ce..f62f2dedf6 100644
--- a/stepmania/src/RageUtil.h
+++ b/stepmania/src/RageUtil.h
@@ -23,7 +23,7 @@
 #define SAFE_RELEASE(p)      { if(p) { (p)->Release(); (p)=NULL; } }
 
 #define ZERO(x)	memset(&x, 0, sizeof(x))
-#define COPY(a,b) { ASSERT(sizeof(a)==sizeof(b)); memcpy(&a, &b, sizeof(a)); }
+#define COPY(a,b) { ASSERT(sizeof(a)==sizeof(b)); memcpy(&(a), &(b), sizeof(a)); }
 #define ARRAYSIZE(a) (sizeof(a) / sizeof((a)[0]))
 
 /* Common harmless mismatches.  All min(T,T) and max(T,T) cases are handled
diff --git a/stepmania/src/ScreenNameEntry.cpp b/stepmania/src/ScreenNameEntry.cpp
index c3c5727f19..47ddb077ae 100644
--- a/stepmania/src/ScreenNameEntry.cpp
+++ b/stepmania/src/ScreenNameEntry.cpp
@@ -60,7 +60,7 @@ const char NAME_CHARS[] =
 {
 	' ',' ',' ',' ','A','B','C','D','E','F','G','H','I','J','K','L','M','N','O','P','Q','R','S','T','U','V','W','X','Y','Z'
 };
-#define NUM_NAME_CHARS (sizeof(NAME_CHARS)/sizeof(char))
+#define NUM_NAME_CHARS (ARRAYSIZE(NAME_CHARS))
 #define HEIGHT_OF_ALL_CHARS		(NUM_NAME_CHARS * g_fCharsSpacingY)