loplugin:shouldreturnbool

return value is ignored by call site, so just drop it

Change-Id: Ibba9b6b898d3277a44ccf864ba302d97282bffff
Reviewed-on: https://gerrit.libreoffice.org/62494
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

sw/inc/fesh.hxx

@@ -337,7 +337,7 @@ public:
     /// Set size of draw objects.
     void SetObjRect( const SwRect& rRect );
 
-    long BeginDrag( const Point *pPt, bool bProp );
+    void BeginDrag( const Point *pPt, bool bProp );
     void Drag     ( const Point *pPt, bool bProp );
     void EndDrag  ();
     void BreakDrag();

sw/source/core/frmedt/feshview.cxx

@@ -695,7 +695,7 @@ void SwFEShell::StartCropImage()
     SetDragMode( SdrDragMode::Crop );
 }
 
-long SwFEShell::BeginDrag( const Point* pPt, bool bIsShift)
+void SwFEShell::BeginDrag( const Point* pPt, bool bIsShift)
 {
     SdrView *pView = Imp()->GetDrawView();
     if ( pView && pView->AreObjectsMarked() )
@@ -706,9 +706,7 @@ long SwFEShell::BeginDrag( const Point* pPt, bool bIsShift)
         if (pView->BegDragObj( *pPt, nullptr, pHdl ))
             pView->GetDragMethod()->SetShiftPressed( bIsShift );
         ::FrameNotify( this );
-        return 1;
     }
-    return 0;
 }
 
 void SwFEShell::Drag( const Point *pPt, bool )