November 2020 - Wine-commits

Brendan Shanks : include: Force stack alignment on x86_64 with Clang.
by Alexandre Julliard 05 Nov '20

05 Nov '20

Module: wine Branch: master Commit: 91741446c7c8d2e6eb4c87774f20ef8eb9295254 URL: https://source.winehq.org/git/wine.git/?a=commit;h=91741446c7c8d2e6eb4c8777… Author: Brendan Shanks <bshanks(a)codeweavers.com> Date: Wed Nov 4 17:29:28 2020 -0800 include: Force stack alignment on x86_64 with Clang. Signed-off-by: Brendan Shanks <bshanks(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- include/msvcrt/corecrt.h | 8 ++++++-- include/windef.h | 8 ++++++-- 2 files changed, 12 insertions(+), 4 deletions(-) diff --git a/include/msvcrt/corecrt.h b/include/msvcrt/corecrt.h index ad46c56591c..587dcce2a54 100644 --- a/include/msvcrt/corecrt.h +++ b/include/msvcrt/corecrt.h @@ -57,6 +57,10 @@ #endif #endif +#ifndef __has_attribute +#define __has_attribute(x) 0 +#endif + #if !defined(_MSC_VER) && !defined(__stdcall) # ifdef __i386__ # ifdef __GNUC__ @@ -69,7 +73,7 @@ # error You need to define __stdcall for your compiler # endif # elif defined(__x86_64__) && defined (__GNUC__) -# if (__GNUC__ > 5) || ((__GNUC__ == 5) && (__GNUC_MINOR__ >= 3)) +# if __has_attribute(__force_align_arg_pointer__) # define __stdcall __attribute__((ms_abi)) __attribute__((__force_align_arg_pointer__)) # else # define __stdcall __attribute__((ms_abi)) @@ -91,7 +95,7 @@ # define __cdecl __attribute__((__cdecl__)) # endif # elif defined(__x86_64__) && defined (__GNUC__) -# if (__GNUC__ > 5) || ((__GNUC__ == 5) && (__GNUC_MINOR__ >= 3)) +# if __has_attribute(__force_align_arg_pointer__) # define __cdecl __attribute__((ms_abi)) __attribute__((__force_align_arg_pointer__)) # else # define __cdecl __attribute__((ms_abi)) diff --git a/include/windef.h b/include/windef.h index 521c3ab4512..db5c60641d4 100644 --- a/include/windef.h +++ b/include/windef.h @@ -31,6 +31,10 @@ # endif /* STRICT */ #endif /* NO_STRICT */ +#ifndef __has_attribute +#define __has_attribute(x) 0 +#endif + #ifdef __cplusplus extern "C" { #endif @@ -62,7 +66,7 @@ extern "C" { # error You need to define __stdcall for your compiler # endif # elif defined(__x86_64__) && defined (__GNUC__) -# if (__GNUC__ > 5) || ((__GNUC__ == 5) && (__GNUC_MINOR__ >= 3)) +# if __has_attribute(__force_align_arg_pointer__) # define __stdcall __attribute__((ms_abi)) __attribute__((__force_align_arg_pointer__)) # else # define __stdcall __attribute__((ms_abi)) @@ -84,7 +88,7 @@ extern "C" { # define __cdecl __attribute__((__cdecl__)) # endif # elif defined(__x86_64__) && defined (__GNUC__) -# if (__GNUC__ > 5) || ((__GNUC__ == 5) && (__GNUC_MINOR__ >= 3)) +# if __has_attribute(__force_align_arg_pointer__) # define __cdecl __attribute__((ms_abi)) __attribute__((__force_align_arg_pointer__)) # else # define __cdecl __attribute__((ms_abi))

1 0

Alexandre Julliard : gdi32: Don't abort enumeration if a font fails to load.
by Alexandre Julliard 05 Nov '20

05 Nov '20

Module: wine Branch: master Commit: fe6bb5f72394b4d56c6f4956363f347b050ce4ee URL: https://source.winehq.org/git/wine.git/?a=commit;h=fe6bb5f72394b4d56c6f4956… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Nov 5 09:59:22 2020 +0100 gdi32: Don't abort enumeration if a font fails to load. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdi32/font.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/gdi32/font.c b/dlls/gdi32/font.c index 89197361fe2..300b8fc1106 100644 --- a/dlls/gdi32/font.c +++ b/dlls/gdi32/font.c @@ -2772,11 +2772,11 @@ static BOOL enum_face_charsets( const struct gdi_font_family *family, struct gdi { struct gdi_font_enum_data *data; - if (!(data = HeapAlloc( GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(*data) ))) return FALSE; - if (!get_face_enum_data( face, &data->elf, &data->ntm )) + if (!(data = HeapAlloc( GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(*data) )) || + !get_face_enum_data( face, &data->elf, &data->ntm )) { HeapFree( GetProcessHeap(), 0, data ); - return FALSE; + return TRUE; } face->cached_enum_data = data; }

