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

Libreoffice Gerrit user logerrit at kemper.freedesktop.org
Tue Dec 25 03:59:39 UTC 2018


 sw/source/filter/ww8/wrtw8nds.cxx |    4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

New commits:
commit 1a4a314132acb2d2980d1bebd4238960b747c99e
Author:     Justin Luth <justin.luth at collabora.com>
AuthorDate: Mon Dec 24 19:58:07 2018 +0300
Commit:     Justin Luth <justin_luth at sil.org>
CommitDate: Tue Dec 25 04:59:19 2018 +0100

    sw filter ww8: code cleanup - value already set
    
    Due to a partial revert, the boolean variable is
    being set to the value it was guaranteed to already be.
    
    Change-Id: Iae0e5461a2cc6552e973bd6b46803dc17167380a
    Reviewed-on: https://gerrit.libreoffice.org/65598
    Tested-by: Jenkins
    Reviewed-by: Justin Luth <justin_luth at sil.org>

diff --git a/sw/source/filter/ww8/wrtw8nds.cxx b/sw/source/filter/ww8/wrtw8nds.cxx
index 7cde0949f748..fffd85ea4c3a 100644
--- a/sw/source/filter/ww8/wrtw8nds.cxx
+++ b/sw/source/filter/ww8/wrtw8nds.cxx
@@ -2992,9 +2992,7 @@ bool MSWordExportBase::NoPageBreakSection( const SfxItemSet* pSet )
 
         if (bNoPageBreak)
         {
-            if (SfxItemState::SET != pSet->GetItemState(RES_BREAK, true, &pI))
-                bNoPageBreak = true;
-            else
+            if (SfxItemState::SET == pSet->GetItemState(RES_BREAK, true, &pI))
             {
                 SvxBreak eBreak = static_cast<const SvxFormatBreakItem*>(pI)->GetBreak();
                 switch (eBreak)


More information about the Libreoffice-commits mailing list