ignore clicks on a toolbar but outside any button

[wxWidgets.git] / src / msw / font.cpp

diff --git a/src/msw/font.cpp b/src/msw/font.cpp
index cd97c822e59c35a7cc60ebb13d49222f029eb3bd..16993de87863b1204a6446b825911d04b69749ae 100644 (file)
--- a/src/msw/font.cpp
+++ b/src/msw/font.cpp
@@ -1,12 +1,12 @@
 /////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////

-// Name:        font.cpp
+// Name:        src/msw/font.cpp

 // Purpose:     wxFont class
 // Author:      Julian Smart
 // Modified by:
 // Created:     01/02/97
 // RCS-ID:      $Id$
 // Purpose:     wxFont class
 // Author:      Julian Smart
 // Modified by:
 // Created:     01/02/97
 // RCS-ID:      $Id$

-// Copyright:   (c) Julian Smart and Markus Holzem
-// Licence:       wxWindows licence
+// Copyright:   (c) wxWindows team
+// Licence:     wxWindows licence

 /////////////////////////////////////////////////////////////////////////////
 
 // ============================================================================
 /////////////////////////////////////////////////////////////////////////////
 
 // ============================================================================


@@ -37,10 +37,12 @@
     #include "wx/log.h"
 #endif // WX_PRECOMP
 
     #include "wx/log.h"
 #endif // WX_PRECOMP
 

+#include "wx/msw/private.h"
+

 #include "wx/fontutil.h"
 #include "wx/fontutil.h"

-#include "wx/tokenzr.h"
+#include "wx/fontmap.h"

 
 

-#include "wx/msw/private.h"
+#include "wx/tokenzr.h"

 
 IMPLEMENT_DYNAMIC_CLASS(wxFont, wxGDIObject)
 
 
 IMPLEMENT_DYNAMIC_CLASS(wxFont, wxGDIObject)
 


@@ -48,8 +50,8 @@ IMPLEMENT_DYNAMIC_CLASS(wxFont, wxGDIObject)
 // constants
 // ----------------------------------------------------------------------------
 
 // constants
 // ----------------------------------------------------------------------------
 

-// the default font size in points
-static const int wxDEFAULT_FONT_SIZE = 12;
+// the mask used to extract the pitch from LOGFONT::lfPitchAndFamily field
+static const int PITCH_MASK = FIXED_PITCH | VARIABLE_PITCH;

 
 // ----------------------------------------------------------------------------
 // wxFontRefData - the internal description of the font
 
 // ----------------------------------------------------------------------------
 // wxFontRefData - the internal description of the font


