tweak(debug): Add error handling for rename failures in Debug.cpp

Previously, the code assumed that the `rename` function would always succeed. This commit introduces error handling. If `rename` fails, the code now logs an error message and removes the file that could not be renamed.

This removes 3 compiler warnings.

Author: Skyaero42

Core/GameEngine/Source/Common/System/Debug.cpp

@@ -404,7 +404,12 @@ void DebugInit(int flags)
 		strlcat(theLogFileName, ".txt", ARRAY_SIZE(theLogFileNamePrev));
 
 		remove(theLogFileNamePrev);
-		rename(theLogFileName, theLogFileNamePrev);
+		if (rename(theLogFileName, theLogFileNamePrev) != 0)
+		{
+			DEBUG_ASSERTLOG(false, ("Could not rename log file and will remove instead"));
+			remove(theLogFileName);
+		}
+
 		theLogFile = fopen(theLogFileName, "w");
 		if (theLogFile != NULL)
 		{
@@ -738,7 +743,11 @@ void ReleaseCrash(const char *reason)
 	strlcat(curbuf, RELEASECRASH_FILE_NAME, ARRAY_SIZE(curbuf));
 
  	remove(prevbuf);
-	rename(curbuf, prevbuf);
+	if (rename(curbuf, prevbuf) != 0)
+	{
+		DEBUG_ASSERTLOG(false, ("Could not rename buffer file and will remove instead"));
+		remove(curbuf);
+	}
 
 	theReleaseCrashLogFile = fopen(curbuf, "w");
 	if (theReleaseCrashLogFile)
@@ -827,7 +836,11 @@ void ReleaseCrashLocalized(const AsciiString& p, const AsciiString& m)
 	strlcat(curbuf, RELEASECRASH_FILE_NAME, ARRAY_SIZE(curbuf));
 
  	remove(prevbuf);
-	rename(curbuf, prevbuf);
+	if (rename(curbuf, prevbuf) != 0)
+	{
+		DEBUG_ASSERTLOG(false, ("Could not rename buffer file and will remove instead"));
+		remove(curbuf);
+	}
 
 	theReleaseCrashLogFile = fopen(curbuf, "w");
 	if (theReleaseCrashLogFile)