From e2aa85a6fbc039e8b1950f0e4b8484595d0b47eb Mon Sep 17 00:00:00 2001
From: Caolán McNamara <caolanm@redhat.com>
Date: Wed, 30 Oct 2019 11:34:27 +0000
Subject: avoid intermediate vcl::Windows
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Change-Id: Iac6bc83265e007a699a8993b89ac2efaa3739d95
Reviewed-on: https://gerrit.libreoffice.org/81761
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
---
 editeng/inc/edtspell.hxx            |  5 +----
 editeng/source/editeng/edtspell.cxx |  6 +++---
 editeng/source/editeng/impedit4.cxx |  3 ++-
 editeng/source/misc/splwrap.cxx     | 22 +++++++++-------------
 4 files changed, 15 insertions(+), 21 deletions(-)

(limited to 'editeng')

diff --git a/editeng/inc/edtspell.hxx b/editeng/inc/edtspell.hxx
index d21ce140cf8c..51e4df5e0635 100644
--- a/editeng/inc/edtspell.hxx
+++ b/editeng/inc/edtspell.hxx
@@ -50,10 +50,7 @@ protected:
     virtual bool            SpellMore() override;
 
 public:
-    EditSpellWrapper( vcl::Window* pWin,
-            bool bIsStart,
-            EditView* pView );
-
+    EditSpellWrapper(weld::Window* pWin, bool bIsStart, EditView* pView);
 };
 
 /**
diff --git a/editeng/source/editeng/edtspell.cxx b/editeng/source/editeng/edtspell.cxx
index 93bdb213da52..e64b052ba65c 100644
--- a/editeng/source/editeng/edtspell.cxx
+++ b/editeng/source/editeng/edtspell.cxx
@@ -38,9 +38,9 @@ using namespace com::sun::star::beans;
 using namespace com::sun::star::linguistic2;
 
 
-EditSpellWrapper::EditSpellWrapper( vcl::Window* _pWin,
-        bool bIsStart, EditView* pView ) :
-    SvxSpellWrapper( _pWin, bIsStart, false/*bIsAllRight*/ )
+EditSpellWrapper::EditSpellWrapper(weld::Window* pWindow,
+        bool bIsStart, EditView* pView )
+    : SvxSpellWrapper(pWindow, bIsStart, false/*bIsAllRight*/)
 {
     SAL_WARN_IF( !pView, "editeng", "One view has to be abandoned!" );
     // Keep IgnoreList, delete ReplaceList...
diff --git a/editeng/source/editeng/impedit4.cxx b/editeng/source/editeng/impedit4.cxx
index f1cb5072735b..fe130230ee65 100644
--- a/editeng/source/editeng/impedit4.cxx
+++ b/editeng/source/editeng/impedit4.cxx
@@ -1447,7 +1447,8 @@ EESpellState ImpEditEngine::Spell( EditView* pEditView, bool bMultipleDoc )
     else if ( CreateEPaM( aEditDoc.GetStartPaM() ) == pSpellInfo->aSpellStart )
         bIsStart = true;
 
-    std::unique_ptr<EditSpellWrapper> pWrp(new EditSpellWrapper( Application::GetDefDialogParent(),
+    vcl::Window* pParent = Application::GetDefDialogParent();
+    std::unique_ptr<EditSpellWrapper> pWrp(new EditSpellWrapper(pParent ? pParent->GetFrameWeld() : nullptr,
             bIsStart, pEditView ));
     pWrp->SpellDocument();
     pWrp.reset();
diff --git a/editeng/source/misc/splwrap.cxx b/editeng/source/misc/splwrap.cxx
index 979be61afa71..0c9e23f360fc 100644
--- a/editeng/source/misc/splwrap.cxx
+++ b/editeng/source/misc/splwrap.cxx
@@ -45,10 +45,6 @@
 #include <map>
 #include <memory>
 
-#define WAIT_ON() if(pWin != nullptr) { pWin->EnterWait(); }
-
-#define WAIT_OFF() if(pWin != nullptr) { pWin->LeaveWait(); }
-
 using namespace ::com::sun::star;
 using namespace ::com::sun::star::uno;
 using namespace ::com::sun::star::beans;
@@ -139,7 +135,7 @@ SvxSpellWrapper::~SvxSpellWrapper()
  *
  --------------------------------------------------------------------*/
 
-SvxSpellWrapper::SvxSpellWrapper( vcl::Window* pWn,
+SvxSpellWrapper::SvxSpellWrapper( weld::Window* pWn,
     const bool bStart, const bool bIsAllRight ) :
 
     pWin        ( pWn ),
@@ -156,7 +152,7 @@ SvxSpellWrapper::SvxSpellWrapper( vcl::Window* pWn,
 }
 
 
-SvxSpellWrapper::SvxSpellWrapper( vcl::Window* pWn,
+SvxSpellWrapper::SvxSpellWrapper( weld::Window* pWn,
         Reference< XHyphenator > const &xHyphenator,
         const bool bStart, const bool bOther ) :
     pWin        ( pWn ),
@@ -275,7 +271,7 @@ void SvxSpellWrapper::SpellDocument( )
         {
             EditAbstractDialogFactory* pFact = EditAbstractDialogFactory::Create();
             ScopedVclPtr<AbstractHyphenWordDialog> pDlg(pFact->CreateHyphenWordDialog(
-                            pWin ? pWin->GetFrameWeld() : nullptr,
+                            pWin,
                             xHyphWord->getWord(),
                             LanguageTag( xHyphWord->getLocale() ).getLanguageType(),
                             xHyph, this ));
@@ -347,16 +343,16 @@ bool SvxSpellWrapper::SpellNext( )
     else
     {
         // a BODY_area done, ask for the other BODY_area
-        WAIT_OFF();
+        xWait.reset();
 
         const char* pResId = bReverse ? RID_SVXSTR_QUERY_BW_CONTINUE : RID_SVXSTR_QUERY_CONTINUE;
-        std::unique_ptr<weld::MessageDialog> xBox(Application::CreateMessageDialog(pWin ? pWin->GetFrameWeld() : nullptr,
+        std::unique_ptr<weld::MessageDialog> xBox(Application::CreateMessageDialog(pWin,
                                                                  VclMessageType::Question, VclButtonsType::YesNo,
                                                                  EditResId(pResId)));
         if (xBox->run() != RET_YES)
         {
             // sacrifice the other area if necessary ask for special area
-            WAIT_ON();
+            xWait.reset(new weld::WaitObject(pWin));
             bStartDone = bEndDone = true;
             return SpellNext();
         }
@@ -366,7 +362,7 @@ bool SvxSpellWrapper::SpellNext( )
             SpellStart( bStartChk ? SvxSpellArea::BodyStart : SvxSpellArea::BodyEnd );
             bGoOn = true;
         }
-        WAIT_ON();
+        xWait.reset(new weld::WaitObject(pWin));
     }
     return bGoOn;
 }
@@ -419,7 +415,7 @@ bool SvxSpellWrapper::FindSpellError()
 {
     ShowLanguageErrors();
 
-    WAIT_ON();
+    xWait.reset(new weld::WaitObject(pWin));
     bool bSpell = true;
 
     Reference< XDictionary >  xAllRightDic;
@@ -465,7 +461,7 @@ bool SvxSpellWrapper::FindSpellError()
             bSpell = SpellNext();
         }
     }
-    WAIT_OFF();
+    xWait.reset();
     return GetLast().is();
 }
 
-- 
cgit v1.2.3