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

[Caolán McNamara]

 sw/source/core/unocore/swunohelper.cxx |    7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

New commits:
commit 3a6866c2953c17a55536fa434b9fceda670d0685
Author: Caolán McNamara <caol...@redhat.com>
Date:   Wed Apr 29 21:20:43 2015 +0100

    coverity#1296209 Logically dead code (gold)
    
    Change-Id: I68be0901ab0ee37caaa9338b6c499d0086c2c968

diff --git a/sw/source/core/unocore/swunohelper.cxx 
b/sw/source/core/unocore/swunohelper.cxx
index ceb061b..05dd0da 100644
--- a/sw/source/core/unocore/swunohelper.cxx
+++ b/sw/source/core/unocore/swunohelper.cxx
@@ -282,7 +282,8 @@ bool needToMapFillItemsToSvxBrushItemTypes(const 
SfxItemSet& rSet,
     // here different FillStyles can be excluded for export; it will depend on 
the
     // quality these fallbacks can reach. That again is done in 
getSvxBrushItemFromSourceSet,
     // take a look there how the superset of DrawObject FillStyles is mapped 
to SvxBrushItem.
-    switch (pXFillStyleItem->GetValue())
+    const drawing::FillStyle eFill = pXFillStyleItem->GetValue();
+    switch (eFill)
     {
         case drawing::FillStyle_NONE:
             return false; // ignoring some extremely limited XFillColorItem 
eval
@@ -294,13 +295,13 @@ bool needToMapFillItemsToSvxBrushItemTypes(const 
SfxItemSet& rSet,
             {
                 case MID_BACK_COLOR:
                     // Gradient/Hatch always have emulated color
-                    return (drawing::FillStyle_SOLID != nMID)
+                    return (drawing::FillStyle_SOLID != eFill)
                         || SfxItemState::SET == 
rSet.GetItemState(XATTR_FILLCOLOR)
                         || SfxItemState::SET == 
rSet.GetItemState(XATTR_FILLTRANSPARENCE)
                         || SfxItemState::SET == 
rSet.GetItemState(XATTR_FILLFLOATTRANSPARENCE);
                 case MID_BACK_COLOR_R_G_B:
                     // Gradient/Hatch always have emulated color
-                    return (drawing::FillStyle_SOLID != nMID)
+                    return (drawing::FillStyle_SOLID != eFill)
                         || SfxItemState::SET == 
rSet.GetItemState(XATTR_FILLCOLOR);
                 case MID_BACK_COLOR_TRANSPARENCY:
                     return SfxItemState::SET == 
rSet.GetItemState(XATTR_FILLTRANSPARENCE)

_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits