actions/itgmania212121
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

absolute coordinate check: constants with no offset are ok

Browse Source
This commit is contained in:
Chris Danford
2004-11-06 06:54:04 +00:00
parent b50eb9399d
commit 0f7a5c873d
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
stepmania/src/Actor.cpp
+1 -1
View File
@@ -634,7 +634,7 @@ void Actor::HandleCommand( const ParsedCommand &command )
#if defined(DEBUG)
if( sName == "x" || sName == "y" ) // an absolute X or Y position
{
if( sParam(1).Find('-') == -1 && sParam(1).Find('+') == -1 )
if( isdigit(sParam(1)[0]) && sParam(1).Find('-') == -1 && sParam(1).Find('+') == -1 )
{
LOG->Warn( "Command '%s' should contain a SCREEN_* constant", command.GetOriginalCommandString().c_str() );
}
stepmania/src/ThemeManager.cpp
+1 -1
View File
@@ -624,7 +624,7 @@ float ThemeManager::GetMetricF( const CString &sClassName, const CString &sValue
#if defined(DEBUG)
if( sValueName.Right(1) == "X" || sValueName.Right(1) == "Y" ) // an absolute X or Y position
{
if( sValue.Find('-') == -1 && sValue.Find('+') == -1 )
if( isdigit(sValue[0]) && sValue.Find('-') == -1 && sValue.Find('+') == -1 )
{
LOG->Warn( "Absolute position metric '%s'-'%s' should contain a SCREEN_* constant", sClassName.c_str(), sValueName.c_str() );
}