fix VC7 warnings

stepmania/src/ActorScroller.cpp

@@ -103,7 +103,7 @@ void ActorScroller::DrawPrimitives()
 	}
 	else
 	{
-		for( int i=0; i<m_SubActors.size(); i++ )
+		for( unsigned i=0; i<m_SubActors.size(); i++ )
 		{
 			float fItemOffset = i - m_fCurrentItem;
 

stepmania/src/ScreenSMOnlineLogin.cpp

@@ -39,7 +39,7 @@ ScreenSMOnlineLogin::ScreenSMOnlineLogin( const CString& sName ) : ScreenWithMen
 
 	m_textUserName.LoadFromFont( THEME->GetPathF(m_sName,"text") );
 	m_textUserName.SetName( "UserName" );
-	m_textUserName.SetWrapWidthPixels( SCREEN_WIDTH );
+	m_textUserName.SetWrapWidthPixels( (int)SCREEN_WIDTH );
 	SET_XY_AND_ON_COMMAND( m_textUserName );
 	this->AddChild( &m_textUserName );
 
@@ -192,7 +192,7 @@ void ScreenSMOnlineLogin::UpdateTextInput()
 {
 	CString PasswordText;
 
-	for ( int i = 0; i < m_sPassword.length(); ++i )
+	for ( unsigned i = 0; i < m_sPassword.length(); ++i )
 		PasswordText+='*';
 
 	m_textPassword.SetText( PasswordText );  

stepmania/src/ScreenSelectMaster.cpp

@@ -475,8 +475,6 @@ bool ScreenSelectMaster::ChangeSelection( PlayerNumber pn, int iNewChoice )
 	if( GetPage(m_iChoice[pn]) != GetPage(iNewChoice) )
 		return ChangePage( iNewChoice );
 
-	bool bMoveAll = SHARED_PREVIEW_AND_CURSOR || GetCurrentPage()!=PAGE_1;
-
 	FOREACH_PlayerNumber( p )
 	{
 		const int iOldChoice = m_iChoice[p];

stepmania/src/XmlFile.cpp

@@ -699,7 +699,6 @@ const XNode *XNode::GetChild( const char* name ) const
 	multimap<CString, XNode*>::const_iterator it = m_childs.find( name );
 	if( it != m_childs.end() )
 	{
-		XNode* p = it->second;
 		DEBUG_ASSERT( name == it->second->m_sName );
 		return it->second;
 	}

stepmania/src/XmlFile.h

@@ -31,30 +31,30 @@ typedef multimap<CString,XNode*> XNodes;
 #define FOREACH_Attr( pNode, Var ) \
 	XAttrs::iterator Var##Iter; \
 	XAttr *Var = NULL; \
-	for( Var##Iter = (pNode)->m_attrs.begin(); \
+	for( Var##Iter = (pNode)->m_attrs.begin(), Var = Var##Iter->second; \
-		(Var##Iter != (pNode)->m_attrs.end() && (Var = Var##Iter->second) ),  Var##Iter != (pNode)->m_attrs.end(); \
+		Var##Iter != (pNode)->m_attrs.end(); \
-		++Var##Iter )
+		++Var##Iter, Var = Var##Iter->second )
 
 #define FOREACH_CONST_Attr( pNode, Var ) \
 	XAttrs::const_iterator Var##Iter; \
 	const XAttr *Var = NULL; \
-	for( Var##Iter = (pNode)->m_attrs.begin(); \
+	for( Var##Iter = (pNode)->m_attrs.begin(), Var = Var##Iter->second; \
-		(Var##Iter != (pNode)->m_attrs.end() && (Var = Var##Iter->second) ),  Var##Iter != (pNode)->m_attrs.end(); \
+		Var##Iter != (pNode)->m_attrs.end(); \
-		++Var##Iter )
+		++Var##Iter, Var = Var##Iter->second )
 
 #define FOREACH_Child( pNode, Var ) \
 	XNodes::iterator Var##Iter; \
 	XNode *Var = NULL; \
-	for( Var##Iter = (pNode)->m_childs.begin(); \
+	for( Var##Iter = (pNode)->m_childs.begin(), Var = Var##Iter->second; \
-		(Var##Iter != (pNode)->m_childs.end() && (Var = Var##Iter->second) ),  Var##Iter != (pNode)->m_childs.end(); \
+		Var##Iter != (pNode)->m_childs.end(); \
-		++Var##Iter )
+		++Var##Iter, Var = Var##Iter->second )
 
 #define FOREACH_CONST_Child( pNode, Var ) \
 	XNodes::const_iterator Var##Iter; \
 	const XNode *Var = NULL; \
-	for( Var##Iter = (pNode)->m_childs.begin(); \
+	for( Var##Iter = (pNode)->m_childs.begin(), Var = Var##Iter->second; \
-		(Var##Iter != (pNode)->m_childs.end() && (Var = Var##Iter->second) ),  Var##Iter != (pNode)->m_childs.end(); \
+		Var##Iter != (pNode)->m_childs.end(); \
-		++Var##Iter )
+		++Var##Iter, Var = Var##Iter->second )
 
 
 // Entity Encode/Decode Support