Cache-friendler blending loops in FontFreeType::getGlyphBitmapByIndex.

j-jorge · j-jorge · commit e19730b62765 · 2025-12-04T19:01:58.000+01:00
diff --git a/core/2d/FontFreeType.cpp b/core/2d/FontFreeType.cpp
@@ -545,9 +545,9 @@ unsigned char* FontFreeType::getGlyphBitmapByIndex(unsigned int glyphIndex,
 
                 auto px = outlineMinX - blendImageMinX;
                 auto py = blendImageMaxY - outlineMaxY;
-                for (int x = 0; x < outlineWidth; ++x)
+                for (int y = 0; y < outlineHeight; ++y)
                 {
-                    for (int y = 0; y < outlineHeight; ++y)
+                    for (int x = 0; x < outlineWidth; ++x)
                     {
                         index                 = px + x + ((py + y) * blendWidth);
                         index2                = x + (y * outlineWidth);
@@ -557,9 +557,9 @@ unsigned char* FontFreeType::getGlyphBitmapByIndex(unsigned int glyphIndex,
 
                 px = glyphMinX - blendImageMinX;
                 py = blendImageMaxY - glyphMaxY;
-                for (int x = 0; x < outWidth; ++x)
+                for (int y = 0; y < outHeight; ++y)
                 {
-                    for (int y = 0; y < outHeight; ++y)
+                    for (int x = 0; x < outWidth; ++x)
                     {
                         index                     = px + x + ((y + py) * blendWidth);
                         index2                    = x + (y * outWidth);

Original file line number	Diff line number	Diff line change
`@@ -545,9 +545,9 @@ unsigned char* FontFreeType::getGlyphBitmapByIndex(unsigned int glyphIndex,`
`545`	`545`
`546`	`546`	`auto px = outlineMinX - blendImageMinX;`
`547`	`547`	`auto py = blendImageMaxY - outlineMaxY;`
`548`		`- for (int x = 0; x < outlineWidth; ++x)`
	`548`	`+ for (int y = 0; y < outlineHeight; ++y)`
`549`	`549`	`{`
`550`		`- for (int y = 0; y < outlineHeight; ++y)`
	`550`	`+ for (int x = 0; x < outlineWidth; ++x)`
`551`	`551`	`{`
`552`	`552`	`index = px + x + ((py + y) * blendWidth);`
`553`	`553`	`index2 = x + (y * outlineWidth);`
`@@ -557,9 +557,9 @@ unsigned char* FontFreeType::getGlyphBitmapByIndex(unsigned int glyphIndex,`
`557`	`557`
`558`	`558`	`px = glyphMinX - blendImageMinX;`
`559`	`559`	`py = blendImageMaxY - glyphMaxY;`
`560`		`- for (int x = 0; x < outWidth; ++x)`
	`560`	`+ for (int y = 0; y < outHeight; ++y)`
`561`	`561`	`{`
`562`		`- for (int y = 0; y < outHeight; ++y)`
	`562`	`+ for (int x = 0; x < outWidth; ++x)`
`563`	`563`	`{`
`564`	`564`	`index = px + x + ((y + py) * blendWidth);`
`565`	`565`	`index2 = x + (y * outWidth);`