gdi32: Antialias fake bold or italic fonts regardless of GASP table.

If the client requests a bold typeface of a font which doesn't actually provide a bold typeface, Wine fakes it using FT_Outline_Embolden(). Likewise, it fakes italic by apply a shear transform. These techniques work passably when antialiasing is in effect. However, when antialiasing is disabled at small sizes by the font's GASP table, the results are horrible. The glyphs have many spurs or appear furry. On the theory that fonts disable antialiasing via the GASP table at small sizes because hinting should produce pixel-perfect results and that that rationale is voided by the application of procedural bolding or italicizing, we ignore the GASP table and always antialias for fake bold or italic fonts. There is a new registry setting to disable this and revert to the previous behavior: [HKEY_CURRENT_USER\Software\Wine\Fonts] "AntialiasFakeBoldOrItalic"="n" Signed-off-by: Ken Thomases <ken@codeweavers.com>

gdi32: Antialias fake bold or italic fonts regardless of GASP table.
a05cdaa1 · Ken Thomases · Alexandre Julliard · 1a45f75d · a05cdaa1
Commit a05cdaa1 authored Sep 29, 2015 by Ken Thomases Committed by Alexandre Julliard Oct 01, 2015
Show whitespace changes
Inline Side-by-side

Showing with 20 additions and 1 deletion

freetype.c dlls/gdi32/freetype.c +20 -1

No files found.
--- a/dlls/gdi32/freetype.c
+++ b/dlls/gdi32/freetype.c
@@ -503,6 +503,7 @@ static struct list mappings_list = LIST_INIT( mappings_list );

 static UINT default_aa_flags;
 static HKEY hkey_font_cache;
+static BOOL antialias_fakes = TRUE;

 static CRITICAL_SECTION freetype_cs;
 static CRITICAL_SECTION_DEBUG critsect_debug =
@@ -4143,6 +4144,7 @@ static void reorder_font_list(void)
 */
 BOOL WineEngInit(void)
 {
+    HKEY hkey;
    DWORD disposition;
    HANDLE font_mutex;

@@ -4155,6 +4157,23 @@ BOOL WineEngInit(void)
    init_fontconfig();
 #endif

+    if (!RegOpenKeyExW(HKEY_CURRENT_USER, wine_fonts_key, 0, KEY_READ, &hkey))
+    {
+        static const WCHAR antialias_fake_bold_or_italic[] = { 'A','n','t','i','a','l','i','a','s','F','a','k','e',
+                                                               'B','o','l','d','O','r','I','t','a','l','i','c',0 };
+        static const WCHAR true_options[] = { 'y','Y','t','T','1',0 };
+        DWORD type, size;
+        WCHAR buffer[20];
+
+        size = sizeof(buffer);
+        if (!RegQueryValueExW(hkey, antialias_fake_bold_or_italic, NULL, &type, (BYTE*)buffer, &size) &&
+            type == REG_SZ && size >= 1)
+        {
+            antialias_fakes = (strchrW(true_options, buffer[0]) != NULL);
+        }
+        RegCloseKey(hkey);
+    }
+
    if((font_mutex = CreateMutexW(NULL, FALSE, font_mutex_nameW)) == NULL)
    {
        ERR("Failed to create font mutex\n");
@@ -5534,7 +5553,7 @@ done:
            case GGO_GRAY4_BITMAP:
            case GGO_GRAY8_BITMAP:
            case WINE_GGO_GRAY16_BITMAP:
-                if (is_hinting_enabled())
+                if ((!antialias_fakes || (!ret->fake_bold && !ret->fake_italic)) && is_hinting_enabled())
                {
                    WORD gasp_flags;
                    if (get_gasp_flags( ret, &gasp_flags ) && !(gasp_flags & GASP_DOGRAY))