1 0

Alexandre Julliard : gdi32: Store scale as an integer.
by Alexandre Julliard 05 Nov '20

05 Nov '20

Module: wine Branch: master Commit: 8768f605273c88d1041afcdd4375a87b9adb8d32 URL: https://source.winehq.org/git/wine.git/?a=commit;h=8768f605273c88d1041afcdd… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Nov 5 09:59:32 2020 +0100 gdi32: Store scale as an integer. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdi32/font.c | 4 ++-- dlls/gdi32/freetype.c | 4 ++-- dlls/gdi32/gdi_private.h | 2 +- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/dlls/gdi32/font.c b/dlls/gdi32/font.c index 300b8fc1106..6a76e28d06d 100644 --- a/dlls/gdi32/font.c +++ b/dlls/gdi32/font.c @@ -1698,7 +1698,7 @@ static struct gdi_font *alloc_gdi_font( const WCHAR *file, void *data_ptr, SIZE_ font->refcount = 1; font->matrix.eM11 = font->matrix.eM22 = 1.0; - font->scale_y = 1.0; + font->scale_y = 1; font->kern_count = -1; list_init( &font->child_fonts ); @@ -3686,8 +3686,8 @@ static struct gdi_font *select_font( LOGFONTW *lf, FMAT2 dcmat, BOOL can_use_bit /* The jump between unscaled and doubled is delayed by 1 */ else if (scale == 2 && scaled_height - height > (face->size.height / 4 - 1)) scale--; font->scale_y = scale; + TRACE("font scale y: %d\n", font->scale_y); } - TRACE("font scale y: %f\n", font->scale_y); if (!font_funcs->load_font( font )) { diff --git a/dlls/gdi32/freetype.c b/dlls/gdi32/freetype.c index c23ed714b4c..ff39ffd9541 100644 --- a/dlls/gdi32/freetype.c +++ b/dlls/gdi32/freetype.c @@ -2308,13 +2308,13 @@ static BOOL get_transform_matrices( struct gdi_font *font, BOOL vertical, const width_ratio = font->scale_y; /* Scaling transform */ - if (width_ratio != 1.0 || font->scale_y != 1.0) + if (width_ratio != 1.0 || font->scale_y != 1) { FT_Matrix scale_mat; scale_mat.xx = FT_FixedFromFloat( width_ratio ); scale_mat.xy = 0; scale_mat.yx = 0; - scale_mat.yy = FT_FixedFromFloat( font->scale_y ); + scale_mat.yy = font->scale_y << 16; pFT_Matrix_Multiply( &scale_mat, &matrices[matrix_unrotated] ); needs_transform = TRUE; diff --git a/dlls/gdi32/gdi_private.h b/dlls/gdi32/gdi_private.h index 12f469af979..c5703689d46 100644 --- a/dlls/gdi32/gdi_private.h +++ b/dlls/gdi32/gdi_private.h @@ -335,7 +335,7 @@ struct gdi_font LOGFONTW lf; FMAT2 matrix; UINT face_index; - double scale_y; + INT scale_y; INT aveWidth; INT ppem; SHORT yMax;

1 0

Alexandre Julliard : gdi32: Use FT_MulDiv() instead of the kernel32 function.
by Alexandre Julliard 05 Nov '20

05 Nov '20