@@ -57,21 +59,12 @@ static const int wxDEFAULT_FONT_SIZE = 12;
 
 class WXDLLEXPORT wxFontRefData: public wxGDIRefData
 {
 
 class WXDLLEXPORT wxFontRefData: public wxGDIRefData
 {

-friend class WXDLLEXPORT wxFont;
-

 public:
 public:

+    // constructors

     wxFontRefData()
     {
     wxFontRefData()
     {

-        Init(wxDEFAULT_FONT_SIZE, wxDEFAULT, wxNORMAL, wxNORMAL, FALSE,
-             "", wxFONTENCODING_DEFAULT);
-    }
-
-    wxFontRefData(const wxFontRefData& data)
-    {
-        Init(data.m_pointSize, data.m_family, data.m_style, data.m_weight,
-             data.m_underlined, data.m_faceName, data.m_encoding);
-
-        m_fontId = data.m_fontId;
+        Init(-1, wxFONTFAMILY_DEFAULT, wxFONTSTYLE_NORMAL, wxFONTWEIGHT_NORMAL,
+             FALSE, _T(""), wxFONTENCODING_DEFAULT);

     }
 
     wxFontRefData(int size,
     }
 
     wxFontRefData(int size,


@@ -85,13 +78,140 @@ public:
         Init(size, family, style, weight, underlined, faceName, encoding);
     }
 
         Init(size, family, style, weight, underlined, faceName, encoding);
     }
 

-    wxFontRefData(const wxNativeFontInfo& info)
+    wxFontRefData(const wxNativeFontInfo& info, WXHFONT hFont = 0)
+    {
+        Init(info, hFont);
+    }
+
+    wxFontRefData(const wxFontRefData& data)

     {
     {

-        Init(info);
+        if ( data.m_nativeFontInfoOk )
+        {
+            Init(data.m_nativeFontInfo);
+        }
+        else
+        {
+            Init(data.m_pointSize, data.m_family, data.m_style, data.m_weight,
+                 data.m_underlined, data.m_faceName, data.m_encoding);
+        }

     }
 
     virtual ~wxFontRefData();
 
     }
 
     virtual ~wxFontRefData();
 

+    // operations
+    bool Alloc(wxFont *font);
+
+    void Free();
+
+    // all wxFont accessors
+    int GetPointSize() const
+    {
+        return m_nativeFontInfoOk ? m_nativeFontInfo.GetPointSize()
+                                  : m_pointSize;
+    }
+
+    int GetFamily() const
+    {
+        return m_family;
+    }
+
+    int GetStyle() const
+    {
+        return m_nativeFontInfoOk ? m_nativeFontInfo.GetStyle()
+                                  : m_style;
+    }
+
+    int GetWeight() const
+    {
+        return m_nativeFontInfoOk ? m_nativeFontInfo.GetWeight()
+                                  : m_weight;
+    }
+
+    bool GetUnderlined() const
+    {
+        return m_nativeFontInfoOk ? m_nativeFontInfo.GetUnderlined()
+                                  : m_underlined;
+    }
+
+    wxString GetFaceName() const
+    {
+        wxString s;
+        if ( m_nativeFontInfoOk )
+            s = m_nativeFontInfo.GetFaceName();
+        else
+            s = m_faceName;
+
+        return s;
+    }
+
+    wxFontEncoding GetEncoding() const
+    {
+        return m_nativeFontInfoOk ? m_nativeFontInfo.GetEncoding()
+                                  : m_encoding;
+    }
+
+    WXHFONT GetHFONT() const { return m_hFont; }
+
+    // ... and setters
+    void SetPointSize(int pointSize)
+    {
+        if ( m_nativeFontInfoOk )
+            m_nativeFontInfo.SetPointSize(pointSize);
+        else
+            m_pointSize = pointSize;
+    }
+
+    void SetFamily(int family)
+    {
+        m_family = family;
+    }
+
+    void SetStyle(int style)
+    {
+        if ( m_nativeFontInfoOk )
+            m_nativeFontInfo.SetStyle((wxFontStyle)style);
+        else
+            m_style = style;
+    }
+
+    void SetWeight(int weight)
+    {
+        if ( m_nativeFontInfoOk )
+            m_nativeFontInfo.SetWeight((wxFontWeight)weight);
+        else
+            m_weight = weight;
+    }
+
+    void SetFaceName(const wxString& faceName)
+    {
+        if ( m_nativeFontInfoOk )
+            m_nativeFontInfo.SetFaceName(faceName);
+        else
+            m_faceName = faceName;
+    }
+
+    void SetUnderlined(bool underlined)
+    {
+        if ( m_nativeFontInfoOk )
+            m_nativeFontInfo.SetUnderlined(underlined);
+        else
+            m_underlined = underlined;
+    }
+
+    void SetEncoding(wxFontEncoding encoding)
+    {
+        if ( m_nativeFontInfoOk )
+            m_nativeFontInfo.SetEncoding(encoding);
+        else
+            m_encoding = encoding;
+    }
+
+    // native font info tests
+    bool HasNativeFontInfo() const { return m_nativeFontInfoOk; }
+
+    const wxNativeFontInfo& GetNativeFontInfo() const
+        { return m_nativeFontInfo; }
+

 protected:
     // common part of all ctors
     void Init(int size,
 protected:
     // common part of all ctors
     void Init(int size,


@@ -102,13 +222,7 @@ protected:
               const wxString& faceName,
               wxFontEncoding encoding);
 
               const wxString& faceName,
               wxFontEncoding encoding);
 

-    void Init(const wxNativeFontInfo& info);
-
-    // If TRUE, the pointer to the actual font is temporary and SHOULD NOT BE
-    // DELETED by destructor
-    bool          m_temporary;
-
-    int           m_fontId;
+    void Init(const wxNativeFontInfo& info, WXHFONT hFont = 0);

 
     // font characterstics
     int           m_pointSize;
 
     // font characterstics
     int           m_pointSize;


@@ -144,7 +258,7 @@ void wxFontRefData::Init(int pointSize,
                          wxFontEncoding encoding)
 {
     m_style = style;
                          wxFontEncoding encoding)
 {
     m_style = style;

-    m_pointSize = pointSize;
+    m_pointSize = pointSize == -1 ? wxNORMAL_FONT->GetPointSize() : pointSize;

     m_family = family;
     m_style = style;
     m_weight = weight;
     m_family = family;
     m_style = style;
     m_weight = weight;


@@ -152,104 +266,283 @@ void wxFontRefData::Init(int pointSize,
     m_faceName = faceName;
     m_encoding = encoding;
 
     m_faceName = faceName;
     m_encoding = encoding;
 

-    m_fontId = 0;
-    m_temporary = FALSE;
-

     m_hFont = 0;
 
     m_nativeFontInfoOk = FALSE;
 }
 
     m_hFont = 0;
 
     m_nativeFontInfoOk = FALSE;
 }
 

-void wxFontRefData::Init(const wxNativeFontInfo& info)
+void wxFontRefData::Init(const wxNativeFontInfo& info, WXHFONT hFont)

 {
 {

-    // extract family from pitch-and-family
-    int lfFamily = info.lf.lfPitchAndFamily;
-    if ( lfFamily & FIXED_PITCH )
-        lfFamily -= FIXED_PITCH;
-    if ( lfFamily & VARIABLE_PITCH )
-        lfFamily -= VARIABLE_PITCH;
+    // hFont may be zero, or it be passed in case we really want to
+    // use the exact font created in the underlying system
+    // (for example where we can't guarantee conversion from HFONT
+    // to LOGFONT back to HFONT)
+    m_hFont = hFont;
+
+    m_nativeFontInfoOk = TRUE;
+    m_nativeFontInfo = info;
+}
+
+wxFontRefData::~wxFontRefData()
+{
+    Free();
+}
+
+bool wxFontRefData::Alloc(wxFont *font)
+{
+    if ( !m_nativeFontInfoOk )
+    {
+        wxFillLogFont(&m_nativeFontInfo.lf, font);
+        m_nativeFontInfoOk = TRUE;
+    }
+
+    HFONT hfont = ::CreateFontIndirect(&m_nativeFontInfo.lf);
+    if ( !hfont )
+    {
+        wxLogLastError(wxT("CreateFont"));
+
+        return FALSE;
+    }
+
+    m_hFont = (WXHFONT)hfont;
+
+    return TRUE;
+}
+
+void wxFontRefData::Free()
+{
+    if ( m_hFont )
+    {
+        if ( !::DeleteObject((HFONT) m_hFont) )
+        {
+            wxLogLastError(wxT("DeleteObject(font)"));
+        }
+
+        m_hFont = 0;
+    }
+}
+
+// ----------------------------------------------------------------------------
+// wxNativeFontInfo
+// ----------------------------------------------------------------------------
+
+void wxNativeFontInfo::Init()
+{
+    wxZeroMemory(lf);
+}

 
 

-    switch ( lfFamily )
+int wxNativeFontInfo::GetPointSize() const
+{
+    // FIXME: using the screen here results in incorrect font size calculation
+    //        for printing!
+    const int ppInch = ::GetDeviceCaps(ScreenHDC(), LOGPIXELSY);
+
+    return (int) (((72.0*(double)abs(lf.lfHeight)) / (double) ppInch) + 0.5);
+}
+
+wxFontStyle wxNativeFontInfo::GetStyle() const
+{
+    return lf.lfItalic ? wxFONTSTYLE_ITALIC : wxFONTSTYLE_NORMAL;
+}
+
+wxFontWeight wxNativeFontInfo::GetWeight() const
+{
+    if ( lf.lfWeight <= 300 )
+        return wxFONTWEIGHT_LIGHT;
+
+    if ( lf.lfWeight >= 600 )
+        return wxFONTWEIGHT_BOLD;
+
+    return wxFONTWEIGHT_NORMAL;
+}
+
+bool wxNativeFontInfo::GetUnderlined() const
+{
+    return lf.lfUnderline != 0;
+}
+
+wxString wxNativeFontInfo::GetFaceName() const
+{
+    return lf.lfFaceName;
+}
+
+wxFontFamily wxNativeFontInfo::GetFamily() const
+{
+    wxFontFamily family;
+
+    // extract family from pitch-and-family
+    switch ( lf.lfPitchAndFamily & ~PITCH_MASK )

     {
         case FF_ROMAN:
     {
         case FF_ROMAN:

-            m_family = wxROMAN;
+            family = wxFONTFAMILY_ROMAN;

             break;
 
             break;
 

+        default:
+            wxFAIL_MSG( _T("unknown LOGFONT::lfFamily value") );
+            // fall through
+

         case FF_SWISS:
         case FF_SWISS:

-            m_family = wxSWISS;
+            family = wxFONTFAMILY_SWISS;

             break;
 
         case FF_SCRIPT:
             break;
 
         case FF_SCRIPT:

-            m_family = wxSCRIPT;
+            family = wxFONTFAMILY_SCRIPT;

             break;
 
         case FF_MODERN:
             break;
 
         case FF_MODERN:

-            m_family = wxMODERN;
+            family = wxFONTFAMILY_MODERN;

             break;
 
         case FF_DECORATIVE:
             break;
 
         case FF_DECORATIVE:

-            m_family = wxDECORATIVE;
+            family = wxFONTFAMILY_DECORATIVE;

             break;
             break;

+    }
+
+    return family;
+}
+
+wxFontEncoding wxNativeFontInfo::GetEncoding() const
+{
+    return wxGetFontEncFromCharSet(lf.lfCharSet);
+}
+
+void wxNativeFontInfo::SetPointSize(int pointsize)
+{
+#if wxFONT_SIZE_COMPATIBILITY
+    // Incorrect, but compatible with old wxWindows behaviour
+    lf.lfHeight = (pointSize*ppInch)/72;
+#else // wxFONT_SIZE_COMPATIBILITY
+    // FIXME: using the screen here results in incorrect font size calculation
+    //        for printing!
+    const int ppInch = ::GetDeviceCaps(ScreenHDC(), LOGPIXELSY);
+
+    lf.lfHeight = -(int)((pointsize*((double)ppInch)/72.0) + 0.5);
+#endif // wxFONT_SIZE_COMPATIBILITY/!wxFONT_SIZE_COMPATIBILITY
+}

 
 

+void wxNativeFontInfo::SetStyle(wxFontStyle style)
+{
+    switch ( style )
+    {

         default:
         default:

-            m_family = wxSWISS;
+            wxFAIL_MSG( _T("unknown font style") );
+            // fall through
+
+        case wxFONTSTYLE_NORMAL:
+            break;
+
+        case wxFONTSTYLE_ITALIC:
+        case wxFONTSTYLE_SLANT:
+            lf.lfItalic = TRUE;
+            break;

     }
     }

+}

 
 

-    // weight and style
-    switch ( info.lf.lfWeight )
+void wxNativeFontInfo::SetWeight(wxFontWeight weight)
+{
+    switch ( weight )

     {
     {

-        case FW_LIGHT:
-            m_weight = wxLIGHT;
+        default:
+            wxFAIL_MSG( _T("unknown font weight") );
+            // fall through
+
+        case wxFONTWEIGHT_NORMAL:
+            lf.lfWeight = FW_NORMAL;

             break;
 
             break;
 

-        default:
-        case FW_NORMAL:
-            m_weight = wxNORMAL;
+        case wxFONTWEIGHT_LIGHT:
+            lf.lfWeight = FW_LIGHT;

             break;
 
             break;
 

-        case FW_BOLD:
-            m_weight = wxBOLD;
+        case wxFONTWEIGHT_BOLD:
+            lf.lfWeight = FW_BOLD;

             break;
     }
             break;
     }

+}

 
 

-    m_style = info.lf.lfItalic ? wxITALIC : wxNORMAL;
+void wxNativeFontInfo::SetUnderlined(bool underlined)
+{
+    lf.lfUnderline = underlined;
+}

 
 

-    m_underlined = info.lf.lfUnderline != 0;
+void wxNativeFontInfo::SetFaceName(wxString facename)
+{
+    wxStrncpy(lf.lfFaceName, facename, WXSIZEOF(lf.lfFaceName));
+}

 
 

-    m_faceName = info.lf.lfFaceName;
+void wxNativeFontInfo::SetFamily(wxFontFamily family)
+{
+    int ff_family;
+    wxString facename;

 
 

-    int height = abs(info.lf.lfHeight);
+    switch ( family )
+    {
+        case wxSCRIPT:
+            ff_family = FF_SCRIPT;
+            facename = _T("Script");
+            break;

 
 

-    // remember that 1pt = 1/72inch
-    const int ppInch = ::GetDeviceCaps(ScreenHDC(), LOGPIXELSY);
-    m_pointSize = (int) (((72.0*((double)height))/(double) ppInch) + 0.5);
+        case wxDECORATIVE:
+            ff_family = FF_DECORATIVE;
+            facename = _T("Old English Text MT");
+            break;

 
 

-    m_encoding = wxGetFontEncFromCharSet(info.lf.lfCharSet);
+        case wxROMAN:
+            ff_family = FF_ROMAN;
+            facename = _T("Times New Roman");
+            break;

 
 

-    m_fontId = 0;
-    m_temporary = FALSE;
+        case wxTELETYPE:
+        case wxMODERN:
+            ff_family = FF_MODERN;
+            facename = _T("Courier New");
+            break;

 
 

-    m_hFont = 0;
+        case wxSWISS:
+            ff_family = FF_SWISS;
+            facename = _T("Arial");
+            break;

 
 

-    m_nativeFontInfoOk = TRUE;
-    m_nativeFontInfo = info;
+        case wxDEFAULT:
+        default:
+            ff_family = FF_SWISS;
+            facename = _T("MS Sans Serif");
+    }
+
+    lf.lfPitchAndFamily = DEFAULT_PITCH | ff_family;
+
+    if ( !wxStrlen(lf.lfFaceName) )
+    {
+        SetFaceName(facename);
+    }

 }
 
 }
 

-wxFontRefData::~wxFontRefData()
+void wxNativeFontInfo::SetEncoding(wxFontEncoding encoding)

 {
 {

-    if ( m_hFont )
+    wxNativeEncodingInfo info;
+    if ( !wxGetNativeFontEncoding(encoding, &info) )

     {
     {

-        if ( !::DeleteObject((HFONT) m_hFont) )
+#if wxUSE_FONTMAP
+        if ( wxTheFontMapper->GetAltForEncoding(encoding, &info) )

         {
         {

-            wxLogLastError(wxT("DeleteObject(font)"));
+            if ( !info.facename.empty() )
+            {
+                // if we have this encoding only in some particular facename, use
+                // the facename - it is better to show the correct characters in a
+                // wrong facename than unreadable text in a correct one
+                SetFaceName(info.facename);
+            }
+        }
+        else
+#endif // wxUSE_FONTMAP
+        {
+            // unsupported encoding, replace with the default
+            info.charset = DEFAULT_CHARSET;

         }
     }
         }
     }

-}
-

 
 

-// ----------------------------------------------------------------------------
-// wxNativeFontInfo
-// ----------------------------------------------------------------------------
+    lf.lfCharSet = info.charset;
+}

 
 bool wxNativeFontInfo::FromString(const wxString& s)
 {
 
 bool wxNativeFontInfo::FromString(const wxString& s)
 {


@@ -257,10 +550,10 @@ bool wxNativeFontInfo::FromString(const wxString& s)
 
     wxStringTokenizer tokenizer(s, _T(";"));
 
 
     wxStringTokenizer tokenizer(s, _T(";"));
 

+    // first the version

     wxString token = tokenizer.GetNextToken();
     wxString token = tokenizer.GetNextToken();

-    //
-    //  Ignore the version for now
-    //
+    if ( token != _T('0') )
+        return FALSE;

 
     token = tokenizer.GetNextToken();
     if ( !token.ToLong(&l) )
 
     token = tokenizer.GetNextToken();
     if ( !token.ToLong(&l) )


@@ -365,15 +658,13 @@ wxString wxNativeFontInfo::ToString() const
 
 void wxFont::Init()
 {
 
 void wxFont::Init()
 {

-    if ( wxTheFontList )
-        wxTheFontList->Append(this);

 }
 
 }
 

-bool wxFont::Create(const wxNativeFontInfo& info)
+bool wxFont::Create(const wxNativeFontInfo& info, WXHFONT hFont)

 {
     UnRef();
 
 {
     UnRef();
 

-    m_refData = new wxFontRefData(info);
+    m_refData = new wxFontRefData(info, hFont);

 
     RealizeResource();
 
 
     RealizeResource();
 


@@ -403,7 +694,9 @@ bool wxFont::Create(int pointSize,
     // wxDEFAULT is a valid value for the font size too so we must treat it
     // specially here (otherwise the size would be 70 == wxDEFAULT value)
     if ( pointSize == wxDEFAULT )
     // wxDEFAULT is a valid value for the font size too so we must treat it
     // specially here (otherwise the size would be 70 == wxDEFAULT value)
     if ( pointSize == wxDEFAULT )

-        pointSize = wxDEFAULT_FONT_SIZE;
+    {
+        pointSize = wxNORMAL_FONT->GetPointSize();
+    }

 
     m_refData = new wxFontRefData(pointSize, family, style, weight,
                                   underlined, faceName, encoding);
 
     m_refData = new wxFontRefData(pointSize, family, style, weight,
                                   underlined, faceName, encoding);


@@ -415,8 +708,6 @@ bool wxFont::Create(int pointSize,
 
 wxFont::~wxFont()
 {
 
 wxFont::~wxFont()
 {

-    if ( wxTheFontList )
-        wxTheFontList->DeleteObject(this);

 }
 
 // ----------------------------------------------------------------------------
 }
 
 // ----------------------------------------------------------------------------


@@ -432,37 +723,18 @@ bool wxFont::RealizeResource()
         return TRUE;
     }
 
         return TRUE;
     }
 

-    if(!M_FONTDATA->m_nativeFontInfoOk)
-    {
-      wxFillLogFont(&M_FONTDATA->m_nativeFontInfo.lf, this);
-      M_FONTDATA->m_nativeFontInfoOk = TRUE;
-    }
-
-    M_FONTDATA->m_hFont = (WXHFONT)::CreateFontIndirect(&M_FONTDATA->m_nativeFontInfo.lf);
-    M_FONTDATA->m_faceName = M_FONTDATA->m_nativeFontInfo.lf.lfFaceName;
-    if ( !M_FONTDATA->m_hFont )
-    {
-        wxLogLastError(wxT("CreateFont"));
-
-        return FALSE;
-    }
-
-    return TRUE;
+    return M_FONTDATA->Alloc(this);

 }
 
 bool wxFont::FreeResource(bool WXUNUSED(force))
 {
     if ( GetResourceHandle() )
     {
 }
 
 bool wxFont::FreeResource(bool WXUNUSED(force))
 {
     if ( GetResourceHandle() )
     {

-        if ( !::DeleteObject((HFONT) M_FONTDATA->m_hFont) )
-        {
-            wxLogLastError(wxT("DeleteObject(font)"));
-        }
-
-        M_FONTDATA->m_hFont = 0;
+        M_FONTDATA->Free();

 
         return TRUE;
     }
 
         return TRUE;
     }

+

     return FALSE;
 }
 
     return FALSE;
 }
 


@@ -473,15 +745,12 @@ WXHANDLE wxFont::GetResourceHandle()
 
 WXHFONT wxFont::GetHFONT() const
 {
 
 WXHFONT wxFont::GetHFONT() const
 {

-    if ( !M_FONTDATA )
-        return 0;
-    else
-        return (WXHANDLE)M_FONTDATA->m_hFont;
+    return M_FONTDATA ? M_FONTDATA->GetHFONT() : 0;

 }
 
 bool wxFont::IsFree() const
 {
 }
 
 bool wxFont::IsFree() const
 {

-    return (M_FONTDATA && (M_FONTDATA->m_hFont == 0));
+    return M_FONTDATA && (M_FONTDATA->GetHFONT() == 0);

 }
 
 void wxFont::Unshare()
 }
 
 void wxFont::Unshare()


@@ -507,8 +776,7 @@ void wxFont::SetPointSize(int pointSize)
 {
     Unshare();
 
 {
     Unshare();
 

-    M_FONTDATA->m_pointSize = pointSize;
-    M_FONTDATA->m_nativeFontInfoOk = FALSE;
+    M_FONTDATA->SetPointSize(pointSize);

 
     RealizeResource();
 }
 
     RealizeResource();
 }


@@ -517,8 +785,7 @@ void wxFont::SetFamily(int family)
 {
     Unshare();
 
 {
     Unshare();
 

-    M_FONTDATA->m_family = family;
-    M_FONTDATA->m_nativeFontInfoOk = FALSE;
+    M_FONTDATA->SetFamily(family);

 
     RealizeResource();
 }
 
     RealizeResource();
 }


@@ -527,8 +794,7 @@ void wxFont::SetStyle(int style)
 {
     Unshare();
 
 {
     Unshare();
 

-    M_FONTDATA->m_style = style;
-    M_FONTDATA->m_nativeFontInfoOk = FALSE;
+    M_FONTDATA->SetStyle(style);

 
     RealizeResource();
 }
 
     RealizeResource();
 }


@@ -537,8 +803,7 @@ void wxFont::SetWeight(int weight)
 {
     Unshare();
 
 {
     Unshare();
 

-    M_FONTDATA->m_weight = weight;
-    M_FONTDATA->m_nativeFontInfoOk = FALSE;
+    M_FONTDATA->SetWeight(weight);

 
     RealizeResource();
 }
 
     RealizeResource();
 }


@@ -547,8 +812,7 @@ void wxFont::SetFaceName(const wxString& faceName)
 {
     Unshare();
 
 {
     Unshare();
 

-    M_FONTDATA->m_faceName = faceName;
-    M_FONTDATA->m_nativeFontInfoOk = FALSE;
+    M_FONTDATA->SetFaceName(faceName);

 
     RealizeResource();
 }
 
     RealizeResource();
 }


@@ -557,8 +821,7 @@ void wxFont::SetUnderlined(bool underlined)
 {
     Unshare();
 
 {
     Unshare();
 

-    M_FONTDATA->m_underlined = underlined;
-    M_FONTDATA->m_nativeFontInfoOk = FALSE;
+    M_FONTDATA->SetUnderlined(underlined);

 
     RealizeResource();
 }
 
     RealizeResource();
 }


@@ -567,8 +830,7 @@ void wxFont::SetEncoding(wxFontEncoding encoding)
 {
     Unshare();
 
 {
     Unshare();
 

-    M_FONTDATA->m_encoding = encoding;
-    M_FONTDATA->m_nativeFontInfoOk = FALSE;
+    M_FONTDATA->SetEncoding(encoding);

 
     RealizeResource();
 }
 
     RealizeResource();
 }


@@ -579,7 +841,7 @@ void wxFont::SetNativeFontInfo(const wxNativeFontInfo& info)
 
     FreeResource();
 
 
     FreeResource();
 

-    M_FONTDATA->Init(info);
+    *M_FONTDATA = wxFontRefData(info);

 
     RealizeResource();
 }
 
     RealizeResource();
 }


@@ -592,63 +854,71 @@ int wxFont::GetPointSize() const
 {
     wxCHECK_MSG( Ok(), 0, wxT("invalid font") );
 
 {
     wxCHECK_MSG( Ok(), 0, wxT("invalid font") );
 

-    return M_FONTDATA->m_pointSize;
+    return M_FONTDATA->GetPointSize();

 }
 
 int wxFont::GetFamily() const
 {
     wxCHECK_MSG( Ok(), 0, wxT("invalid font") );
 
 }
 
 int wxFont::GetFamily() const
 {
     wxCHECK_MSG( Ok(), 0, wxT("invalid font") );
 

-    return M_FONTDATA->m_family;
-}
-
-int wxFont::GetFontId() const
-{
-    wxCHECK_MSG( Ok(), 0, wxT("invalid font") );
-
-    return M_FONTDATA->m_fontId;
+    return M_FONTDATA->GetFamily();

 }
 
 int wxFont::GetStyle() const
 {
     wxCHECK_MSG( Ok(), 0, wxT("invalid font") );
 
 }
 
 int wxFont::GetStyle() const
 {
     wxCHECK_MSG( Ok(), 0, wxT("invalid font") );
 

-    return M_FONTDATA->m_style;
+    return M_FONTDATA->GetStyle();

 }
 
 int wxFont::GetWeight() const
 {
     wxCHECK_MSG( Ok(), 0, wxT("invalid font") );
 
 }
 
 int wxFont::GetWeight() const
 {
     wxCHECK_MSG( Ok(), 0, wxT("invalid font") );
 

-    return M_FONTDATA->m_weight;
+    return M_FONTDATA->GetWeight();

 }
 
 bool wxFont::GetUnderlined() const
 {
     wxCHECK_MSG( Ok(), FALSE, wxT("invalid font") );
 
 }
 
 bool wxFont::GetUnderlined() const
 {
     wxCHECK_MSG( Ok(), FALSE, wxT("invalid font") );
 

-    return M_FONTDATA->m_underlined;
+    return M_FONTDATA->GetUnderlined();

 }
 
 wxString wxFont::GetFaceName() const
 {
     wxCHECK_MSG( Ok(), wxT(""), wxT("invalid font") );
 
 }
 
 wxString wxFont::GetFaceName() const
 {
     wxCHECK_MSG( Ok(), wxT(""), wxT("invalid font") );
 

-    return M_FONTDATA->m_faceName;
+    return M_FONTDATA->GetFaceName();

 }
 
 wxFontEncoding wxFont::GetEncoding() const
 {
     wxCHECK_MSG( Ok(), wxFONTENCODING_DEFAULT, wxT("invalid font") );
 
 }
 
 wxFontEncoding wxFont::GetEncoding() const
 {
     wxCHECK_MSG( Ok(), wxFONTENCODING_DEFAULT, wxT("invalid font") );
 

-    return M_FONTDATA->m_encoding;
+    return M_FONTDATA->GetEncoding();

 }
 
 wxNativeFontInfo *wxFont::GetNativeFontInfo() const
 {
 }
 
 wxNativeFontInfo *wxFont::GetNativeFontInfo() const
 {

-    if( M_FONTDATA->m_nativeFontInfoOk )
-        return new wxNativeFontInfo(M_FONTDATA->m_nativeFontInfo);
+    if ( M_FONTDATA->HasNativeFontInfo() )
+        return new wxNativeFontInfo(M_FONTDATA->GetNativeFontInfo());

 
     return 0;
 }
 
 
     return 0;
 }
 

+bool wxFont::IsFixedWidth() const
+{
+    if ( M_FONTDATA->HasNativeFontInfo() )
+    {
+        // the two low-order bits specify the pitch of the font, the rest is
+        // family
+        BYTE pitch = M_FONTDATA->GetNativeFontInfo().
+                        lf.lfPitchAndFamily & PITCH_MASK;
+
+        return pitch == FIXED_PITCH;
+    }
+
+    return wxFontBase::IsFixedWidth();
+}
+