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 /vcl/source | |
| 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 'vcl/source')
| -rw-r--r-- | vcl/source/fontsubset/sft.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/vcl/source/fontsubset/sft.cxx b/vcl/source/fontsubset/sft.cxx index 9fcf26f97d9c..bc4db89582d0 100644 --- a/vcl/source/fontsubset/sft.cxx +++ b/vcl/source/fontsubset/sft.cxx @@ -2234,7 +2234,7 @@ append(std::bitset<N> & rSet, size_t const nOffset, sal_uInt32 const nValue) { for (size_t i = 0; i < 32; ++i) { - rSet.set(nOffset + i, (nValue & (1 << i)) != 0); + rSet.set(nOffset + i, (nValue & (1U << i)) != 0); } } |