Better place for coordinate mirroring and removal

[wxWidgets.git] / src / univ / scrarrow.cpp

diff --git a/src/univ/scrarrow.cpp b/src/univ/scrarrow.cpp
index 4a47d3fb00969c673d8f2369b58a48dbd54250dc..76347bfa07ee818d277e0f93b1521cb0539bf6ca 100644 (file)
--- a/src/univ/scrarrow.cpp
+++ b/src/univ/scrarrow.cpp
@@ -45,7 +45,9 @@ struct wxScrollArrowCaptureData
         m_arrowPressed = wxScrollArrows::Arrow_None;
         m_window = NULL;
         m_btnCapture = -1;
         m_arrowPressed = wxScrollArrows::Arrow_None;
         m_window = NULL;
         m_btnCapture = -1;

+#if wxUSE_TIMER

         m_timerScroll = NULL;
         m_timerScroll = NULL;

+#endif // wxUSE_TIMER

     }
 
     ~wxScrollArrowCaptureData()
     }
 
     ~wxScrollArrowCaptureData()


@@ -53,7 +55,9 @@ struct wxScrollArrowCaptureData
         if ( m_window )
             m_window->ReleaseMouse();
 
         if ( m_window )
             m_window->ReleaseMouse();
 

+#if wxUSE_TIMER

         delete m_timerScroll;
         delete m_timerScroll;

+#endif // wxUSE_TIMER

     }
 
     // the arrow being held pressed (may be Arrow_None)
     }
 
     // the arrow being held pressed (may be Arrow_None)


@@ -65,14 +69,18 @@ struct wxScrollArrowCaptureData
     // the window which has captured the mouse
     wxWindow *m_window;
 
     // the window which has captured the mouse
     wxWindow *m_window;
 

+#if wxUSE_TIMER

     // the timer for generating the scroll events
     wxScrollTimer *m_timerScroll;
     // the timer for generating the scroll events
     wxScrollTimer *m_timerScroll;

+#endif

 };
 
 // ----------------------------------------------------------------------------
 // wxScrollArrowTimer: a wxScrollTimer which calls OnArrow()
 // ----------------------------------------------------------------------------
 
 };
 
 // ----------------------------------------------------------------------------
 // wxScrollArrowTimer: a wxScrollTimer which calls OnArrow()
 // ----------------------------------------------------------------------------
 

+#if wxUSE_TIMER
+

 class wxScrollArrowTimer : public wxScrollTimer
 {
 public:
 class wxScrollArrowTimer : public wxScrollTimer
 {
 public:


@@ -95,6 +103,8 @@ protected:
     wxScrollArrows::Arrow m_arrow;
 };
 
     wxScrollArrows::Arrow m_arrow;
 };
 

+#endif // wxUSE_TIMER
+

 // ============================================================================
 // implementation of wxScrollArrows
 // ============================================================================
 // ============================================================================
 // implementation of wxScrollArrows
 // ============================================================================


@@ -166,6 +176,7 @@ bool wxScrollArrows::HandleMouseMove(const wxMouseEvent& event) const
         arrow = m_control->HitTest(event.GetPosition());
     }
 
         arrow = m_control->HitTest(event.GetPosition());
     }
 

+#if wxUSE_TIMER

     if ( m_captureData && m_captureData->m_timerScroll)
     {
         // the mouse is captured, we may want to pause scrolling if it goes
     if ( m_captureData && m_captureData->m_timerScroll)
     {
         // the mouse is captured, we may want to pause scrolling if it goes


@@ -199,6 +210,7 @@ bool wxScrollArrows::HandleMouseMove(const wxMouseEvent& event) const
 
         return false;
     }
 
         return false;
     }

+#endif // wxUSE_TIMER

 
     // reset the wxCONTROL_CURRENT flag for the arrows which don't have the
     // mouse and set it for the one which has
 
     // reset the wxCONTROL_CURRENT flag for the arrows which don't have the
     // mouse and set it for the one which has


@@ -242,22 +254,28 @@ bool wxScrollArrows::HandleMouse(const wxMouseEvent& event) const
             m_captureData->m_window = m_control->GetWindow();
             m_captureData->m_window->CaptureMouse();
 
             m_captureData->m_window = m_control->GetWindow();
             m_captureData->m_window->CaptureMouse();
 

-            // start scrolling                       
+#if wxUSE_TIMER
+            // start scrolling

             wxScrollArrowTimer *tmpTimerScroll =
                 new wxScrollArrowTimer(m_control, arrow);
             wxScrollArrowTimer *tmpTimerScroll =
                 new wxScrollArrowTimer(m_control, arrow);

+#endif // wxUSE_TIMER

 
 

-            // Because in some cases wxScrollArrowTimer can cause 
-            // m_captureData to be destructed we need to test if it 
+            // Because in some cases wxScrollArrowTimer can cause
+            // m_captureData to be destructed we need to test if it

             // is still valid before using.
             if (m_captureData)
             {
             // is still valid before using.
             if (m_captureData)
             {

+#if wxUSE_TIMER

                 m_captureData->m_timerScroll = tmpTimerScroll;
                 m_captureData->m_timerScroll = tmpTimerScroll;

+#endif // wxUSE_TIMER

 
                 m_control->SetArrowFlag(arrow, wxCONTROL_PRESSED, true);
             }
             else
             {
 
                 m_control->SetArrowFlag(arrow, wxCONTROL_PRESSED, true);
             }
             else
             {

+#if wxUSE_TIMER

                 delete tmpTimerScroll;
                 delete tmpTimerScroll;

+#endif // wxUSE_TIMER

             }
         }
         //else: mouse already captured, nothing to do
             }
         }
         //else: mouse already captured, nothing to do