Module: wine Branch: master Commit: fd9e1612d9a0c40dfefead092bb11d6cf79cf5e1 URL: https://source.winehq.org/git/wine.git/?a=commit;h=fd9e1612d9a0c40dfefead09… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Nov 5 10:01:32 2020 +0100 gdi32: Use FT_MulDiv() instead of the kernel32 function. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdi32/freetype.c | 19 +++++++++---------- 1 file changed, 9 insertions(+), 10 deletions(-) diff --git a/dlls/gdi32/freetype.c b/dlls/gdi32/freetype.c index ff39ffd9541..c73305ad494 100644 --- a/dlls/gdi32/freetype.c +++ b/dlls/gdi32/freetype.c @@ -146,6 +146,7 @@ MAKE_FUNCPTR(FT_Library_Version); MAKE_FUNCPTR(FT_Load_Glyph); MAKE_FUNCPTR(FT_Load_Sfnt_Table); MAKE_FUNCPTR(FT_Matrix_Multiply); +MAKE_FUNCPTR(FT_MulDiv); #ifdef FT_MULFIX_INLINED #define pFT_MulFix FT_MULFIX_INLINED #else @@ -1509,6 +1510,7 @@ static BOOL init_freetype(void) LOAD_FUNCPTR(FT_Load_Glyph) LOAD_FUNCPTR(FT_Load_Sfnt_Table) LOAD_FUNCPTR(FT_Matrix_Multiply) + LOAD_FUNCPTR(FT_MulDiv) #ifndef FT_MULFIX_INLINED LOAD_FUNCPTR(FT_MulFix) #endif @@ -1635,11 +1637,9 @@ static LONG calc_ppem_for_height(FT_Face ft_face, LONG height) if(height > 0) { USHORT windescent = get_fixed_windescent(pOS2->usWinDescent); if(pOS2->usWinAscent + windescent == 0) - ppem = MulDiv(ft_face->units_per_EM, height, - pHori->Ascender - pHori->Descender); + ppem = pFT_MulDiv(ft_face->units_per_EM, height, pHori->Ascender - pHori->Descender); else - ppem = MulDiv(ft_face->units_per_EM, height, - pOS2->usWinAscent + windescent); + ppem = pFT_MulDiv(ft_face->units_per_EM, height, pOS2->usWinAscent + windescent); if(ppem > MAX_PPEM) { WARN("Ignoring too large height %d, ppem %d\n", height, ppem); ppem = 1; @@ -2340,7 +2340,7 @@ static BOOL get_transform_matrices( struct gdi_font *font, BOOL vertical, const FT_Matrix rotation_mat; FT_Vector angle; - pFT_Vector_Unit( &angle, MulDiv( 1 << 16, font->lf.lfOrientation, 10 ) ); + pFT_Vector_Unit( &angle, pFT_MulDiv( 1 << 16, font->lf.lfOrientation, 10 ) ); rotation_mat.xx = angle.x; rotation_mat.xy = -angle.y; rotation_mat.yx = angle.y; @@ -2401,7 +2401,7 @@ static BOOL get_bold_glyph_outline(FT_GlyphSlot glyph, LONG ppem, FT_Glyph_Metri if(!pFT_Outline_Embolden) return FALSE; - strength = MulDiv(ppem, 1 << 6, 24); + strength = pFT_MulDiv(ppem, 1 << 6, 24); err = pFT_Outline_Embolden(&glyph->outline, strength); if(err) { TRACE("FT_Ouline_Embolden returns %d\n", err); @@ -2452,8 +2452,7 @@ static FT_Vector get_advance_metric(struct gdi_font *incoming_font, struct gdi_f if (freetype_set_outline_text_metrics(incoming_font) && !(incoming_font->otm.otmTextMetrics.tmPitchAndFamily & TMPF_FIXED_PITCH)) { UINT avg_advance; - em_scale = MulDiv(incoming_font->ppem, 1 << 16, - get_ft_face(incoming_font)->units_per_EM); + em_scale = pFT_MulDiv(incoming_font->ppem, 1 << 16, get_ft_face(incoming_font)->units_per_EM); avg_advance = pFT_MulFix(incoming_font->ntmAvgWidth, em_scale); fixed_pitch_full = (avg_advance > 0 && (base_advance + 63) >> 6 == @@ -3395,7 +3394,7 @@ static BOOL CDECL freetype_set_outline_text_metrics( struct gdi_font *font ) strlenW( (WCHAR *)font->otm.otmpFaceName ) + 1 + strlenW( (WCHAR *)font->otm.otmpFullName ) + 1) * sizeof(WCHAR); - em_scale = (FT_Fixed)MulDiv(font->ppem, 1 << 16, ft_face->units_per_EM); + em_scale = (FT_Fixed)pFT_MulDiv(font->ppem, 1 << 16, ft_face->units_per_EM); pOS2 = pFT_Get_Sfnt_Table(ft_face, ft_sfnt_os2); if(!pOS2) { @@ -3651,7 +3650,7 @@ static BOOL CDECL freetype_get_char_width_info( struct gdi_font *font, struct ch if ((pHori = pFT_Get_Sfnt_Table(ft_face, ft_sfnt_hhea))) { - FT_Fixed em_scale = MulDiv(font->ppem, 1 << 16, ft_face->units_per_EM); + FT_Fixed em_scale = pFT_MulDiv(font->ppem, 1 << 16, ft_face->units_per_EM); info->lsb = (SHORT)pFT_MulFix(pHori->min_Left_Side_Bearing, em_scale); info->rsb = (SHORT)pFT_MulFix(pHori->min_Right_Side_Bearing, em_scale); return TRUE;

1 0

Alexandre Julliard : gdi32: Use ntdll functions for codepage conversions.
by Alexandre Julliard 05 Nov '20

05 Nov '20

