diff options
author | Dr. David Alan Gilbert <dave@treblig.org> | 2021-01-30 17:30:39 +0000 |
---|---|---|
committer | Michael Stahl <michael.stahl@allotropia.de> | 2021-02-01 12:25:35 +0100 |
commit | c3a8fe7b40465134e500d5698b1455d2181aed4f (patch) | |
tree | 40962e4a6a2e7795ec8937b2750dc336e15c33b6 /lotuswordpro | |
parent | e267fe1a3dffc39bf5076dd6413ed3b0e698378c (diff) |
cppcheck: Clean up signed 32 bit shift by 31
cppcheck moans about 1<<n where n can be 31; ubsan also complains;
they're probably right even if it's unlikely to ever be a problem.
Use 1U << 31
Change-Id: Ic83d7d240db4595c2562c1f91116491044d70bab
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/110173
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@allotropia.de>
Diffstat (limited to 'lotuswordpro')
-rw-r--r-- | lotuswordpro/source/filter/explode.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lotuswordpro/source/filter/explode.cxx b/lotuswordpro/source/filter/explode.cxx index 359aab5f36cd..4b4ffa25c6cf 100644 --- a/lotuswordpro/source/filter/explode.cxx +++ b/lotuswordpro/source/filter/explode.cxx @@ -197,7 +197,7 @@ sal_uInt32 Decompression::ReadBits(sal_uInt16 iCount, sal_uInt32 & nBits) m_nBitsLeft -= iCount; /* return need bits, zeroing the bits above that */ - nBits = val & ((1 << iCount) - 1); + nBits = val & ((1U << iCount) - 1); return 0; } |