Rémi Bernon : gdi32: Lazily create and cache fontconfig patterns.
Alexandre Julliard
julliard at winehq.org
Fri Nov 20 14:54:32 CST 2020
Module: wine
Branch: master
Commit: f7e646ccb7b26b949986e53bca38674dd664009f
URL: https://source.winehq.org/git/wine.git/?a=commit;h=f7e646ccb7b26b949986e53bca38674dd664009f
Author: Rémi Bernon <rbernon at codeweavers.com>
Date: Fri Nov 20 10:40:42 2020 +0000
gdi32: Lazily create and cache fontconfig patterns.
Signed-off-by: Rémi Bernon <rbernon at codeweavers.com>
Signed-off-by: Huw Davies <huw at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>
---
dlls/gdi32/freetype.c | 24 +++++++++++-------------
1 file changed, 11 insertions(+), 13 deletions(-)
diff --git a/dlls/gdi32/freetype.c b/dlls/gdi32/freetype.c
index 7d8bc5f5b05..993110e6fe0 100644
--- a/dlls/gdi32/freetype.c
+++ b/dlls/gdi32/freetype.c
@@ -1343,21 +1343,22 @@ static UINT parse_aa_pattern( FcPattern *pattern )
return aa_flags;
}
-static FcPattern *create_family_pattern( const char *name )
+static FcPattern *create_family_pattern( const char *name, FcPattern **cached )
{
- FcPattern *ret, *pattern = pFcPatternCreate();
+ FcPattern *ret = NULL, *tmp, *pattern = pFcPatternCreate();
FcResult result;
-
+ if (*cached) return *cached;
pFcPatternAddString( pattern, FC_FAMILY, (const FcChar8 *)name );
pFcPatternAddString( pattern, FC_NAMELANG, (const FcChar8 *)"en-us" );
pFcPatternAddString( pattern, FC_PRGNAME, (const FcChar8 *)"wine" );
pFcConfigSubstitute( NULL, pattern, FcMatchPattern );
pFcDefaultSubstitute( pattern );
- ret = pFcFontMatch( NULL, pattern, &result );
+ tmp = pFcFontMatch( NULL, pattern, &result );
pFcPatternDestroy( pattern );
- if (ret && result == FcResultMatch) return ret;
- pFcPatternDestroy( ret );
- return NULL;
+ if (result != FcResultMatch) pFcPatternDestroy( tmp );
+ else if ((ret = InterlockedCompareExchangePointer( (void **)cached, tmp, NULL ))) pFcPatternDestroy( tmp );
+ else ret = tmp;
+ return ret;
}
static void fontconfig_add_font( FcPattern *pattern, DWORD flags )
@@ -1458,9 +1459,6 @@ static void init_fontconfig(void)
default_aa_flags = parse_aa_pattern( pattern );
pFcPatternDestroy( pattern );
}
- pattern_serif = create_family_pattern( "serif" );
- pattern_fixed = create_family_pattern( "monospace" );
- pattern_sans = create_family_pattern( "sans" );
TRACE( "enabled, default flags = %x\n", default_aa_flags );
fontconfig_enabled = TRUE;
@@ -2213,9 +2211,9 @@ static BOOL CDECL fontconfig_enum_family_fallbacks( DWORD pitch_and_family, int
char *str;
DWORD len;
- if ((pitch_and_family & FIXED_PITCH) || (pitch_and_family & 0xf0) == FF_MODERN) pat = pattern_fixed;
- else if ((pitch_and_family & 0xf0) == FF_ROMAN) pat = pattern_serif;
- else pat = pattern_sans;
+ if ((pitch_and_family & FIXED_PITCH) || (pitch_and_family & 0xf0) == FF_MODERN) pat = create_family_pattern( "monospace", &pattern_fixed );
+ else if ((pitch_and_family & 0xf0) == FF_ROMAN) pat = create_family_pattern( "serif", &pattern_serif );
+ else pat = create_family_pattern( "sans", &pattern_sans );
if (!pat) return FALSE;
if (pFcPatternGetString( pat, FC_FAMILY, index, (FcChar8 **)&str ) != FcResultMatch) return FALSE;
More information about the wine-cvs
mailing list