Changed this during testing so I didn't have to recompile everything

each time I changed it.  Why make this inline?  It's not performance-critical
at that level, and it's more code in the headers to slow down compiles.

stepmania/src/RageUtil.cpp

@@ -308,6 +308,14 @@ void splitrelpath( const CString &Path, CString& Dir, CString& FName, CString& E
 	}
 }
 
+CString GetExtension( CString sPath )
+{
+	CString Dir, FName, Ext;
+	splitrelpath( sPath, Dir, FName, Ext );
+	Ext.MakeLower();
+	return Ext;
+}
+
 /* mkdir -p.  Doesn't fail if Path already exists and is a directory. */
 bool CreateDirectories( CString Path )
 {

stepmania/src/RageUtil.h

@@ -174,13 +174,7 @@ void splitrelpath(
 	CString& Ext 
 );
 
-inline CString GetExtension( CString sPath )
+CString GetExtension( CString sPath );
-{
-	CString Dir, FName, Ext;
-	splitrelpath( sPath, Dir, FName, Ext );
-	Ext.MakeLower();
-	return Ext;
-};
 
 typedef int longchar;
 extern const wchar_t INVALID_CHAR;