From ce2b515d2a0567667a902ed339aa45fb689b40cc Mon Sep 17 00:00:00 2001
From: mankrip <96807997+mankrip@users.noreply.github.com>
Date: Wed, 21 Feb 2024 08:55:32 -0300
Subject: [PATCH] Fixed: backslash characters in savegames skips string chars

ED_NewString had a bug in its newline parser, which always skipped the first character after a backslash even when it's not the 'n' char. This resulted in string corruption across subsequent saves & loads in the same gameplay session.
---
 quakespasm/Quake/pr_edict.c | 18 ++++++++----------
 1 file changed, 8 insertions(+), 10 deletions(-)

diff --git a/quakespasm/Quake/pr_edict.c b/quakespasm/Quake/pr_edict.c
index c52f7c0e4..45c5ab563 100644
--- a/quakespasm/Quake/pr_edict.c
+++ b/quakespasm/Quake/pr_edict.c
@@ -742,23 +742,21 @@ static string_t ED_NewString (const char *string)
 	int		i, l;
 	string_t	num;
 
-	l = strlen(string) + 1;
-	num = PR_AllocString (l, &new_p);
+	l = strlen (string) + 1;
+	new = Hunk_Alloc (l);
+	new_p = new;
 
-	for (i = 0; i < l; i++)
+	for (i = 0 ; i < l ; ++i)
 	{
-		if (string[i] == '\\' && i < l-1)
+		if (i < l - 1 && string[i] == '\\' && string[i + 1] == 'n')
 		{
-			i++;
-			if (string[i] == 'n')
-				*new_p++ = '\n';
-			else
-				*new_p++ = '\\';
+			// converts both characters into a single newline character
+			++i;
+			*new_p++ = '\n';
 		}
 		else
 			*new_p++ = string[i];
 	}
-
 	return num;
 }