Module: wine Branch: master Commit: 8db165000d7188468bf55f9f73deca13a82a5749 URL: https://source.winehq.org/git/wine.git/?a=commit;h=8db165000d7188468bf55f9f… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Nov 5 10:02:50 2020 +0100 gdi32: Use ntdll functions for codepage conversions. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdi32/freetype.c | 62 ++++++++++++++++++++++++++++++--------------------- 1 file changed, 36 insertions(+), 26 deletions(-) diff --git a/dlls/gdi32/freetype.c b/dlls/gdi32/freetype.c index c73305ad494..7324622a4e3 100644 --- a/dlls/gdi32/freetype.c +++ b/dlls/gdi32/freetype.c @@ -561,14 +561,11 @@ static LPWSTR strdupW(LPCWSTR p) return ret; } -static WCHAR *towstr(UINT cp, const char *str) +static WCHAR *towstr(const char *str) { - int len; - WCHAR *wstr; - - len = MultiByteToWideChar(cp, 0, str, -1, NULL, 0); - wstr = HeapAlloc(GetProcessHeap(), 0, len * sizeof(WCHAR)); - MultiByteToWideChar(cp, 0, str, -1, wstr, len); + DWORD len = strlen(str) + 1; + WCHAR *wstr = RtlAllocateHeap( GetProcessHeap(), 0, len * sizeof(WCHAR) ); + RtlMultiByteToUnicodeN( wstr, len * sizeof(WCHAR), &len, str, len ); return wstr; } @@ -697,10 +694,21 @@ static const LANGID mac_langid_table[] = MAKELANGID(LANG_AZERI,SUBLANG_AZERI_LATIN), /* TT_MAC_LANGID_AZERBAIJANI_ROMAN_SCRIPT */ }; -static inline WORD get_mac_code_page( const FT_SfntName *name ) +static CPTABLEINFO *get_mac_code_page( const FT_SfntName *name ) { - if (name->encoding_id == TT_MAC_ID_SIMPLIFIED_CHINESE) return 10008; /* special case */ - return 10000 + name->encoding_id; + static CPTABLEINFO tables[100]; + int id = name->encoding_id; + USHORT *ptr; + SIZE_T size; + + if (name->encoding_id == TT_MAC_ID_SIMPLIFIED_CHINESE) id = 8; /* special case */ + if (id >= ARRAY_SIZE(tables)) return NULL; + if (!tables[id].CodePage) + { + if (NtGetNlsSectionPtr( 11, 10000 + id, NULL, (void **)&ptr, &size )) return NULL; + RtlInitCodePageTable( ptr, &tables[id] ); + } + return &tables[id]; } static int match_name_table_language( const FT_SfntName *name, LANGID lang ) @@ -723,7 +731,7 @@ static int match_name_table_language( const FT_SfntName *name, LANGID lang ) } break; case TT_PLATFORM_MACINTOSH: - if (!IsValidCodePage( get_mac_code_page( name ))) return 0; + if (!get_mac_code_page( name )) return 0; if (name->language_id >= ARRAY_SIZE( mac_langid_table )) return 0; name_lang = mac_langid_table[name->language_id]; break; @@ -754,8 +762,8 @@ static int match_name_table_language( const FT_SfntName *name, LANGID lang ) static WCHAR *copy_name_table_string( const FT_SfntName *name ) { WCHAR *ret; - WORD codepage; - int i; + CPTABLEINFO *cp; + DWORD i; switch (name->platform_id) { @@ -767,11 +775,11 @@ static WCHAR *copy_name_table_string( const FT_SfntName *name ) ret[i] = 0; return ret; case TT_PLATFORM_MACINTOSH: - codepage = get_mac_code_page( name ); - i = MultiByteToWideChar( codepage, 0, (char *)name->string, name->string_len, NULL, 0 ); - ret = HeapAlloc( GetProcessHeap(), 0, (i + 1) * sizeof(WCHAR) ); - MultiByteToWideChar( codepage, 0, (char *)name->string, name->string_len, ret, i ); - ret[i] = 0; + if (!(cp = get_mac_code_page( name ))) return NULL; + ret = RtlAllocateHeap( GetProcessHeap(), 0, (name->string_len + 1) * sizeof(WCHAR) ); + RtlCustomCPToUnicodeN( cp, ret, name->string_len * sizeof(WCHAR), &i, + (char *)name->string, name->string_len ); + ret[i / sizeof(WCHAR)] = 0; return ret; } return NULL; @@ -816,7 +824,7 @@ static WCHAR *ft_face_get_family_name( FT_Face ft_face, LANGID langid ) if ((family_name = get_face_name( ft_face, TT_NAME_ID_FONT_FAMILY, langid ))) return family_name; - return towstr( CP_ACP, ft_face->family_name ); + return towstr( ft_face->family_name ); } static WCHAR *ft_face_get_style_name( FT_Face ft_face, LANGID langid ) @@ -826,7 +834,7 @@ static WCHAR *ft_face_get_style_name( FT_Face ft_face, LANGID langid ) if ((style_name = get_face_name( ft_face, TT_NAME_ID_FONT_SUBFAMILY, langid ))) return style_name; - return towstr( CP_ACP, ft_face->style_name ); + return towstr( ft_face->style_name ); } static WCHAR *ft_face_get_full_name( FT_Face ft_face, LANGID langid ) @@ -2053,16 +2061,17 @@ static BOOL CDECL fontconfig_enum_family_fallbacks( DWORD pitch_and_family, int { #ifdef SONAME_LIBFONTCONFIG FcPattern *pat; - FcChar8 *str; + 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 (!pat) return FALSE; - if (pFcPatternGetString( pat, FC_FAMILY, index, &str ) != FcResultMatch) return FALSE; - if (!MultiByteToWideChar( CP_UTF8, 0, (char *)str, -1, buffer, LF_FACESIZE )) - buffer[LF_FACESIZE - 1] = 0; + if (pFcPatternGetString( pat, FC_FAMILY, index, (FcChar8 **)&str ) != FcResultMatch) return FALSE; + RtlUTF8ToUnicodeN( buffer, (LF_FACESIZE - 1) * sizeof(WCHAR), &len, str, strlen(str) ); + buffer[len / sizeof(WCHAR)] = 0; return TRUE; #endif return FALSE; @@ -2228,10 +2237,11 @@ static BOOL CDECL freetype_get_glyph_index( struct gdi_font *font, UINT *glyph, if (!(*glyph = get_glyph_index_symbol( font, *glyph ))) { WCHAR wc = *glyph; + DWORD len; char ch; - if (WideCharToMultiByte( CP_ACP, 0, &wc, 1, &ch, 1, NULL, NULL )) - *glyph = get_glyph_index_symbol( font, (unsigned char)ch ); + RtlUnicodeToMultiByteN( &ch, 1, &len, &wc, sizeof(wc) ); + if (len) *glyph = get_glyph_index_symbol( font, (unsigned char)ch ); } return TRUE; }

