[FlxBitmapFont] allow extra whitespace when parsing bmfont rows (#3556)

* allow extra whitespace when parsing bmfont rows

* formatting and modifications to tests

* pull from correct capture group

* make test spaces consistent

* Move to separate test

---------

Co-authored-by: George Kurelic <Gkurelic@gmail.com>

Author: MondayHopscotch

flixel/graphics/frames/bmfont/BMFontUtil.hx

@@ -8,15 +8,15 @@ using StringTools;
 @:noCompletion
 class BMFontUtil
 {
-	static var attFinder = ~/(\w+?)=("?)(.*?)\2(?=\s|$)/;
+	static var attFinder = ~/(\w+?)=(\s*)("?)(.*?)\3(?=\s|$)/;
 
 	public static function forEachAttribute(text:UnicodeString, callback:(key:String, value:UnicodeString)->Void)
 	{
 		var index = 0;
 		while (attFinder.match(text.substr(index)))
 		{
 			final key = attFinder.matched(1);
-			final value = attFinder.matched(3);
+			final value = attFinder.matched(4);
 			callback(key, value);
 
 			final nextIndex = text.length - attFinder.matchedRight().length;

tests/unit/src/flixel/graphics/frames/bmfont/BMFontTest.hx

@@ -24,6 +24,25 @@ class BMFontTest extends FlxTest
 		assertFont(font);
 	}
 
+	@Test
+	function testTextFormatWithSpaces()
+	{
+		var text =
+		'info face="Arial Black" size=32 bold=0 italic=0 charset="" unicode=1 stretchH=100 smooth=1 aa=1 padding=1,2,3,4 spacing=2,1 outline=0'
+		+ '\ncommon lineHeight=32 base=25 scaleW=256 scaleH=256 pages=1 packed=0 alphaChnl=1 redChnl=0 greenChnl=0 blueChnl=0'
+		+ '\npage id=0 file="arial_black_0.png"'
+		+ '\nchars count=3'
+		+ '\nchar id=64   x=0     y=0     width=25    height=24    xoffset=-5    yoffset=7     xadvance=17    page=0   chnl=15'
+		+ '\nchar id= 65  x= 27   y= 0    width= 26   height= 21   xoffset= -5   yoffset= 7    xadvance= 18   page= 0  chnl= 15'
+		+ '\nchar id=  84 x=  55  y=  0   width=  23  height=  21  xoffset=  -4  yoffset=  7   xadvance=  16  page=  0 chnl=  15'
+		+ '\nkernings count=2'
+		+ '\nkerning first=84  second=65  amount=-2  '
+		+ '\nkerning first= 65 second= 84 amount= -2 ';
+		
+		var font = BMFont.parse(cast text);
+		assertFont(font);
+	}
+	
 	@Test
 	function testXMLFormat()
 	{
@@ -149,4 +168,4 @@ class BMFontTest extends FlxTest
 		Assert.areEqual(expected.second, actual.second);
 		Assert.areEqual(expected.amount, actual.amount);
 	}
-}
+}