Reverted previous 'fixes' now that the array bug has been cured

[wxWidgets.git] / src / generic / caret.cpp

diff --git a/src/generic/caret.cpp b/src/generic/caret.cpp
index 59a9112c2cb600852faf37e8114e294fe2be32d6..d199e5290f94b9f0efd3d9f5a39465fc410cb476 100644 (file)
--- a/src/generic/caret.cpp
+++ b/src/generic/caret.cpp
@@ -1,6 +1,6 @@
 ///////////////////////////////////////////////////////////////////////////////
-// Name:        generic/caret.h
-// Purpose:     generic wxCaret class
+// Name:        generic/caret.cpp
+// Purpose:     generic wxCaret class implementation
 // Author:      Vadim Zeitlin (original code by Robert Roebling)
 // Modified by:
 // Created:     25.05.99
@@ -17,7 +17,7 @@
 // headers
 // ----------------------------------------------------------------------------
 
-#if 0 //def __GNUG__
+#ifdef __GNUG__
     #pragma implementation "caret.h"
 #endif
 
@@ -28,8 +28,12 @@
     #pragma hdrstop
 #endif
 
+#if wxUSE_CARET
+
 #ifndef WX_PRECOMP
-    #include "wx/wx.h"
+    #include "wx/window.h"
+    #include "wx/dcclient.h"
+    #include "wx/dcmemory.h"
 #endif //WX_PRECOMP
 
 #include "wx/caret.h"
@@ -46,9 +50,29 @@ static int gs_blinkTime = 500;  // in milliseconds
 // ============================================================================
 
 // ----------------------------------------------------------------------------
-// wxCaret static functions and data
+// timer stuff
 // ----------------------------------------------------------------------------
 
+wxCaretTimer::wxCaretTimer(wxCaret *caret) 
+{ 
+    m_caret = caret; 
+}
+
+void wxCaretTimer::Notify() 
+{ 
+    m_caret->OnTimer(); 
+}
+
+void wxCaret::OnTimer()
+{
+    // don't blink the caret when we don't have the focus
+    if ( m_hasFocus )
+        Blink();
+}
+
+// ----------------------------------------------------------------------------
+// wxCaret static functions and data
+// ----------------------------------------------------------------------------
 
 int wxCaretBase::GetBlinkTime()
 {
@@ -66,6 +90,12 @@ void wxCaretBase::SetBlinkTime(int milliseconds)
 
 void wxCaret::InitGeneric()
 {
+    m_hasFocus = TRUE;
+    m_blinkedOut = TRUE;
+
+    m_xOld =
+    m_yOld = -1;
+    m_bmpUnderCaret.Create(m_width, m_height);
 }
 
 wxCaret::~wxCaret()
@@ -73,7 +103,8 @@ wxCaret::~wxCaret()
     if ( IsVisible() )
     {
         // stop blinking
-        m_timer.Stop();
+        if ( m_timer.IsRunning() )
+            m_timer.Stop();
     }
 }
 
@@ -83,10 +114,12 @@ wxCaret::~wxCaret()
 
 void wxCaret::DoShow()
 {
-    m_timer.Start(GetBlinkTime());
+    int blinkTime = GetBlinkTime();
+    if ( blinkTime )
+        m_timer.Start(blinkTime);
 
-    m_blinkedOut = TRUE;
-    Blink();
+    if ( m_blinkedOut )
+        Blink();
 }
 
 void wxCaret::DoHide()
@@ -101,11 +134,51 @@ void wxCaret::DoHide()
 
 void wxCaret::DoMove()
 {
-    if ( IsVisible() && !m_blinkedOut )
+    if ( IsVisible() )
     {
+        if ( !m_blinkedOut )
+        {
+            // hide it right now and it will be shown the next time it blinks
+            Blink();
+
+            // but if the caret is not blinking, we should blink it back into
+            // visibility manually
+            if ( !m_timer.IsRunning() )
+                Blink();
+        }
+    }
+    //else: will be shown at the correct location when it is shown
+}
+
+// ----------------------------------------------------------------------------
+// handling the focus
+// ----------------------------------------------------------------------------
+
+void wxCaret::OnSetFocus()
+{
+    m_hasFocus = TRUE;
+
+    if ( IsVisible() )
+        Refresh();
+}
+
+void wxCaret::OnKillFocus()
+{
+    m_hasFocus = FALSE;
+
+    if ( IsVisible() )
+    {
+        // the caret must be shown - otherwise, if it is hidden now, it will
+        // stay so until the focus doesn't return because it won't blink any
+        // more
+
+        // hide it first if it isn't hidden ...
+        if ( !m_blinkedOut )
+            Blink();
+
+        // .. and show it in the new style
         Blink();
     }
-    //else: will be shown at the correct location next time it blinks
 }
 
 // ----------------------------------------------------------------------------
@@ -116,24 +189,61 @@ void wxCaret::Blink()
 {
     m_blinkedOut = !m_blinkedOut;
 
-    wxClientDC dc(GetWindow());
-    if ( !m_blinkedOut )
+    Refresh();
+}
+
+void wxCaret::Refresh()
+{
+    wxClientDC dcWin(GetWindow());
+    wxMemoryDC dcMem;
+    dcMem.SelectObject(m_bmpUnderCaret);
+    if ( m_blinkedOut )
     {
-        DoDraw(&dc);
+        // restore the old image
+        dcWin.Blit(m_xOld, m_yOld, m_width, m_height,
+                   &dcMem, 0, 0);
+        m_xOld =
+        m_yOld = -1;
     }
     else
     {
-        // FIXME can't be less efficient than this... (+1 needed!)
-        wxRect rect(m_x, m_y, m_width + 1, m_height + 1);
-        GetWindow()->Refresh(FALSE, &rect);
+        if ( m_xOld == -1 && m_yOld == -1 )
+        {
+            // save the part we're going to overdraw
+
+            int x = m_x,
+                y = m_y;
+#if defined(__WXGTK__) && !defined(__WX_DC_BLIT_FIXED__)
+            wxPoint pt = dcWin.GetDeviceOrigin();
+            x += pt.x;
+            y += pt.y;
+#endif // broken wxGTK wxDC::Blit
+            dcMem.Blit(0, 0, m_width, m_height,
+                       &dcWin, x, y);
+
+            m_xOld = m_x;
+            m_yOld = m_y;
+        }
+        //else: we already saved the image below the caret, don't do it any
+        //      more
+
+        // and draw the caret there
+        DoDraw(&dcWin);
     }
 }
 
 void wxCaret::DoDraw(wxDC *dc)
 {
     dc->SetPen( *wxBLACK_PEN );
-    dc->DrawLine( m_x, m_y, m_x+m_width, m_y );
-    dc->DrawLine( m_x, m_y+m_height, m_x+m_width, m_y+m_height );
-    dc->DrawLine( m_x+(m_width/2), m_y, m_x+(m_width/2), m_y+m_height );
-//  dc->DrawLine( m_x+(m_width/2)+1, m_y, m_x+(m_width/2)+1, m_y+m_height );
+
+    dc->SetBrush(*(m_hasFocus ? wxBLACK_BRUSH : wxTRANSPARENT_BRUSH));
+    dc->SetPen(*wxBLACK_PEN);
+
+    // VZ: unfortunately, the rectangle comes out a pixel smaller when this is
+    //     done under wxGTK - no idea why
+    //dc->SetLogicalFunction(wxINVERT);
+
+    dc->DrawRectangle(m_x, m_y, m_width, m_height);
 }
+
+#endif // wxUSE_CARET