1 0

Alexandre Julliard : gdi32: Use ntdll functions for DOS<->Unix path conversions.
by Alexandre Julliard 05 Nov '20

05 Nov '20

Module: wine Branch: master Commit: acc1f6545f2afa6b8365933833aa50f759cf0656 URL: https://source.winehq.org/git/wine.git/?a=commit;h=acc1f6545f2afa6b83659338… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Nov 5 10:04:15 2020 +0100 gdi32: Use ntdll functions for DOS<->Unix path conversions. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdi32/freetype.c | 59 ++++++++++++++++++++++++++++++++++++++++++++++++--- 1 file changed, 56 insertions(+), 3 deletions(-) diff --git a/dlls/gdi32/freetype.c b/dlls/gdi32/freetype.c index 7324622a4e3..c3626f18358 100644 --- a/dlls/gdi32/freetype.c +++ b/dlls/gdi32/freetype.c @@ -93,6 +93,8 @@ #endif #endif /* HAVE_FT2BUILD_H */ +#include "ntstatus.h" +#define WIN32_NO_STATUS #include "windef.h" #include "winbase.h" #include "winternl.h" @@ -1097,6 +1099,57 @@ fail: return NULL; } +static WCHAR *get_dos_file_name( LPCSTR str ) +{ + WCHAR *buffer; + SIZE_T len = strlen(str) + 1; + + len += 8; /* \??\unix prefix */ + if (!(buffer = RtlAllocateHeap( GetProcessHeap(), 0, len * sizeof(WCHAR) ))) return NULL; + if (wine_unix_to_nt_file_name( str, buffer, &len )) + { + RtlFreeHeap( GetProcessHeap(), 0, buffer ); + return NULL; + } + if (buffer[5] == ':') + { + /* get rid of the \??\ prefix */ + /* FIXME: should implement RtlNtPathNameToDosPathName and use that instead */ + memmove( buffer, buffer + 4, (len - 4) * sizeof(WCHAR) ); + } + else buffer[1] = '\\'; + return buffer; +} + +static char *get_unix_file_name( LPCWSTR dosW ) +{ + UNICODE_STRING nt_name; + NTSTATUS status; + SIZE_T size = 256; + char *buffer; + + if (!RtlDosPathNameToNtPathName_U( dosW, &nt_name, NULL, NULL )) return NULL; + for (;;) + { + if (!(buffer = RtlAllocateHeap( GetProcessHeap(), 0, size ))) + { + RtlFreeUnicodeString( &nt_name ); + return NULL; + } + status = wine_nt_to_unix_file_name( &nt_name, buffer, &size, FILE_OPEN_IF ); + if (status != STATUS_BUFFER_TOO_SMALL) break; + RtlFreeHeap( GetProcessHeap(), 0, buffer ); + } + RtlFreeUnicodeString( &nt_name ); + if (status && status != STATUS_NO_SUCH_FILE) + { + RtlFreeHeap( GetProcessHeap(), 0, buffer ); + RtlSetLastWin32ErrorAndNtStatusFromNtStatus( status ); + return NULL; + } + return buffer; +} + static INT AddFontToList(const WCHAR *dos_name, const char *unix_name, void *font_data_ptr, DWORD font_data_size, DWORD flags) { @@ -1129,7 +1182,7 @@ static INT AddFontToList(const WCHAR *dos_name, const char *unix_name, void *fon } #endif /* HAVE_CARBON_CARBON_H */ - if (!dos_name && unix_name) dos_name = filename = wine_get_dos_file_name( unix_name ); + if (!dos_name && unix_name) dos_name = filename = get_dos_file_name( unix_name ); do { ft_face = new_ft_face( unix_name, font_data_ptr, font_data_size, face_index, flags & ADDFONT_ALLOW_BITMAP ); @@ -1157,7 +1210,7 @@ static INT AddFontToList(const WCHAR *dos_name, const char *unix_name, void *fon static INT CDECL freetype_add_font( const WCHAR *file, DWORD flags ) { int ret = 0; - char *unixname = wine_get_unix_file_name( file ); + char *unixname = get_unix_file_name( file ); if (unixname) { @@ -2110,7 +2163,7 @@ static BOOL CDECL freetype_load_font( struct gdi_font *font ) if (font->file[0]) { - char *filename = wine_get_unix_file_name( font->file ); + char *filename = get_unix_file_name( font->file ); data->mapping = map_font_file( filename ); HeapFree( GetProcessHeap(), 0, filename ); if (!data->mapping)

