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

Julien Nabet serval2412 at yahoo.fr
Tue Feb 5 22:41:29 PST 2013


 sc/source/core/data/table5.cxx |   10 ----------
 1 file changed, 10 deletions(-)

New commits:
commit 444ae4d04adf6c9cdcd1222ef3b8b2e5bf45db0a
Author: Julien Nabet <serval2412 at yahoo.fr>
Date:   Wed Feb 6 07:38:58 2013 +0100

    Fix 'bSkipBreaks' is assigned a value that is never used
    
    See http://nabble.documentfoundation.org/Cppcheck-bSkipBreaks-is-assigned-a-value-that-is-never-used-sc-td4034724.html
    
    Change-Id: I846d433c49ad6951539ad283514ce84263236bbb

diff --git a/sc/source/core/data/table5.cxx b/sc/source/core/data/table5.cxx
index 143221a..ffbda84 100644
--- a/sc/source/core/data/table5.cxx
+++ b/sc/source/core/data/table5.cxx
@@ -120,7 +120,6 @@ void ScTable::UpdatePageBreaks( const ScRange* pUserArea )
 
     bool bSkipColBreaks = false;
     bool bSkipRowBreaks = false;
-    bool bSkipBreaks = false;
 
     if ( pStyleSet->GetItemState( ATTR_PAGE_SCALETOPAGES, false, &pItem ) == SFX_ITEM_SET )
     {
@@ -138,15 +137,6 @@ void ScTable::UpdatePageBreaks( const ScRange* pUserArea )
             bSkipRowBreaks = true;
     }
 
-    if (!bSkipBreaks && pStyleSet->GetItemState(ATTR_PAGE_SCALETO, false, &pItem) == SFX_ITEM_SET)
-    {
-        const ScPageScaleToItem& rScaleToItem = static_cast<const ScPageScaleToItem&>(
-            pStyleSet->Get(ATTR_PAGE_SCALETO));
-        if (rScaleToItem.GetWidth() > 0 || rScaleToItem.GetHeight() > 0)
-            // when fitting to a fixed width x height, ignore manual breaks.
-            bSkipBreaks = true;
-    }
-
     //--------------------------------------------------------------------------
 
     long nPageSizeX = aPageSizeTwips.Width();


More information about the Libreoffice-commits mailing list