[Libreoffice-commits] core.git: sw/source

Caolán McNamara caolanm at redhat.com
Sat Apr 7 15:47:47 UTC 2018


 sw/source/uibase/shells/tabsh.cxx |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit defad92cdc44dcfef4092ae7324f241d35855e49
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Thu Apr 5 08:37:48 2018 +0100

    coverity#1433789 Unintended sign extension
    
    Change-Id: Idad795503eb95e6fa77e620edc46140da554e61b
    Reviewed-on: https://gerrit.libreoffice.org/52443
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/sw/source/uibase/shells/tabsh.cxx b/sw/source/uibase/shells/tabsh.cxx
index 20e225f61ef5..993203be5690 100644
--- a/sw/source/uibase/shells/tabsh.cxx
+++ b/sw/source/uibase/shells/tabsh.cxx
@@ -225,9 +225,9 @@ static SwTableRep*  lcl_TableParamToItemSet( SfxItemSet& rSet, SwWrtShell &rSh )
     pRep->SetSpace(aCols.GetRightMax());
 
     sal_uInt16 nPercent = 0;
-    long nWidth = ::GetTableWidth(pFormat, aCols, &nPercent, &rSh );
+    auto nWidth = ::GetTableWidth(pFormat, aCols, &nPercent, &rSh );
     // The table width is wrong for relative values.
-    if(nPercent)
+    if (nPercent)
         nWidth = pRep->GetSpace() * nPercent / 100;
     const sal_uInt16 nAlign = pFormat->GetHoriOrient().GetHoriOrient();
     pRep->SetAlign(nAlign);


More information about the Libreoffice-commits mailing list