feat: handle common comment case

nodecraft · Nov 27, 2024 · c1a1ce2 · c1a1ce2
1 parent e8b91f5
commit c1a1ce2
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/ini.js b/ini.js
@@ -227,7 +227,11 @@ const unsafe = (val) => {
 			}
 			isEscaping = false;
 		}else if(commentChars.includes(char)){
-			break;
+			// Check if there's spaces around this comment character
+			// If there is, then we're done parsing at the character before this one
+			if(val.charAt(i - 1) === ' ' && val.charAt(i + 1) === ' '){
+				break;
+			}
 		}else if(char === '\\'){
 			isEscaping = true;
 		}else{

diff --git a/test/foo.js b/test/foo.js
@@ -400,7 +400,7 @@ test('ignores invalid line (=)', function(t){
 
 test("unsafe escape values", function(t){
 	t.equal(ini.unsafe(''), '');
-	t.equal(ini.unsafe('x;y'), 'x');
+	t.equal(ini.unsafe('x;y'), 'xy');
 	t.equal(ini.unsafe('x  # y'), 'x');
 	t.equal(ini.unsafe('x "\\'), 'x "\\');
 	t.end();