sw/source/core/layout/paintfrm.cxx | 2 +-
sw/source/core/txtnode/ndtxt.cxx | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
New commits:
commit 927974d0bb84f3ee4021d3ef73de310585821576
Author: Noel Grandin <noelgran...@gmail.com>
AuthorDate: Sat Oct 12 19:03:04 2024 +0200
Commit: Noel Grandin <noel.gran...@collabora.co.uk>
CommitDate: Sun Oct 13 19:54:51 2024 +0200
cid#1608387 Overflowed constant
the underflow is deliberate and fine, and there is no easy way
to restructure this loop, so just use a signed type.
Change-Id: Ib7b7c2e382c606548d73ecbf237146cf6345a223
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/174854
Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>
Tested-by: Jenkins
diff --git a/sw/source/core/txtnode/ndtxt.cxx b/sw/source/core/txtnode/ndtxt.cxx
index 972cccd9b512..e8fa1b248da4 100644
--- a/sw/source/core/txtnode/ndtxt.cxx
+++ b/sw/source/core/txtnode/ndtxt.cxx
@@ -2800,7 +2800,7 @@ void SwTextNode::EraseText(const SwContentIndex &rIdx,
const sal_Int32 nCount,
// GCAttr(); don't remove all empty ones, just the ones that are in the
// range but not at the end of the range.
- for ( size_t i = 0; m_pSwpHints && i < m_pSwpHints->Count(); ++i )
+ for ( sal_Int32 i = 0; m_pSwpHints && i <
static_cast<sal_Int32>(m_pSwpHints->Count()); ++i )
{
SwTextAttr *pHt = m_pSwpHints->Get(i);
commit 3fd420ec6375c1be646eda347805e71cfd9f501e
Author: Noel Grandin <noelgran...@gmail.com>
AuthorDate: Sat Oct 12 19:01:41 2024 +0200
Commit: Noel Grandin <noel.gran...@collabora.co.uk>
CommitDate: Sun Oct 13 19:54:41 2024 +0200
cid#1607998 Overflowed constant
the underflow here is fine, and there is no easy way to restructure
this code to prevent the underflow without making it very ugly, so
just use a signed type for the loop variable.
Change-Id: I4e37ce6d440b7a78732066d61729504f58e86c6e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/174853
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>
diff --git a/sw/source/core/layout/paintfrm.cxx
b/sw/source/core/layout/paintfrm.cxx
index 5cad611eab7e..460a56767df4 100644
--- a/sw/source/core/layout/paintfrm.cxx
+++ b/sw/source/core/layout/paintfrm.cxx
@@ -1028,7 +1028,7 @@ void SwSubsRects::PaintSubsidiary( OutputDevice *pOut,
SwTaggedPDFHelper aTaggedPDFHelper( nullptr, nullptr, nullptr, *pOut );
// Remove all help line that are almost covered (tables)
- for (size_type i = 0; i != m_aLineRects.size(); ++i)
+ for (sal_Int32 i = 0; i != static_cast<sal_Int32>(m_aLineRects.size());
++i)
{
SwLineRect& rLi = m_aLineRects[i];
const bool bVerticalSubs = rLi.Height() > rLi.Width();
