]> git.saurik.com Git - wxWidgets.git/blobdiff - src/mac/carbon/font.cpp
remove some duplication in font scaling code
[wxWidgets.git] / src / mac / carbon / font.cpp
index 84920cb6c427c12a5a95eff42cf811e34d616e41..beee3a6e188015418e470aa760337ab7ecec2d16 100644 (file)
 #include <ATSUnicode.h>
 #endif
 
+#include <map>
 
 IMPLEMENT_DYNAMIC_CLASS(wxFont, wxGDIObject)
 
 
 class WXDLLEXPORT wxFontRefData: public wxGDIRefData
 {
-    friend class wxFont;
-
 public:
     wxFontRefData()
     {
@@ -62,13 +61,78 @@ public:
         Init(size, family, style, weight, underlined, faceName, encoding);
     }
 
+#if wxMAC_USE_CORE_TEXT
+    wxFontRefData( wxUint32 coreTextFontType );
+    wxFontRefData( CTFontRef font );
+    wxFontRefData( CTFontDescriptorRef fontdescriptor, int size );
+#endif
+
     virtual ~wxFontRefData();
 
-    void SetNoAntiAliasing( bool no = true )
-    { m_noAA = no; }
+    void SetNoAntiAliasing( bool no = true ) { m_noAA = no; }
+
+    bool GetNoAntiAliasing() const { return m_noAA; }
+
+    void SetPointSize( int size )
+    {
+        m_pointSize = size;
+        MacInvalidateNativeFont();
+    }
+
+    int GetPointSize() const { return m_pointSize; }
+
+    void SetFamily( int family )
+    {
+        m_family = family;
+        MacInvalidateNativeFont();
+    }
+
+
+    int GetFamily() const { return m_family; }
+
+    void SetStyle( int style )
+    {
+        m_style = style;
+        MacInvalidateNativeFont();
+    }
+
+
+    int GetStyle() const { return m_style; }
+
+    void SetWeight( int weight )
+    {
+        m_weight = weight;
+        MacInvalidateNativeFont();
+    }
+
+
+    int GetWeight() const { return m_weight; }
+
+    void SetUnderlined( bool u )
+    {
+        m_underlined = u;
+        MacInvalidateNativeFont();
+    }
 
-    bool GetNoAntiAliasing() const
-    { return m_noAA; }
+    bool GetUnderlined() const { return m_underlined; }
+
+    void SetFaceName( const wxString& facename )
+    {
+        m_faceName = facename;
+        MacInvalidateNativeFont();
+    }
+
+    const wxString& GetFaceName() const { return m_faceName; }
+
+    void SetEncoding( wxFontEncoding encoding )
+    {
+        m_encoding = encoding;
+        MacInvalidateNativeFont();
+    }
+
+    wxFontEncoding GetEncoding() const { return m_encoding; }
+
+    void MacInvalidateNativeFont();
 
     void MacFindFont();
 
@@ -82,6 +146,9 @@ protected:
               const wxString& faceName,
               wxFontEncoding encoding);
 
+#if wxMAC_USE_CORE_TEXT
+    void Init( CTFontRef font );
+#endif
     // font characterstics
     int             m_pointSize;
     int             m_family;
@@ -110,11 +177,13 @@ public:
     // for true themeing support we must store the correct font
     // information here, as this speeds up and optimizes rendering
     ThemeFontID     m_macThemeFontID ;
-    ATSUStyle       m_macATSUStyle ;
 #endif
 #if wxMAC_USE_CORE_TEXT
     wxCFRef<CTFontRef> m_ctFont;
-    CTFontUIFontType m_macUIFontType;
+    wxCFRef<CTFontDescriptorRef> m_ctFontDescriptor;
+#endif
+#if wxMAC_USE_CORE_TEXT || wxMAC_USE_ATSU_TEXT
+    ATSUStyle       m_macATSUStyle ;
 #endif
     wxNativeFontInfo  m_info;
 };
