[Libreoffice-commits] core.git: sw/source
Michael Stahl
mstahl at redhat.com
Thu Mar 12 03:44:59 PDT 2015
sw/source/filter/ww8/ww8par6.cxx | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
New commits:
commit 7d6ce7bad60fe939950655281c117f2329c60dec
Author: Michael Stahl <mstahl at redhat.com>
Date: Thu Mar 12 00:06:35 2015 +0100
sw: PVS-Studio V616 'WW8_TOP' named constant with the value of 0
... is used in the bitwise operation
(regression from de0866b507e259f1726d1015293aad8fcadd763a)
Change-Id: Id2377543df7bce31aadf8ad536ef96bcf818963f
Reviewed-on: https://gerrit.libreoffice.org/14838
Reviewed-by: Caolán McNamara <caolanm at redhat.com>
Tested-by: Caolán McNamara <caolanm at redhat.com>
diff --git a/sw/source/filter/ww8/ww8par6.cxx b/sw/source/filter/ww8/ww8par6.cxx
index 2d63bfa..451fba5 100644
--- a/sw/source/filter/ww8/ww8par6.cxx
+++ b/sw/source/filter/ww8/ww8par6.cxx
@@ -4736,16 +4736,16 @@ void SwWW8ImplReader::Read_Border(sal_uInt16 , const sal_uInt8*, short nLen)
Rectangle aInnerDist;
GetBorderDistance( aBrcs, aInnerDist );
- if ((nBorder & WW8_LEFT)==WW8_LEFT)
+ if (nBorder & (1 << WW8_LEFT))
aBox.SetDistance( (sal_uInt16)aInnerDist.Left(), BOX_LINE_LEFT );
- if ((nBorder & WW8_TOP)==WW8_TOP)
+ if (nBorder & (1 << WW8_TOP))
aBox.SetDistance( (sal_uInt16)aInnerDist.Top(), BOX_LINE_TOP );
- if ((nBorder & WW8_RIGHT)==WW8_RIGHT)
+ if (nBorder & (1 << WW8_RIGHT))
aBox.SetDistance( (sal_uInt16)aInnerDist.Right(), BOX_LINE_RIGHT );
- if ((nBorder & WW8_BOT)==WW8_BOT)
+ if (nBorder & (1 << WW8_BOT))
aBox.SetDistance( (sal_uInt16)aInnerDist.Bottom(), BOX_LINE_BOTTOM );
NewAttr( aBox );
More information about the Libreoffice-commits
mailing list