fix wxGTK1 compilation after wxTextEntry DoGetValue() change

[wxWidgets.git] / src / osx / carbon / statbrma.cpp

diff --git a/src/osx/carbon/statbrma.cpp b/src/osx/carbon/statbrma.cpp
index fc136abeb2cce95f70dd454970305001e25d82f2..a1f4cb28f6818e99ef2714e592ef36a3273e943d 100644 (file)
--- a/src/osx/carbon/statbrma.cpp
+++ b/src/osx/carbon/statbrma.cpp
@@ -1,5 +1,5 @@
 ///////////////////////////////////////////////////////////////////////////////
 ///////////////////////////////////////////////////////////////////////////////

-// Name:        src/mac/carbon/statbarma.cpp
+// Name:        src/osx/carbon/statbarma.cpp

 // Purpose:     native implementation of wxStatusBar (optional)
 // Author:      Stefan Csomor
 // Modified by:
 // Purpose:     native implementation of wxStatusBar (optional)
 // Author:      Stefan Csomor
 // Modified by:


@@ -23,6 +23,9 @@
 
 #include "wx/osx/private.h"
 
 
 #include "wx/osx/private.h"
 

+// Margin between the field text and the field rect
+#define wxFIELD_TEXT_MARGIN 2
+

 
 BEGIN_EVENT_TABLE(wxStatusBarMac, wxStatusBarGeneric)
     EVT_PAINT(wxStatusBarMac::OnPaint)
 
 BEGIN_EVENT_TABLE(wxStatusBarMac, wxStatusBarGeneric)
     EVT_PAINT(wxStatusBarMac::OnPaint)


@@ -67,55 +70,60 @@ bool wxStatusBarMac::Create(wxWindow *parent, wxWindowID id,
     return true;
 }
 
     return true;
 }
 

-void wxStatusBarMac::DrawFieldText(wxDC& dc, int i)
+void wxStatusBarMac::DrawFieldText(wxDC& dc, const wxRect& rect, int i, int textHeight)

 {
     int w, h;
     GetSize( &w , &h );
 {
     int w, h;
     GetSize( &w , &h );

-    wxRect rect;
-    GetFieldRect( i, rect );

 
     if ( !MacIsReallyHilited() )
         dc.SetTextForeground( wxColour( 0x80, 0x80, 0x80 ) );
 
     wxString text(GetStatusText( i ));
 
 
     if ( !MacIsReallyHilited() )
         dc.SetTextForeground( wxColour( 0x80, 0x80, 0x80 ) );
 
     wxString text(GetStatusText( i ));
 

-    wxCoord x, y;
-    dc.GetTextExtent(text, &x, &y);
+    /*wxCoord x, y;
+    dc.GetTextExtent(text, &x, &y);    -- seems unused (FM)*/

 
 

-    int leftMargin = 2;
-    int xpos = rect.x + leftMargin + 1;
+    int xpos = rect.x + wxFIELD_TEXT_MARGIN + 1;

     int ypos = 1;
 
     if ( MacGetTopLevelWindow()->GetExtraStyle() & wxFRAME_EX_METAL )
         ypos++;
 
     dc.SetClippingRegion(rect.x, 0, rect.width, h);
     int ypos = 1;
 
     if ( MacGetTopLevelWindow()->GetExtraStyle() & wxFRAME_EX_METAL )
         ypos++;
 
     dc.SetClippingRegion(rect.x, 0, rect.width, h);

-

     dc.DrawText(text, xpos, ypos);
     dc.DrawText(text, xpos, ypos);

-

     dc.DestroyClippingRegion();
 }
 
     dc.DestroyClippingRegion();
 }
 

-void wxStatusBarMac::DrawField(wxDC& dc, int i)
+void wxStatusBarMac::DrawField(wxDC& dc, int i, int textHeight)

 {
 {

-    DrawFieldText(dc, i);
+    wxRect rect;
+    GetFieldRect(i, rect);
+
+    DrawFieldText(dc, rect, i, textHeight);

 }
 
 void wxStatusBarMac::SetStatusText(const wxString& text, int number)
 {
 }
 
 void wxStatusBarMac::SetStatusText(const wxString& text, int number)
 {

-    wxCHECK_RET( (number >= 0) && (number < m_nFields),
+    // NOTE: seems this function is identic to wxStatusBarGeneric::SetStatusText;
+    //       probably can be removed without problems (FM)
+
+    wxCHECK_RET( (number >= 0) && ((size_t)number < m_panes.GetCount()),

         wxT("invalid status bar field index") );
 
         wxT("invalid status bar field index") );
 

-    if ( m_statusStrings[number] == text )
-        return ;
+    if ( GetStatusText(number) == text )
+        return;
+
+    wxStatusBarGeneric::SetStatusText(text, number);

 
 

-    m_statusStrings[number] = text;

     wxRect rect;
     GetFieldRect(number, rect);
     wxRect rect;
     GetFieldRect(number, rect);

+

     int w, h;
     GetSize( &w, &h );
     int w, h;
     GetSize( &w, &h );

+

     rect.y = 0;
     rect.height = h ;
     rect.y = 0;
     rect.height = h ;

+

     Refresh( true, &rect );
     Update();
 }
     Refresh( true, &rect );
     Update();
 }


@@ -162,13 +170,15 @@ void wxStatusBarMac::OnPaint(wxPaintEvent& WXUNUSED(event))
         dc.DrawLine(0, 0, w, 0);
     }
 
         dc.DrawLine(0, 0, w, 0);
     }
 

-    int i;
-    if ( GetFont().Ok() )
+    if ( GetFont().IsOk() )

         dc.SetFont(GetFont());
     dc.SetBackgroundMode(wxTRANSPARENT);
 
         dc.SetFont(GetFont());
     dc.SetBackgroundMode(wxTRANSPARENT);
 

-    for ( i = 0; i < m_nFields; i ++ )
-        DrawField(dc, i);
+    // compute char height only once for all panes:
+    int textHeight = dc.GetCharHeight();
+
+    for ( size_t i = 0; i < m_panes.GetCount(); i ++ )
+        DrawField(dc, i, textHeight);

 }
 
 void wxStatusBarMac::MacHiliteChanged()
 }
 
 void wxStatusBarMac::MacHiliteChanged()