@@ -147,9 +216,6 @@ void wxFontRefData::Init(int pointSize,
     m_faceName = faceName;
     m_encoding = encoding;
     m_noAA = false;
-#ifdef wxMAC_USE_CORE_TEXT
-    m_macUIFontType = kCTFontNoFontType;
-#endif
 #if wxMAC_USE_ATSU_TEXT
     m_macFontFamily = 0 ;
     m_macFontSize = 0;
@@ -157,39 +223,58 @@ void wxFontRefData::Init(int pointSize,
     m_macATSUFontID = 0;
     m_macATSUAdditionalQDStyles = 0 ;
     m_macThemeFontID = kThemeCurrentPortFont ;
+#endif
+#if wxMAC_USE_CORE_TEXT || wxMAC_USE_ATSU_TEXT
     m_macATSUStyle = NULL ;
 #endif
 }
 
 wxFontRefData::~wxFontRefData()
 {
+#if wxMAC_USE_CORE_TEXT || wxMAC_USE_ATSU_TEXT
+    if ( m_macATSUStyle )
+    {
+        ::ATSUDisposeStyle((ATSUStyle)m_macATSUStyle);
+        m_macATSUStyle = NULL ;
+    }
+#endif
+}
+
+void wxFontRefData::MacInvalidateNativeFont()
+{
+#if wxMAC_USE_CORE_TEXT
+    m_ctFont.reset();
+    m_ctFontDescriptor.reset();
+#endif
+#if wxMAC_USE_CORE_TEXT || wxMAC_USE_ATSU_TEXT
     if ( m_macATSUStyle )
     {
         ::ATSUDisposeStyle((ATSUStyle)m_macATSUStyle);
         m_macATSUStyle = NULL ;
     }
+#endif
 }
 
 #if wxMAC_USE_CORE_TEXT
 
-/* code mixed together from 2 different routines from Core Text Manual Common Operations */ 
+/* from Core Text Manual Common Operations */
 
-static CTFontRef wxMacCreateCTFont(CFStringRef iFamilyName, CTFontSymbolicTraits iTraits, CGFloat iSize )
+static CTFontDescriptorRef wxMacCreateCTFontDescriptor(CFStringRef iFamilyName, CTFontSymbolicTraits iTraits )
 {
     CTFontDescriptorRef descriptor = NULL;
     CFMutableDictionaryRef attributes;
-    
+
     assert(iFamilyName != NULL);
     // Create a mutable dictionary to hold our attributes.
     attributes = CFDictionaryCreateMutable(kCFAllocatorDefault, 0,
                                            &kCFTypeDictionaryKeyCallBacks, &kCFTypeDictionaryValueCallBacks);
     check(attributes != NULL);
-    
-    if (attributes != NULL) {        
+
+    if (attributes != NULL) {
         // Add a family name to our attributes.
         CFDictionaryAddValue(attributes, kCTFontFamilyNameAttribute, iFamilyName);
-        
-        
+
+
         if ( iTraits ) {
             CFMutableDictionaryRef traits;
             CFNumberRef symTraits;
@@ -198,17 +283,17 @@ static CTFontRef wxMacCreateCTFont(CFStringRef iFamilyName, CTFontSymbolicTraits
             symTraits = CFNumberCreate(kCFAllocatorDefault, kCFNumberSInt32Type,
                                        &iTraits);
             check(symTraits != NULL);
-            
+
             if (symTraits != NULL) {
                 // Create a dictionary to hold our traits values.
                 traits = CFDictionaryCreateMutable(kCFAllocatorDefault, 0,
                                                    &kCFTypeDictionaryKeyCallBacks, &kCFTypeDictionaryValueCallBacks);
                 check(traits != NULL);
-                
+
                 if (traits != NULL) {
                     // Add the symbolic traits value to the traits dictionary.
                     CFDictionaryAddValue(traits, kCTFontSymbolicTrait, symTraits);
-                    
+
                     // Add the traits attribute to our attributes.
                     CFDictionaryAddValue(attributes, kCTFontTraitsAttribute, traits);
                     CFRelease(traits);
@@ -216,46 +301,80 @@ static CTFontRef wxMacCreateCTFont(CFStringRef iFamilyName, CTFontSymbolicTraits
                 CFRelease(symTraits);
             }
         }
-        // Create the font descriptor with our attributes and input size.
+        // Create the font descriptor with our attributes
         descriptor = CTFontDescriptorCreateWithAttributes(attributes);
         check(descriptor != NULL);
-        
+
         CFRelease(attributes);
     }
     // Return our font descriptor.
-    CTFontRef font = CTFontCreateWithFontDescriptor( descriptor, iSize, NULL);
-    CFRelease( descriptor );
-    return font ;
+    return descriptor ;
+}
+
+wxFontRefData::wxFontRefData( wxUint32 coreTextFontType )
+{
+    CTFontRef font = CTFontCreateUIFontForLanguage( coreTextFontType, 0.0, NULL ) ;
+    if ( CTFontGetSize(m_ctFont) == 0 )
+    {
+        CFRelease(font);
+        font = CTFontCreateUIFontForLanguage( coreTextFontType, 12.0, NULL );
+    }
+    Init( font );
+}
+
+wxFontRefData::wxFontRefData( CTFontRef font )
+{
+    Init( font );
+}
+
+wxFontRefData::wxFontRefData( CTFontDescriptorRef fontdescriptor, int size )
+{
+    if ( size == 0 )
+    {
+        wxCFRef< CFNumberRef > value( (CFNumberRef) CTFontDescriptorCopyAttribute( fontdescriptor, kCTFontSizeAttribute ) );
+
+        float fsize;
+        if ( CFNumberGetValue( value , kCFNumberFloatType , &fsize ) )
+        {
+            size = (int) fsize + 0.5 ;
+        }
+    }
+    Init( CTFontCreateWithFontDescriptor(fontdescriptor, size,NULL)  );
+}
+
+void wxFontRefData::Init( CTFontRef font )
+{
+    Init(10, wxDEFAULT, wxNORMAL, wxNORMAL,
+         false, wxT("applicationfont"), wxFONTENCODING_DEFAULT);
+
+    m_ctFont.reset( font );
 }
 
 #endif
 
 void wxFontRefData::MacFindFont()
 {
-    OSStatus status = noErr;
 
 #if wxMAC_USE_CORE_TEXT
     if (  UMAGetSystemVersion() >= 0x1050 )
     {
-        if ( m_faceName.empty() && m_family == wxDEFAULT && m_macUIFontType == kCTFontNoFontType )
+        if ( m_faceName.empty() && m_family == wxDEFAULT )
         {
-            m_macUIFontType = kCTFontSystemFontType;
+            m_ctFont.reset(CTFontCreateUIFontForLanguage( kCTFontSystemFontType, 0.0, NULL ));
         }
-        
-        if ( m_macUIFontType != kCTFontNoFontType )
+
+        if ( m_ctFont )
         {
-            m_ctFont.reset(CTFontCreateUIFontForLanguage( m_macUIFontType, 0.0, NULL ));
-            
-            wxMacCFStringHolder name( CTFontCopyFamilyName( m_ctFont ) );
+            wxCFStringRef name( CTFontCopyFamilyName( m_ctFont ) );
             m_faceName = name.AsString();
-            if ( CTFontGetSize(m_ctFont) == 0 )
-            {
-                m_ctFont.reset(CTFontCreateUIFontForLanguage( m_macUIFontType, 12, NULL ));
-            }
-            
             m_pointSize = CTFontGetSize(m_ctFont) ;
-            // reset this so that future manipulation don't fall back
-            m_macUIFontType = kCTFontNoFontType;
+            CTFontSymbolicTraits traits = CTFontGetSymbolicTraits( m_ctFont );
+            if ( traits & kCTFontItalicTrait )
+                m_style = wxITALIC;
+            if (  traits & kCTFontBoldTrait )
+                m_weight = wxBOLD ;
+            if ( !m_ctFontDescriptor.get() )
+                m_ctFontDescriptor.reset( CTFontCopyFontDescriptor( m_ctFont ) );
         }
         else
         {
@@ -268,36 +387,118 @@ void wxFontRefData::MacFindFont()
                     case wxDECORATIVE :
                         m_faceName = wxT("Times");
                         break ;
-                        
+
                     case wxSWISS :
                         m_faceName =  wxT("Lucida Grande");
                         break ;
-                        
+
                     case wxMODERN :
                     case wxTELETYPE:
                         m_faceName =  wxT("Monaco");
                         break ;
-                        
+
                     default:
                         m_faceName =  wxT("Times");
                         break ;
                 }
             }
 
-            wxMacCFStringHolder cf( m_faceName, wxLocale::GetSystemEncoding() );
+            
             CTFontSymbolicTraits traits = 0;
-        
+
             if (m_weight == wxBOLD)
                 traits |= kCTFontBoldTrait;
             if (m_style == wxITALIC || m_style == wxSLANT)
                 traits |= kCTFontItalicTrait;
+     
+// use font descriptor caching
+#if 1
+            wxString lookupname = wxString::Format( "%s_%ld", m_faceName.c_str(), traits );
+            
+            static std::map< std::wstring , wxCFRef< CTFontDescriptorRef > > fontdescriptorcache ;
+            
+            m_ctFontDescriptor = fontdescriptorcache[ std::wstring(lookupname.wc_str()) ];
+            if ( !m_ctFontDescriptor )
+            {
+                wxCFStringRef cf( m_faceName, wxLocale::GetSystemEncoding() );
+                m_ctFontDescriptor.reset( wxMacCreateCTFontDescriptor( cf, traits ) );
+                fontdescriptorcache[ std::wstring(lookupname.wc_str()) ] = m_ctFontDescriptor;
+            }
+#else
+            wxCFStringRef cf( m_faceName, wxLocale::GetSystemEncoding() );
+            m_ctFontDescriptor.reset( wxMacCreateCTFontDescriptor( cf, traits ) );
+#endif
+            
+// use font caching
+#if 1
+            wxString lookupnameWithSize = wxString::Format( "%s_%ld_%ld", m_faceName.c_str(), traits, m_pointSize );
             
-            m_ctFont.reset( wxMacCreateCTFont( cf, traits, m_pointSize ) );
+            static std::map< std::wstring , wxCFRef< CTFontRef > > fontcache ;
+            m_ctFont = fontcache[ std::wstring(lookupnameWithSize.wc_str()) ];
+            if ( !m_ctFont )
+            {
+                m_ctFont.reset( CTFontCreateWithFontDescriptor( m_ctFontDescriptor, m_pointSize, NULL ) );
+                fontcache[ std::wstring(lookupnameWithSize.wc_str()) ] = m_ctFont;
+            }
+#else
+            m_ctFont.reset( CTFontCreateWithFontDescriptor( m_ctFontDescriptor, m_pointSize, NULL ) );
+#endif
+        }
+#if wxMAC_USE_ATSU_TEXT == 0
+        OSStatus status = noErr;
+        CTFontDescriptorRef desc = m_ctFontDescriptor ;
+        ATSFontRef atsfont = CTFontGetPlatformFont( m_ctFont, &desc );
+        FMFont fmfont = FMGetFontFromATSFontRef( atsfont );
+        ATSUAttributeTag atsuTags[] =
+        {
+            kATSUFontTag ,
+            kATSUSizeTag ,
+            kATSUVerticalCharacterTag,
+            kATSUQDBoldfaceTag ,
+            kATSUQDItalicTag ,
+            kATSUQDUnderlineTag ,
+        };
+        ByteCount atsuSizes[sizeof(atsuTags) / sizeof(ATSUAttributeTag)] =
+        {
+            sizeof( ATSUFontID ) ,
+            sizeof( Fixed ) ,
+            sizeof( ATSUVerticalCharacterType),
+            sizeof( Boolean ) ,
+            sizeof( Boolean ) ,
+            sizeof( Boolean ) ,
+        };
+        Boolean kTrue = true ;
+        Boolean kFalse = false ;
+
+        Fixed atsuSize = IntToFixed( m_pointSize );
+        ATSUVerticalCharacterType kHorizontal = kATSUStronglyHorizontal;
+        ATSUAttributeValuePtr    atsuValues[sizeof(atsuTags) / sizeof(ATSUAttributeTag)] =
+        {
+            &fmfont ,
+            &atsuSize ,
+            &kHorizontal,
+            (m_weight == wxBOLD) ? &kTrue : &kFalse ,
+            (m_style == wxITALIC || m_style == wxSLANT) ? &kTrue : &kFalse ,
+            (m_underlined) ? &kTrue : &kFalse ,
+        };
+
+        if ( m_macATSUStyle )
+        {
+            ::ATSUDisposeStyle((ATSUStyle)m_macATSUStyle);
+            m_macATSUStyle = NULL ;
         }
+        status = ::ATSUCreateStyle((ATSUStyle *)&m_macATSUStyle);
+        wxASSERT_MSG( status == noErr , wxT("couldn't create ATSU style") );
+        status = ::ATSUSetAttributes(
+                                     (ATSUStyle)m_macATSUStyle,
+                                     sizeof(atsuTags) / sizeof(ATSUAttributeTag) ,
+                                     atsuTags, atsuSizes, atsuValues);
+#endif
     }
 #endif
 #if wxMAC_USE_ATSU_TEXT
     {
+        OSStatus status = noErr;
         Str255 qdFontName ;
         if ( m_macThemeFontID != kThemeCurrentPortFont )
         {
@@ -337,16 +538,16 @@ void wxFontRefData::MacFindFont()
                         case wxDECORATIVE :
                             m_faceName = wxT("Times");
                             break ;
-                            
+
                         case wxSWISS :
                             m_faceName =  wxT("Lucida Grande");
                             break ;
-                            
+
                         case wxMODERN :
                         case wxTELETYPE:
                             m_faceName =  wxT("Monaco");
                             break ;
-                            
+
                         default:
                             m_faceName =  wxT("Times");
                             break ;
@@ -368,7 +569,7 @@ void wxFontRefData::MacFindFont()
                     m_macFontFamily = GetAppFont();
                 else
                 {
-                    wxMacCFStringHolder cf( m_faceName, wxLocale::GetSystemEncoding() );
+                    wxCFStringRef cf( m_faceName, wxLocale::GetSystemEncoding() );
                     ATSFontFamilyRef atsfamily = ATSFontFamilyFindFromName( cf , kATSOptionFlagsDefault );
                     if ( atsfamily == (ATSFontFamilyRef) -1 )
                     {
@@ -379,7 +580,7 @@ void wxFontRefData::MacFindFont()
                         m_macFontFamily = FMGetFontFamilyFromATSFontFamilyRef( atsfamily );
                 }
             }
-            
+
             m_macFontStyle = 0;
             if (m_weight == wxBOLD)
                 m_macFontStyle |= bold;
@@ -389,25 +590,25 @@ void wxFontRefData::MacFindFont()
                 m_macFontStyle |= underline;
             m_macFontSize = m_pointSize ;
         }
-        
+
         // we try to get as much styles as possible into ATSU
-        
-        
+
+
         // ATSUFontID and FMFont are equivalent
         FMFontStyle intrinsicStyle = 0 ;
         status = FMGetFontFromFontFamilyInstance( m_macFontFamily , m_macFontStyle , &m_macATSUFontID , &intrinsicStyle);
         wxASSERT_MSG( status == noErr , wxT("couldn't get an ATSUFont from font family") );
         m_macATSUAdditionalQDStyles = m_macFontStyle & (~intrinsicStyle );
-        
+
         if ( m_macATSUStyle )
         {
             ::ATSUDisposeStyle((ATSUStyle)m_macATSUStyle);
             m_macATSUStyle = NULL ;
         }
-        
+
         status = ::ATSUCreateStyle((ATSUStyle *)&m_macATSUStyle);
         wxASSERT_MSG( status == noErr , wxT("couldn't create ATSU style") );
-        
+
         ATSUAttributeTag atsuTags[] =
         {
             kATSUFontTag ,
@@ -430,10 +631,10 @@ void wxFontRefData::MacFindFont()
             sizeof( Boolean ) ,
             sizeof( Boolean ) ,
         };
-        
+
         Boolean kTrue = true ;
         Boolean kFalse = false ;
-        
+
         Fixed atsuSize = IntToFixed( m_macFontSize );
         ATSUVerticalCharacterType kHorizontal = kATSUStronglyHorizontal;
         ATSUAttributeValuePtr    atsuValues[sizeof(atsuTags) / sizeof(ATSUAttributeTag)] =
@@ -447,12 +648,12 @@ void wxFontRefData::MacFindFont()
             (m_macATSUAdditionalQDStyles & condense) ? &kTrue : &kFalse ,
             (m_macATSUAdditionalQDStyles & extend) ? &kTrue : &kFalse ,
         };
-        
+
         status = ::ATSUSetAttributes(
                                      (ATSUStyle)m_macATSUStyle,
                                      sizeof(atsuTags) / sizeof(ATSUAttributeTag) ,
                                      atsuTags, atsuSizes, atsuValues);
-        
+
         wxASSERT_MSG( status == noErr , wxT("couldn't modify ATSU style") );
         return;
     }
@@ -498,31 +699,38 @@ bool wxFont::Create(int pointSize,
 
 #if wxMAC_USE_CORE_TEXT
 
-bool wxFont::MacCreateUIFont(wxUint32 ctFontType )
+bool wxFont::MacCreateFromUIFont(wxUint32 ctFontType )
 {
     UnRef();
 
-    m_refData = new wxFontRefData(
-        12, wxDEFAULT, wxFONTSTYLE_NORMAL, wxFONTWEIGHT_NORMAL,
-        false, wxEmptyString, wxFONTENCODING_DEFAULT );
+    m_refData = new wxFontRefData(ctFontType);
+    RealizeResource();
 
-    M_FONTDATA->m_macUIFontType = ctFontType ;
+    return true;
+}
+
+bool wxFont::MacCreateFromCTFontDescriptor( const void * ctFontDescriptor , int size )
+{
+    UnRef();
+
+    m_refData = new wxFontRefData((CTFontDescriptorRef)ctFontDescriptor, size);;
     RealizeResource();
 
     return true;
 }
 
+
 #endif
 
-bool wxFont::MacCreateThemeFont(wxUint16 themeFontID)
+bool wxFont::MacCreateFromThemeFont(wxUint16 themeFontID)
 {
 #if wxMAC_USE_CORE_TEXT
     if ( UMAGetSystemVersion() >= 0x1050)
     {
-        return MacCreateUIFont(HIThemeGetUIFontType(themeFontID));
+        return MacCreateFromUIFont(HIThemeGetUIFontType(themeFontID));
     }
 #endif
-#if wxMAC_USE_CORE_TEXT
+#if wxMAC_USE_ATSU_TEXT
     {
         UnRef();
 
@@ -532,9 +740,10 @@ bool wxFont::MacCreateThemeFont(wxUint16 themeFontID)
 
         M_FONTDATA->m_macThemeFontID = themeFontID ;
         RealizeResource();
+        return true;
     }
-    return true;
 #endif
+    return false;
 }
 
 wxFont::~wxFont()
@@ -552,7 +761,7 @@ void wxFont::SetEncoding(wxFontEncoding encoding)
 {
     Unshare();
 
-    M_FONTDATA->m_encoding = encoding;
+    M_FONTDATA->SetEncoding( encoding );
 
     RealizeResource();
 }
@@ -572,14 +781,24 @@ void wxFont::Unshare()
     }
 }
 
+wxGDIRefData *wxFont::CreateGDIRefData() const
+{
+    return new wxFontRefData;
+}
+
+wxGDIRefData *wxFont::CloneGDIRefData(const wxGDIRefData *data) const
+{
+    return new wxFontRefData(*wx_static_cast(const wxFontRefData *, data));
+}
+
 void wxFont::SetPointSize(int pointSize)
 {
-    if ( M_FONTDATA->m_pointSize == pointSize )
+    if ( M_FONTDATA->GetPointSize() == pointSize )
         return;
-    
+
     Unshare();
 
-    M_FONTDATA->m_pointSize = pointSize;
+    M_FONTDATA->SetPointSize( pointSize );
 
     RealizeResource();
 }
@@ -588,7 +807,7 @@ void wxFont::SetFamily(int family)
 {
     Unshare();
 
-    M_FONTDATA->m_family = family;
+    M_FONTDATA->SetFamily( family );
 
     RealizeResource();
 }
@@ -597,7 +816,7 @@ void wxFont::SetStyle(int style)
 {
     Unshare();
 
-    M_FONTDATA->m_style = style;
+    M_FONTDATA->SetStyle( style );
 
     RealizeResource();
 }
@@ -606,7 +825,7 @@ void wxFont::SetWeight(int weight)
 {
     Unshare();
 
-    M_FONTDATA->m_weight = weight;
+    M_FONTDATA->SetWeight( weight );
 
     RealizeResource();
 }
@@ -615,7 +834,7 @@ bool wxFont::SetFaceName(const wxString& faceName)
 {
     Unshare();
 
-    M_FONTDATA->m_faceName = faceName;
+    M_FONTDATA->SetFaceName( faceName );
 
     RealizeResource();
 
@@ -626,7 +845,7 @@ void wxFont::SetUnderlined(bool underlined)
 {
     Unshare();
 
-    M_FONTDATA->m_underlined = underlined;
+    M_FONTDATA->SetUnderlined( underlined );
 
     RealizeResource();
 }
@@ -650,7 +869,7 @@ int wxFont::GetPointSize() const
 {
     wxCHECK_MSG( M_FONTDATA != NULL , 0, wxT("invalid font") );
 
-    return M_FONTDATA->m_pointSize;
+    return M_FONTDATA->GetPointSize();
 }
 
 wxSize wxFont::GetPixelSize() const
@@ -672,49 +891,49 @@ int wxFont::GetFamily() const
 {
     wxCHECK_MSG( M_FONTDATA != NULL , 0, wxT("invalid font") );
 
-    return M_FONTDATA->m_family;
+    return M_FONTDATA->GetFamily();
 }
 
 int wxFont::GetStyle() const
 {
     wxCHECK_MSG( M_FONTDATA != NULL , 0, wxT("invalid font") );
 
-    return M_FONTDATA->m_style;
+    return M_FONTDATA->GetStyle() ;
 }
 
 int wxFont::GetWeight() const
 {
     wxCHECK_MSG( M_FONTDATA != NULL , 0, wxT("invalid font") );
 
-    return M_FONTDATA->m_weight;
+    return M_FONTDATA->GetWeight();
 }
 
 bool wxFont::GetUnderlined() const
 {
     wxCHECK_MSG( M_FONTDATA != NULL , false, wxT("invalid font") );
 
-    return M_FONTDATA->m_underlined;
+    return M_FONTDATA->GetUnderlined();
 }
 
 wxString wxFont::GetFaceName() const
 {
     wxCHECK_MSG( M_FONTDATA != NULL , wxEmptyString , wxT("invalid font") );
 
-    return M_FONTDATA->m_faceName;
+    return M_FONTDATA->GetFaceName() ;
 }
 
 wxFontEncoding wxFont::GetEncoding() const
 {
     wxCHECK_MSG( M_FONTDATA != NULL , wxFONTENCODING_DEFAULT , wxT("invalid font") );
 
-    return M_FONTDATA->m_encoding;
+    return M_FONTDATA->GetEncoding() ;
 }
 
 bool wxFont::GetNoAntiAliasing() const
 {
     wxCHECK_MSG( M_FONTDATA != NULL , false, wxT("invalid font") );
 
-    return M_FONTDATA->m_noAA;
+    return M_FONTDATA->GetNoAntiAliasing();
 }
 
 #if wxMAC_USE_ATSU_TEXT
@@ -747,13 +966,6 @@ wxUint32 wxFont::MacGetATSUFontID() const
     return M_FONTDATA->m_macATSUFontID;
 }
 
-void * wxFont::MacGetATSUStyle() const
-{
-    wxCHECK_MSG( M_FONTDATA != NULL , NULL, wxT("invalid font") );
-
-    return M_FONTDATA->m_macATSUStyle;
-}
-
 wxUint32 wxFont::MacGetATSUAdditionalQDStyles() const
 {
     wxCHECK_MSG( M_FONTDATA != NULL , 0, wxT("invalid font") );
@@ -769,6 +981,15 @@ wxUint16 wxFont::MacGetThemeFontID() const
 }
 #endif
 
+#if wxMAC_USE_CORE_TEXT || wxMAC_USE_ATSU_TEXT
+void * wxFont::MacGetATSUStyle() const
+{
+    wxCHECK_MSG( M_FONTDATA != NULL , NULL, wxT("invalid font") );
+
+    return M_FONTDATA->m_macATSUStyle;
+}
+#endif
+
 #if wxMAC_USE_CORE_TEXT
 
 const void * wxFont::MacGetCTFont() const
@@ -778,6 +999,13 @@ const void * wxFont::MacGetCTFont() const
     return (CTFontRef)(M_FONTDATA->m_ctFont);
 }
 
+const void * wxFont::MacGetCTFontDescriptor() const
+{
+    wxCHECK_MSG( M_FONTDATA != NULL , 0, wxT("invalid font") );
+
+    return (CTFontDescriptorRef)(M_FONTDATA->m_ctFontDescriptor);
+}
+
 #endif
 
 const wxNativeFontInfo * wxFont::GetNativeFontInfo() const