1 0

Alexandre Julliard : gdi32: Use ntdll functions for locale support.
by Alexandre Julliard 05 Nov '20

05 Nov '20

Module: wine Branch: master Commit: 013f1033c030d92014e2d75c5894eb785f2b1810 URL: https://source.winehq.org/git/wine.git/?a=commit;h=013f1033c030d92014e2d75c… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Nov 5 10:08:58 2020 +0100 gdi32: Use ntdll functions for locale support. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdi32/freetype.c | 61 ++++++++++++++++++++++++++++++++++----------------- 1 file changed, 41 insertions(+), 20 deletions(-) diff --git a/dlls/gdi32/freetype.c b/dlls/gdi32/freetype.c index c3626f18358..e2020fff70f 100644 --- a/dlls/gdi32/freetype.c +++ b/dlls/gdi32/freetype.c @@ -262,6 +262,7 @@ struct font_mapping static struct list mappings_list = LIST_INIT( mappings_list ); static UINT default_aa_flags; +static LCID system_lcid; static BOOL CDECL freetype_set_outline_text_metrics( struct gdi_font *font ); static BOOL CDECL freetype_set_bitmap_text_metrics( struct gdi_font *font ); @@ -923,7 +924,6 @@ static inline void get_bitmap_size( FT_Face ft_face, struct bitmap_font_size *fa static inline void get_fontsig( FT_Face ft_face, FONTSIGNATURE *fs ) { TT_OS2 *os2; - CHARSETINFO csi; FT_WinFNT_HeaderRec winfnt_header; int i; @@ -956,8 +956,25 @@ static inline void get_fontsig( FT_Face ft_face, FONTSIGNATURE *fs ) { TRACE("pix_h %d charset %d dpi %dx%d pt %d\n", winfnt_header.pixel_height, winfnt_header.charset, winfnt_header.vertical_resolution,winfnt_header.horizontal_resolution, winfnt_header.nominal_point_size); - if (TranslateCharsetInfo( (DWORD*)(UINT_PTR)winfnt_header.charset, &csi, TCI_SRCCHARSET )) - *fs = csi.fs; + switch (winfnt_header.charset) + { + case ANSI_CHARSET: fs->fsCsb[0] = FS_LATIN1; break; + case EASTEUROPE_CHARSET: fs->fsCsb[0] = FS_LATIN2; break; + case RUSSIAN_CHARSET: fs->fsCsb[0] = FS_CYRILLIC; break; + case GREEK_CHARSET: fs->fsCsb[0] = FS_GREEK; break; + case TURKISH_CHARSET: fs->fsCsb[0] = FS_TURKISH; break; + case HEBREW_CHARSET: fs->fsCsb[0] = FS_HEBREW; break; + case ARABIC_CHARSET: fs->fsCsb[0] = FS_ARABIC; break; + case BALTIC_CHARSET: fs->fsCsb[0] = FS_BALTIC; break; + case VIETNAMESE_CHARSET: fs->fsCsb[0] = FS_VIETNAMESE; break; + case THAI_CHARSET: fs->fsCsb[0] = FS_THAI; break; + case SHIFTJIS_CHARSET: fs->fsCsb[0] = FS_JISJAPAN; break; + case GB2312_CHARSET: fs->fsCsb[0] = FS_CHINESESIMP; break; + case HANGEUL_CHARSET: fs->fsCsb[0] = FS_WANSUNG; break; + case CHINESEBIG5_CHARSET: fs->fsCsb[0] = FS_CHINESETRAD; break; + case JOHAB_CHARSET: fs->fsCsb[0] = FS_JOHAB; break; + case SYMBOL_CHARSET: fs->fsCsb[0] = FS_SYMBOL; break; + } } } @@ -988,21 +1005,23 @@ static int AddFaceToList(FT_Face ft_face, const WCHAR *file, void *data_ptr, SIZ struct bitmap_font_size size; FONTSIGNATURE fs; int ret; - WCHAR *family_name = ft_face_get_family_name( ft_face, GetSystemDefaultLCID() ); + WCHAR *family_name = ft_face_get_family_name( ft_face, system_lcid ); WCHAR *second_name = ft_face_get_family_name( ft_face, MAKELANGID(LANG_ENGLISH, SUBLANG_DEFAULT) ); - WCHAR *style_name = ft_face_get_style_name( ft_face, GetSystemDefaultLangID() ); - WCHAR *full_name = ft_face_get_full_name( ft_face, GetSystemDefaultLangID() ); + WCHAR *style_name = ft_face_get_style_name( ft_face, system_lcid ); + WCHAR *full_name = ft_face_get_full_name( ft_face, system_lcid ); /* try to find another secondary name, preferring the lowest langids */ - if (!strcmpiW( family_name, second_name )) + if (!RtlCompareUnicodeStrings( family_name, lstrlenW(family_name), + second_name, lstrlenW(second_name), TRUE )) { HeapFree( GetProcessHeap(), 0, second_name ); second_name = ft_face_get_family_name( ft_face, MAKELANGID(LANG_NEUTRAL, SUBLANG_NEUTRAL) ); - } - if (!strcmpiW( family_name, second_name )) - { - HeapFree( GetProcessHeap(), 0, second_name ); - second_name = NULL; + if (!RtlCompareUnicodeStrings( family_name, lstrlenW(family_name), + second_name, lstrlenW(second_name), TRUE )) + { + RtlFreeHeap( GetProcessHeap(), 0, second_name ); + second_name = NULL; + } } get_fontsig( ft_face, &fs ); @@ -1657,6 +1676,7 @@ BOOL WineEngInit( const struct font_backend_funcs **funcs ) #endif *funcs = &font_funcs; + NtQueryDefaultLocale( FALSE, &system_lcid ); return TRUE; } @@ -2189,9 +2209,9 @@ static BOOL CDECL freetype_load_font( struct gdi_font *font ) if (!font->aa_flags) font->aa_flags = ADDFONT_AA_FLAGS( default_aa_flags ); if (!font->otm.otmpFamilyName) { - font->otm.otmpFamilyName = (char *)ft_face_get_family_name( ft_face, GetSystemDefaultLCID() ); - font->otm.otmpStyleName = (char *)ft_face_get_style_name( ft_face, GetSystemDefaultLangID() ); - font->otm.otmpFaceName = (char *)ft_face_get_full_name( ft_face, GetSystemDefaultLangID() ); + font->otm.otmpFamilyName = (char *)ft_face_get_family_name( ft_face, system_lcid ); + font->otm.otmpStyleName = (char *)ft_face_get_style_name( ft_face, system_lcid ); + font->otm.otmpFaceName = (char *)ft_face_get_full_name( ft_face, system_lcid ); } if (font->scalable) @@ -3445,8 +3465,7 @@ static BOOL CDECL freetype_set_outline_text_metrics( struct gdi_font *font ) /* note: we store actual pointers in the names instead of offsets, they are fixed up when returned to the app */ - if (!(font->otm.otmpFullName = (char *)get_face_name( ft_face, TT_NAME_ID_UNIQUE_ID, - GetSystemDefaultLangID() ))) + if (!(font->otm.otmpFullName = (char *)get_face_name( ft_face, TT_NAME_ID_UNIQUE_ID, system_lcid ))) { static const WCHAR fake_nameW[] = {'f','a','k','e',' ','n','a','m','e', 0}; FIXME("failed to read full_nameW for font %s!\n", wine_dbgstr_w((WCHAR *)font->otm.otmpFamilyName)); @@ -3547,12 +3566,14 @@ static BOOL CDECL freetype_set_outline_text_metrics( struct gdi_font *font ) if (face_has_symbol_charmap(ft_face) || (pOS2->usFirstCharIndex >= 0xf000 && pOS2->usFirstCharIndex < 0xf100)) { TM.tmFirstChar = 0; - switch(GetACP()) + switch (PRIMARYLANGID(system_lcid)) { - case 1255: /* Hebrew */ + case LANG_HEBREW: TM.tmLastChar = 0xf896; break; - case 1257: /* Baltic */ + case LANG_ESTONIAN: + case LANG_LATVIAN: + case LANG_LITHUANIAN: TM.tmLastChar = 0xf8fd; break; default:

