this shouldn't have even compiled ...

stepmania/src/RageException.cpp

@@ -32,7 +32,7 @@ const char* RageException::what() const throw ()
 	return m_sError;
 }
 
-RageException::Throw(const char *fmt, ...)
+void RageException::Throw(const char *fmt, ...)
 {
     va_list	va;
     va_start(va, fmt);
@@ -50,10 +50,10 @@ RageException::Throw(const char *fmt, ...)
 	DebugBreak();
 #endif
 
-	throw RageException("%s", error);
+	throw RageException("%s", error.GetString());
 }
 
-RageException::ThrowNonfatal(const char *fmt, ...)
+void RageException::ThrowNonfatal(const char *fmt, ...)
 {
     va_list	va;
     va_start(va, fmt);

stepmania/src/RageException.h

@@ -26,8 +26,8 @@ public:
 	/* The only difference between these is that Throw triggers debug behavior
 	 * and Nonfatal doesn't.  Nonfatal is used when the exception happens
 	 * normally and will be caught, such as when a driver fails to initialize. */
-	static NORETURN Throw(const char *fmt, ...);
-	static NORETURN ThrowNonfatal(const char *fmt, ...);
+	static void NORETURN Throw(const char *fmt, ...);
+	static void NORETURN ThrowNonfatal(const char *fmt, ...);
 
 protected:
 	CString m_sError;