[Libreoffice-commits] core.git: 2 commits - sw/source

[Caolán McNamara (via logerrit)]

 sw/source/uibase/sidebar/A11yCheckIssuesPanel.cxx |    5 ++++-
 sw/source/uibase/sidebar/A11yCheckIssuesPanel.hxx |    1 +
 2 files changed, 5 insertions(+), 1 deletion(-)

New commits:
commit 4b3d3354119b643ec20aaad187d0a6506ea307fb
Author:     Caolán McNamara <caolan.mcnam...@collabora.com>
AuthorDate: Wed May 17 19:58:40 2023 +0100
Commit:     Caolán McNamara <caolan.mcnam...@collabora.com>
CommitDate: Thu May 18 10:38:52 2023 +0200

    cid#1530394 Uncaught exception
    
    and
    
    cid#1530396 Uncaught exception
    
    Change-Id: Ib484a788fc2defd4a337645526f410ee365f7209
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/151933
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caolan.mcnam...@collabora.com>

diff --git a/sw/source/uibase/sidebar/A11yCheckIssuesPanel.cxx 
b/sw/source/uibase/sidebar/A11yCheckIssuesPanel.cxx
index 448d9492b30b..c0c8595408c6 100644
--- a/sw/source/uibase/sidebar/A11yCheckIssuesPanel.cxx
+++ b/sw/source/uibase/sidebar/A11yCheckIssuesPanel.cxx
@@ -97,7 +97,7 @@ A11yCheckIssuesPanel::A11yCheckIssuesPanel(weld::Widget* 
pParent, SfxBindings* p
     populateIssues();
 }
 
-A11yCheckIssuesPanel::~A11yCheckIssuesPanel()
+void A11yCheckIssuesPanel::ImplDestroy()
 {
     // Restore state when this panel is no longer used
     if (!mbAutomaticCheckEnabled)
@@ -111,6 +111,8 @@ A11yCheckIssuesPanel::~A11yCheckIssuesPanel()
     m_xAccessibilityCheckBox.reset();
 }
 
+A11yCheckIssuesPanel::~A11yCheckIssuesPanel() { 
suppress_fun_call_w_exception(ImplDestroy()); }
+
 void A11yCheckIssuesPanel::populateIssues()
 {
     if (!mpDoc)
diff --git a/sw/source/uibase/sidebar/A11yCheckIssuesPanel.hxx 
b/sw/source/uibase/sidebar/A11yCheckIssuesPanel.hxx
index b3e33c049528..e13760135c25 100644
--- a/sw/source/uibase/sidebar/A11yCheckIssuesPanel.hxx
+++ b/sw/source/uibase/sidebar/A11yCheckIssuesPanel.hxx
@@ -56,6 +56,7 @@ public:
                                  boost::property_tree::ptree& /*rState*/) 
override{};
 
     A11yCheckIssuesPanel(weld::Widget* pParent, SfxBindings* pBindings);
+    void ImplDestroy();
     virtual ~A11yCheckIssuesPanel() override;
 
 private:
commit 2d5bad1d0904ab0d5a205b15cd109caf8eba6717
Author:     Caolán McNamara <caolan.mcnam...@collabora.com>
AuthorDate: Wed May 17 19:56:44 2023 +0100
Commit:     Caolán McNamara <caolan.mcnam...@collabora.com>
CommitDate: Thu May 18 10:38:45 2023 +0200

    cid#1524496 Uninitialized scalar field
    
    Change-Id: I63a201d6f5c51fa2426c44fb63d1e593d31a895b
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/151932
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caolan.mcnam...@collabora.com>

diff --git a/sw/source/uibase/sidebar/A11yCheckIssuesPanel.cxx 
b/sw/source/uibase/sidebar/A11yCheckIssuesPanel.cxx
index 854f7f151230..448d9492b30b 100644
--- a/sw/source/uibase/sidebar/A11yCheckIssuesPanel.cxx
+++ b/sw/source/uibase/sidebar/A11yCheckIssuesPanel.cxx
@@ -74,6 +74,7 @@ A11yCheckIssuesPanel::A11yCheckIssuesPanel(weld::Widget* 
pParent, SfxBindings* p
     , mpDoc(nullptr)
     , maA11yCheckController(FN_STAT_ACCESSIBILITY_CHECK, *pBindings, *this)
     , mnIssueCount(0)
+    , mbAutomaticCheckEnabled(false)
 {
     SwDocShell* pDocSh = dynamic_cast<SwDocShell*>(SfxObjectShell::Current());
     if (!pDocSh)