vcl/quartz/ctfonts.cxx |    2 --
 1 file changed, 2 deletions(-)

New commits:
commit 7017ae2fd7161ee9ed07095f56dd3da9eb9e6b10
Author:     Caolán McNamara <caol...@redhat.com>
AuthorDate: Sat Nov 19 10:08:57 2022 +0000
Commit:     Caolán McNamara <caol...@redhat.com>
CommitDate: Mon Nov 21 09:55:53 2022 +0100

    IsSymbolFont means that its a Windows Symbol Encoding
    
    so kCTFontSymbolicClass doesn't fulfil that meaning, if we want
    something more generic to indicate that a font is only useful
    for "icons, dingbats, technical symbols, and so on." then we need
    a different thing. Its baked in everywhere else that a "SymbolFont"
    uses RTL_TEXTENCODING_SYMBOL.
    
    Change-Id: Ie99b5068077dd385dec26f62e2e991e8381128d6
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/142961
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caol...@redhat.com>

diff --git a/vcl/quartz/ctfonts.cxx b/vcl/quartz/ctfonts.cxx
index 89460c3943a1..93a862653b74 100644
--- a/vcl/quartz/ctfonts.cxx
+++ b/vcl/quartz/ctfonts.cxx
@@ -379,8 +379,6 @@ FontAttributes DevFontFromCTFontDescriptor( 
CTFontDescriptorRef pFD, bool* bFont
     if( CFDictionaryGetValueIfPresent( pAttrDict, kCTFontSymbolicTrait, 
reinterpret_cast<const void**>(&pSymbolNum) ) )
     {
         CFNumberGetValue( pSymbolNum, kCFNumberSInt64Type, &nSymbolTrait );
-        rDFA.SetSymbolFlag( (nSymbolTrait & kCTFontClassMaskTrait) == 
kCTFontSymbolicClass );
-
         if (nSymbolTrait & kCTFontMonoSpaceTrait)
             rDFA.SetPitch(PITCH_FIXED);
     }

Reply via email to