1 0

Alexandre Julliard : gdi32: Use ntdll functions for heap allocations.
by Alexandre Julliard 05 Nov '20

05 Nov '20

Module: wine Branch: master Commit: 54a01ea1ec3fa77620100b643d164ab361927173 URL: https://source.winehq.org/git/wine.git/?a=commit;h=54a01ea1ec3fa77620100b64… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Nov 5 10:11:10 2020 +0100 gdi32: Use ntdll functions for heap allocations. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdi32/freetype.c | 80 +++++++++++++++++++++++++-------------------------- 1 file changed, 40 insertions(+), 40 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=54a01ea1ec3fa7762010…

1 0

Anton Baskanov : amstream: Release filter critical section before calling upstream seeking methods.
by Alexandre Julliard 04 Nov '20

04 Nov '20

Module: wine Branch: master Commit: 8492657ab85ad31102509ff5e257033da07ac56d URL: https://source.winehq.org/git/wine.git/?a=commit;h=8492657ab85ad31102509ff5… Author: Anton Baskanov <baskanov(a)gmail.com> Date: Thu Oct 29 11:40:58 2020 +0700 amstream: Release filter critical section before calling upstream seeking methods. Otherwise a seek might deadlock when the stream calls the filter methods (e.g. WaitUntil) from streaming thread. Signed-off-by: Anton Baskanov <baskanov(a)gmail.com> Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/amstream/filter.c | 29 ++++++++++++++--------------- 1 file changed, 14 insertions(+), 15 deletions(-) diff --git a/dlls/amstream/filter.c b/dlls/amstream/filter.c index b63a0301d64..daeb1fafe71 100644 --- a/dlls/amstream/filter.c +++ b/dlls/amstream/filter.c @@ -795,15 +795,16 @@ static HRESULT WINAPI filter_seeking_GetDuration(IMediaSeeking *iface, LONGLONG EnterCriticalSection(&filter->cs); - if (!(seeking = get_seeking(filter->seekable_stream))) - { - LeaveCriticalSection(&filter->cs); + seeking = get_seeking(filter->seekable_stream); + + LeaveCriticalSection(&filter->cs); + + if (!seeking) return E_NOTIMPL; - } + hr = IMediaSeeking_GetDuration(seeking, duration); IMediaSeeking_Release(seeking); - LeaveCriticalSection(&filter->cs); return hr; } @@ -817,15 +818,16 @@ static HRESULT WINAPI filter_seeking_GetStopPosition(IMediaSeeking *iface, LONGL EnterCriticalSection(&filter->cs); - if (!(seeking = get_seeking(filter->seekable_stream))) - { - LeaveCriticalSection(&filter->cs); + seeking = get_seeking(filter->seekable_stream); + + LeaveCriticalSection(&filter->cs); + + if (!seeking) return E_NOTIMPL; - } + hr = IMediaSeeking_GetStopPosition(seeking, stop); IMediaSeeking_Release(seeking); - LeaveCriticalSection(&filter->cs); return hr; } @@ -860,18 +862,15 @@ static HRESULT WINAPI filter_seeking_SetPositions(IMediaSeeking *iface, LONGLONG seeking = get_seeking(filter->seekable_stream); + LeaveCriticalSection(&filter->cs); + if (!seeking) - { - LeaveCriticalSection(&filter->cs); return E_NOTIMPL; - } hr = IMediaSeeking_SetPositions(seeking, current_ptr, current_flags, stop_ptr, stop_flags); IMediaSeeking_Release(seeking); - LeaveCriticalSection(&filter->cs); - return hr; }

1 0

Anton Baskanov : amstream: Implement MediaStreamFilter::WaitUntil.
by Alexandre Julliard 04 Nov '20

04 Nov '20

Module: wine Branch: master Commit: 28764bd0c56257428e50ef2ce4805d42e1882362 URL: https://source.winehq.org/git/wine.git/?a=commit;h=28764bd0c56257428e50ef2c… Author: Anton Baskanov <baskanov(a)gmail.com> Date: Thu Oct 29 11:40:59 2020 +0700 amstream: Implement MediaStreamFilter::WaitUntil. Signed-off-by: Anton Baskanov <baskanov(a)gmail.com> Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/amstream/filter.c | 87 +++++++++++++++++++- dlls/amstream/tests/amstream.c | 179 ++++++++++++++++++++++++++++++++++++++++- 2 files changed, 260 insertions(+), 6 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=28764bd0c56257428e50…

1 0