diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2018-01-28 11:32:13 +0100 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2018-01-29 08:47:34 +0100 |
| commit | 7d9590897526e26a3c0a7a90504741d4c21070f1 (patch) | |
| tree | 4fe993da39be9c7e21526b7b01ce0737e2a16f79 /editeng | |
| parent | f1e910c5ee7e89379391ff260e4164a6d9d21bff (diff) | |
Values need not be bounded by sal_uInt16, it appears
since ESelection's nStart/EndPos have been changed to sal_Int32 with
3cbfcee36dd16e451d76c561eaaaee0ff29b01a5 "xub_StrLen and tools/string.hxx final
straw".
Change-Id: Icc96355ff71d07cac5d6037c81b2523156384077
Reviewed-on: https://gerrit.libreoffice.org/48773
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'editeng')
| -rw-r--r-- | editeng/source/accessibility/AccessibleStaticTextBase.cxx | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/editeng/source/accessibility/AccessibleStaticTextBase.cxx b/editeng/source/accessibility/AccessibleStaticTextBase.cxx index f22752627da3..f44d1482bb6d 100644 --- a/editeng/source/accessibility/AccessibleStaticTextBase.cxx +++ b/editeng/source/accessibility/AccessibleStaticTextBase.cxx @@ -87,9 +87,9 @@ namespace accessibility sal_Int32 nEndPara, sal_Int32 nEndIndex ) { DBG_ASSERT(nStartPara >= 0 && - nStartIndex >= 0 && nStartIndex <= USHRT_MAX && + nStartIndex >= 0 && nEndPara >= 0 && - nEndIndex >= 0 && nEndIndex <= USHRT_MAX , + nEndIndex >= 0, "AccessibleStaticTextBase_Impl::MakeSelection: index value overflow"); return ESelection(nStartPara, nStartIndex, nEndPara, nEndIndex); @@ -301,7 +301,7 @@ namespace accessibility { // check overflow DBG_ASSERT(nCurrPara >= 0 && - nFlatIndex - nCurrIndex + nCurrCount >= 0 && nFlatIndex - nCurrIndex + nCurrCount <= USHRT_MAX , + nFlatIndex - nCurrIndex + nCurrCount >= 0, "AccessibleStaticTextBase_Impl::Index2Internal: index value overflow"); return EPosition(nCurrPara, nFlatIndex - nCurrIndex + nCurrCount); @@ -313,7 +313,7 @@ namespace accessibility { // check overflow DBG_ASSERT(nCurrPara > 0 && - nFlatIndex - nCurrIndex + nCurrCount >= 0 && nFlatIndex - nCurrIndex + nCurrCount <= USHRT_MAX , + nFlatIndex - nCurrIndex + nCurrCount >= 0, "AccessibleStaticTextBase_Impl::Index2Internal: index value overflow"); return EPosition(nCurrPara-1, nFlatIndex - nCurrIndex + nCurrCount); |