ImplRestore() can be private

It is only called from other member functions.

Also, all the calls are inside conditionals that check mpData &&
mpData->mbCurVisible already, so no need to check anew in the function. Use an
assert() instead.

Change-Id: Ibcde0aa43877b64b9b0ae5117cbda1fb44ee023d

1 files changed, 4 insertions, 5 deletions

diff --git a/vcl/source/window/cursor.cxx b/vcl/source/window/cursor.cxx
index b665be75352d..0d893cad2010 100644
--- a/vcl/source/window/cursor.cxx
+++ b/vcl/source/window/cursor.cxx
@@ -141,11 +141,10 @@ void vcl::Cursor::ImplDraw()
 
 void vcl::Cursor::ImplRestore()
 {
-    if ( mpData && mpData->mbCurVisible )
-    {
-        ImplCursorInvert( mpData );
-        mpData->mbCurVisible = false;
-    }
+    assert( mpData && mpData->mbCurVisible );
+
+    ImplCursorInvert( mpData );
+    mpData->mbCurVisible = false;
 }
 
 void vcl::Cursor::ImplDoShow( bool bDrawDirect, bool bRestore )