don't return whatever encoding was set in SetEncoding(); always return wxFONTENCODING_UTF8 instead since that's the real encoding always used by wxFont under wxGTK

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@60158 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

interface/wx/font.h

@@ -111,6 +111,8 @@ enum wxFontFlag
 
 /**
     Font encodings.
+    
+    See wxFont::SetEncoding().
 */
 enum wxFontEncoding
 {
@@ -406,6 +408,15 @@ public:
     */
     static wxFontEncoding GetDefaultEncoding();
     
+    /**
+        Returns the encoding of this font.
+        
+        Note that under wxGTK the returned value is always @c wxFONTENCODING_UTF8.
+        
+        @see SetEncoding()
+    */
+    virtual wxFontEncoding GetEncoding() const;
+    
     /**
         Returns the face name associated with the font, or the empty string if
         there is no face information.
@@ -542,6 +553,16 @@ public:
     */
     static void SetDefaultEncoding(wxFontEncoding encoding);
     
+    /**
+        Sets the encoding for this font.
+        
+        Note that under wxGTK this function has no effect (because the underlying
+        Pango library always uses @c wxFONTENCODING_UTF8).
+        
+        @see GetEncoding()
+    */
+    virtual void SetEncoding(wxFontEncoding encoding);
+
     /**
         Sets the facename for the font.
         Returns @true if the given face name exists; @false otherwise.

src/gtk/font.cpp

@@ -93,7 +93,6 @@ protected:
     void InitFromNative();
 
 private:
-    wxFontEncoding  m_encoding;
     bool            m_underlined;
     bool            m_noAA;      // No anti-aliasing
 
@@ -115,16 +114,12 @@ void wxFontRefData::Init(int pointSize,
                          wxFontWeight weight,
                          bool underlined,
                          const wxString& faceName,
-                         wxFontEncoding encoding)
+                         wxFontEncoding WXUNUSED(encoding))
 {
     if (family == wxFONTFAMILY_DEFAULT)
         family = wxFONTFAMILY_SWISS;
 
     m_underlined = underlined;
-    m_encoding = encoding;
-    if ( m_encoding == wxFONTENCODING_DEFAULT )
-        m_encoding = wxFont::GetDefaultEncoding();
-
     m_noAA = false;
 
     // Create native font info
@@ -162,16 +157,12 @@ void wxFontRefData::InitFromNative()
 
     // Pango description are never underlined
     m_underlined = false;
-
-    // always with GTK+ 2
-    m_encoding = wxFONTENCODING_UTF8;
 }
 
 wxFontRefData::wxFontRefData( const wxFontRefData& data )
              : wxGDIRefData()
 {
     m_underlined = data.m_underlined;
-    m_encoding = data.m_encoding;
     m_noAA = data.m_noAA;
 
     // Forces a copy of the internal data.  wxNativeFontInfo should probably
@@ -267,13 +258,9 @@ bool wxFontRefData::SetFaceName(const wxString& facename)
     return m_nativeFontInfo.SetFaceName(facename);
 }
 
-void wxFontRefData::SetEncoding(wxFontEncoding encoding)
+void wxFontRefData::SetEncoding(wxFontEncoding WXUNUSED(encoding))
 {
-    m_encoding = encoding;
+    // with GTK+ 2 Pango always uses UTF8 internally, we cannot change it
-
-    // the internal Pango encoding is always UTF8; here we save the
-    // encoding just to make it possible to return it from GetEncoding()
-    // FIXME: this seems wrong; shouldn't GetEncoding() always return wxFONTENCODING_UTF8?
 }
 
 void wxFontRefData::SetNativeFontInfo(const wxNativeFontInfo& info)
@@ -386,7 +373,8 @@ wxFontEncoding wxFont::GetEncoding() const
 {
     wxCHECK_MSG( IsOk(), wxFONTENCODING_SYSTEM, wxT("invalid font") );
 
-    return M_FONTDATA->m_encoding;
+    return wxFONTENCODING_UTF8;
+        // Pango always uses UTF8... see also SetEncoding()
 }
 
 bool wxFont::GetNoAntiAliasing() const