From 6fd20e011e2f6dec4be9b9bfa041145169274a4f Mon Sep 17 00:00:00 2001 From: Stephan Bergmann Date: Fri, 26 Jan 2018 17:58:01 +0100 Subject: Remove redundant checks ...that date back to a909acb7009acadffa53e74ea05ddb88803490f1 "#98735# Refactored unoedit/unoedacc.cxx", before SvxTextForwarder::GetParaBounds was changed from sal_uInt16 to sal_Int32 with 2af1f5691e8d64afd5246d245d7876b5a2cd5cd8 "resolved fdo#35756 import more than 64k HTML table cells" Change-Id: Ia92dcf1fb6742779e5fd51ed3ba8f0db565f8965 Reviewed-on: https://gerrit.libreoffice.org/48715 Tested-by: Jenkins Reviewed-by: Stephan Bergmann --- svx/source/accessibility/AccessibleTextHelper.cxx | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) (limited to 'svx/source') diff --git a/svx/source/accessibility/AccessibleTextHelper.cxx b/svx/source/accessibility/AccessibleTextHelper.cxx index faadef3fc063..8f59f7ceee0d 100644 --- a/svx/source/accessibility/AccessibleTextHelper.cxx +++ b/svx/source/accessibility/AccessibleTextHelper.cxx @@ -21,7 +21,6 @@ // Global header -#include #include #include #include @@ -782,7 +781,7 @@ namespace accessibility for( nCurrPara=0; nCurrPara= 0 && nCurrPara <= USHRT_MAX, + DBG_ASSERT(nCurrPara >= 0, "AccessibleTextHelper_Impl::UpdateVisibleChildren: index value overflow"); aTmpBB = rCacheTF.GetParaBounds( nCurrPara ); @@ -1577,7 +1576,7 @@ namespace accessibility sal_Int32 nChild; for( nChild=mnFirstVisibleChild; nChild <= mnLastVisibleChild; ++nChild ) { - DBG_ASSERT(nChild >= 0 && nChild <= USHRT_MAX, + DBG_ASSERT(nChild >= 0, "AccessibleTextHelper_Impl::getAccessibleAt: index value overflow"); tools::Rectangle aParaBounds( rCacheTF.GetParaBounds( nChild ) ); -- cgit