vcl/unx/generic/glyphs/freetype_glyphcache.cxx |    5 -----
 1 file changed, 5 deletions(-)

New commits:
commit fd1b475143572655db18d2f1bc12309e3f5ab8d6
Author:     Caolán McNamara <caol...@redhat.com>
AuthorDate: Mon Nov 21 09:10:50 2022 +0000
Commit:     Caolán McNamara <caol...@redhat.com>
CommitDate: Mon Nov 21 16:18:05 2022 +0100

    drop FT_Select_Charmap
    
    khaled: "I think this is probably a no-op now as we don’t call 
FT_Get_Char_Index
    anywhere (since mapping characters to glyphs is handled by HarfBuzz
    now). I think the whole symbol font concept might be no-op as HarfBuzz
    will internally use the symbol cmap subtable if the font has one and we
    don’t need to (or can) influence this."
    
    last use of FT_Get_Char_Index was removed with
    
    commit 23c5125148a8110d88385b29570bf0b7d4400458
    Date:   Thu May 10 15:46:06 2018 +0200
    
        Use HarfBuzz to get Kashida width
    
    Change-Id: I0c0b008fa8b8b1e10125cc2aa10b9f079df795d1
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/143018
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caol...@redhat.com>

diff --git a/vcl/unx/generic/glyphs/freetype_glyphcache.cxx 
b/vcl/unx/generic/glyphs/freetype_glyphcache.cxx
index aca47bad7518..1c5f41be951c 100644
--- a/vcl/unx/generic/glyphs/freetype_glyphcache.cxx
+++ b/vcl/unx/generic/glyphs/freetype_glyphcache.cxx
@@ -452,11 +452,6 @@ FreetypeFont::FreetypeFont(FreetypeFontInstance& 
rFontInstance, std::shared_ptr<
      * not need any glyph data from FreeType in this case */
     /*FT_Error rc = */ FT_Set_Pixel_Sizes( maFaceFT, mnWidth, rFSD.mnHeight );
 
-    if (mxFontInfo->IsMicrosoftSymbolEncoded())
-        FT_Select_Charmap(maFaceFT, FT_ENCODING_MS_SYMBOL);
-    else
-        FT_Select_Charmap(maFaceFT, FT_ENCODING_UNICODE);
-
     mbFaceOk = true;
 
     // TODO: query GASP table for load flags

Reply via email to