loplugin:virtualdead in basctl

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

basctl/source/basicide/baside2.cxx

@@ -759,11 +759,6 @@ void ModulWindow::StoreData()
     GetEditorWindow().SetSourceInBasic();
 }
 
-bool ModulWindow::CanClose()
-{
-    return true;
-}
-
 bool ModulWindow::AllowUndo()
 {
     return GetEditorWindow().CanModify();

basctl/source/basicide/baside2.hxx

@@ -323,7 +323,6 @@ public:
     virtual void    GetState( SfxItemSet& ) override;
     virtual void    StoreData() override;
     virtual void    UpdateData() override;
-    virtual bool    CanClose() override;
     // return number of pages to be printed
     virtual sal_Int32 countPages( Printer* pPrinter ) override;
     // print page

basctl/source/basicide/basides1.cxx

@@ -1380,20 +1380,6 @@ void Shell::Deactivate( bool bMDI )
             if( pXDlgWin->IsModified() )
                 MarkDocumentModified( pXDlgWin->GetDocument() );
         }
-
-        // test CanClose to also test during deactivating the BasicIDE whether
-        // the sourcecode is too large in one of the modules...
-        for (auto const& window : aWindowTable)
-        {
-            BaseWindow* pWin = window.second;
-            if ( /* !pWin->IsSuspended() && */ !pWin->CanClose() )
-            {
-                if ( !m_aCurLibName.isEmpty() && ( pWin->IsDocument( m_aCurDocument ) || pWin->GetLibName() != m_aCurLibName ) )
-                    SetCurLib( ScriptDocument::getApplicationScriptDocument(), OUString(), false );
-                SetCurWindow( pWin, true );
-                break;
-            }
-        }
     }
 }
 

basctl/source/basicide/basidesh.cxx

@@ -397,24 +397,8 @@ bool Shell::PrepareClose( bool bUI )
     }
     else
     {
-        bool bCanClose = true;
+        StoreAllWindowData( false );    // don't write on the disk, that will be done later automatically
-        for (auto const& window : aWindowTable)
+        return true;
-        {
-            BaseWindow* pWin = window.second;
-            if ( !pWin->CanClose() )
-            {
-                if ( !m_aCurLibName.isEmpty() && ( pWin->IsDocument( m_aCurDocument ) || pWin->GetLibName() != m_aCurLibName ) )
-                    SetCurLib( ScriptDocument::getApplicationScriptDocument(), OUString(), false );
-                SetCurWindow( pWin, true );
-                bCanClose = false;
-                break;
-            }
-        }
-
-        if ( bCanClose )
-            StoreAllWindowData( false );    // don't write on the disk, that will be done later automatically
-
-        return bCanClose;
     }
 }
 

basctl/source/basicide/bastypes.cxx

@@ -148,11 +148,6 @@ void BaseWindow::StoreData()
 {
 }
 
-bool BaseWindow::CanClose()
-{
-    return true;
-}
-
 bool BaseWindow::AllowUndo()
 {
     return true;

basctl/source/inc/bastypes.hxx

@@ -188,7 +188,6 @@ public:
 
     virtual void    StoreData();
     virtual void    UpdateData();
-    virtual bool    CanClose();
 
     // return number of pages to be printed
     virtual sal_Int32 countPages( Printer* pPrinter ) = 0;

compilerplugins/clang/virtualdead.results

@@ -1,6 +1,3 @@
-basctl/source/inc/bastypes.hxx:191
-    _Bool basctl::BaseWindow::CanClose()
-    1
 basic/source/comp/codegen.cxx:464
     void OffSetAccumulator::start(const unsigned char *,)
     empty