This is weird. The original logic doesn't fit its comment and will
adjust if either ignore X mode or face SPH compat mode is disabled, not
when both are disabled as suggested.

Maybe this should be !(SUBPIXEL_HINTING_INFINALITY &&
(exc->ignore_x_mode || exc->face->sph_compatibility_mode)).

The original logic at https://github.com/madig/freetype2/commit/ed1d8983f3826124e814145b7556fc8f03ae88df#diff-0508f7c392840b939402aade4cbe3563L5776 says

-      if ( !SUBPIXEL_HINTING                      ||
-           ( !exc->ignore_x_mode                ||
-             !exc->face->sph_compatibility_mode ) )

So the comment and your suggestion match :)

_______________________________________________
Freetype-devel mailing list
[email protected]
https://lists.nongnu.org/mailman/listinfo/freetype-devel

Reply via email to