Combine DBG_UTIL and header version of Stt{End}CrsrMove

Having 2 versions (debug and release) of one method (on top of
that in different places - one implemented in header) which are
mostly the same is a recipe for disaster.

Change-Id: Ia38f900ac076d1c1765fea2c7326147b115e343d

sw/inc/crsrsh.hxx

@@ -447,14 +447,8 @@ public:
      */
     void Combine();
 
-#ifdef DBG_UTIL
     void SttCrsrMove();
     void EndCrsrMove( const bool bIdleEnd = false );
-#else
-    void SttCrsrMove() { ++m_nCrsrMove; StartAction(); }
-    void EndCrsrMove( const bool bIdleEnd = false )
-            { EndAction( bIdleEnd, true ); --m_nCrsrMove; }
-#endif
 
     /*
      * When the focus is lost the selected ranges are not displayed anymore.

sw/source/core/crsr/crsrsh.cxx

@@ -314,22 +314,27 @@ void SwCrsrShell::EndAction( const bool bIdleEnd, const bool DoSetPosX )
     }
 }
 
-#ifdef DBG_UTIL
 void SwCrsrShell::SttCrsrMove()
 {
+#ifdef DBG_UTIL
     OSL_ENSURE( m_nCrsrMove < USHRT_MAX, "To many nested CrsrMoves." );
+#endif
     ++m_nCrsrMove;
     StartAction();
 }
 
 void SwCrsrShell::EndCrsrMove( const bool bIdleEnd )
 {
+#ifdef DBG_UTIL
     OSL_ENSURE( m_nCrsrMove, "EndCrsrMove() without SttCrsrMove()." );
+#endif
     EndAction( bIdleEnd, true );
-    if( !--m_nCrsrMove )
+    --m_nCrsrMove;
+#ifdef DBG_UTIL
+    if( !m_nCrsrMove )
         m_bInCMvVisportChgd = false;
-}
 #endif
+}
 
 bool SwCrsrShell::LeftRight( bool bLeft, sal_uInt16 nCnt, sal_uInt16 nMode,
                              bool bVisualAllowed )