diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-05-02 10:00:39 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-05-02 12:37:29 +0200 |
| commit | 03a4b6eb58b29c0ab4bb85f756923c46618d02ff (patch) | |
| tree | e56c5f37b1f501fc29278bb6c397e92f1af98122 | |
| parent | 3f9ffa26f90d7976840c192ef267fe9841759e57 (diff) | |
tdf#117205 impress crash when moving an item in a list
regression from
commit 891e41fac81fbd8d5cdb277b26639abfd25a7143 (patch)
dead code in AccessibleTextHelper_Impl::UpdateVisibleChildren
Change-Id: I0db104fcde91597dc71b6cae6c547a0cedac4af2
Reviewed-on: https://gerrit.libreoffice.org/53721
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
| -rw-r--r-- | svx/source/accessibility/AccessibleTextHelper.cxx | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/svx/source/accessibility/AccessibleTextHelper.cxx b/svx/source/accessibility/AccessibleTextHelper.cxx index 9f1061bf0cac..2707d768c273 100644 --- a/svx/source/accessibility/AccessibleTextHelper.cxx +++ b/svx/source/accessibility/AccessibleTextHelper.cxx @@ -759,12 +759,15 @@ namespace accessibility SvxTextForwarder& rCacheTF = GetTextForwarder(); sal_Int32 nParas=rCacheTF.GetParagraphCount(); - mnFirstVisibleChild = nParas ? 0 : -1; + mnFirstVisibleChild = -1; mnLastVisibleChild = -2; - if (mxFrontEnd.is() && bBroadcastEvents) + for( sal_Int32 nCurrPara=0; nCurrPara<nParas; ++nCurrPara ) { - for( sal_Int32 nCurrPara=0; nCurrPara<nParas; ++nCurrPara ) + if (nCurrPara == 0) + mnFirstVisibleChild = nCurrPara; + mnLastVisibleChild = nCurrPara; + if (mxFrontEnd.is() && bBroadcastEvents) { // child not yet created? ::accessibility::AccessibleParaManager::WeakChild aChild( maParaManager.GetChild(nCurrPara) ); @@ -777,8 +780,6 @@ namespace accessibility } } } - - mnLastVisibleChild = nParas - 1; } catch( const uno::Exception& ) { |