[Libreoffice-commits] core.git: sd/source vcl/source

[Libreoffice Gerrit user]

 sd/source/ui/view/outlview.cxx    |    3 +--
 vcl/source/gdi/pdfwriter_impl.cxx |    3 +--
 2 files changed, 2 insertions(+), 4 deletions(-)

New commits:
commit b5d7f596f26e3e086a6584fe69a8bbcc9c51d0bd
Author:     Mike Kaganski <mike.kaganski at collabora.com>
AuthorDate: Sun Nov 4 21:30:56 2018 +0300
Commit:     Mike Kaganski <mike.kaganski at collabora.com>
CommitDate: Mon Nov 5 07:20:29 2018 +0100

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

diff --git a/sd/source/ui/view/outlview.cxx b/sd/source/ui/view/outlview.cxx
index 544785643416..c2b3576dbfce 100644
--- a/sd/source/ui/view/outlview.cxx
+++ b/sd/source/ui/view/outlview.cxx
@@ -1474,8 +1474,7 @@ void OutlineView::UpdateDocument()
         mrOutlineViewShell.UpdateTitleObject( pPage, pPara );
         mrOutlineViewShell.UpdateOutlineObject( pPage, pPara );
 
-        if( pPara )
-            pPara = GetNextTitle(pPara);
+        pPara = GetNextTitle(pPara);
     }
 }
 
diff --git a/vcl/source/gdi/pdfwriter_impl.cxx b/vcl/source/gdi/pdfwriter_impl.cxx
index b9394607df10..a3b9db4908d0 100644
--- a/vcl/source/gdi/pdfwriter_impl.cxx
+++ b/vcl/source/gdi/pdfwriter_impl.cxx
@@ -6543,8 +6543,7 @@ void PDFWriterImpl::drawLayout( SalLayout& rLayout, const OUString& rText, bool
     if( GetFontInstance()->GetFontFace()->GetWeight() <= WEIGHT_MEDIUM &&
         GetFontInstance()->GetFontSelectPattern().GetWeight() > WEIGHT_MEDIUM )
     {
-        if( ! bPop )
-            aLine.append( "q " );
+        aLine.append("q ");
         bPop = true;
         bABold = true;
     }