[Libreoffice-commits] core.git: Branch 'libreoffice-4-1' - vcl/coretext

Khaled Hosny khaledhosny at eglug.org
Thu Jun 20 04:12:27 PDT 2013


 vcl/coretext/ctfonts.cxx |    1 -
 1 file changed, 1 deletion(-)

New commits:
commit 84b42f8203290276282b48e83403fe7a4833db28
Author: Khaled Hosny <khaledhosny at eglug.org>
Date:   Tue Jun 18 01:48:51 2013 +0200

    Remove pointless TODO note
    
    kCTFontDisplayNameAttribute is something like Foo Regular and Foo Bold,
    we certainly don’t want that.
    
    Change-Id: Ia24a2abfcbb28883b2c3d05f66070583ddb862c9
    (cherry picked from commit a80d2a760121a1b02606d9f5f703c286aa3ba971)
    Reviewed-on: https://gerrit.libreoffice.org/4385
    Reviewed-by: Noel Power <noel.power at suse.com>
    Tested-by: Noel Power <noel.power at suse.com>

diff --git a/vcl/coretext/ctfonts.cxx b/vcl/coretext/ctfonts.cxx
index ae77b71..d4cf1ca 100644
--- a/vcl/coretext/ctfonts.cxx
+++ b/vcl/coretext/ctfonts.cxx
@@ -317,7 +317,6 @@ ImplDevFontAttributes DevFontFromCTFontDescriptor( CTFontDescriptorRef pFD, bool
     rDFA.mbSubsettable = true;
 
     // get font name
-    // TODO: use kCTFontDisplayNameAttribute instead???
     CFStringRef pFamilyName = (CFStringRef)CTFontDescriptorCopyAttribute( pFD, kCTFontFamilyNameAttribute );
     rDFA.SetFamilyName( GetOUString( pFamilyName ) );
     // get font style


More information about the Libreoffice-commits mailing list