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

[Libreoffice Gerrit user]

 sc/source/ui/view/printfun.cxx |    5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

New commits:
commit 17d6c6975baa7340f850c72755bc9e8d09357375
Author:     Mike Kaganski <mike.kaganski at collabora.com>
AuthorDate: Mon Nov 5 13:11:16 2018 +0300
Commit:     Mike Kaganski <mike.kaganski at collabora.com>
CommitDate: Mon Nov 5 13:25:46 2018 +0100

    tdf#120703 PVS: V547 Expression is always true
    
    Change-Id: I7a863499fdae27afa8a0866fac5cfe227af8aa99
    Reviewed-on: https://gerrit.libreoffice.org/62883
    Tested-by: Jenkins
    Reviewed-by: Mike Kaganski <mike.kaganski at collabora.com>

diff --git a/sc/source/ui/view/printfun.cxx b/sc/source/ui/view/printfun.cxx
index f1a7d957a9f0..42cb2fb3f90a 100644
--- a/sc/source/ui/view/printfun.cxx
+++ b/sc/source/ui/view/printfun.cxx
@@ -669,9 +669,8 @@ static void lcl_FillHFParam( ScPrintHFParam& rParam, const SfxItemSet* pHFSet )
 //   now back in the dialog:
 //      rParam.nHeight += rParam.nDistance;             // not in the dialog any more ???
 
-        if (rParam.pBorder)
-            rParam.nHeight += lcl_LineTotal( rParam.pBorder->GetTop() ) +
-                              lcl_LineTotal( rParam.pBorder->GetBottom() );
+        rParam.nHeight += lcl_LineTotal( rParam.pBorder->GetTop() ) +
+                          lcl_LineTotal( rParam.pBorder->GetBottom() );
 
         rParam.nManHeight = rParam.nHeight;
     }