[Libreoffice-commits] core.git: include/svtools svtools/inc svtools/source

Stephan Bergmann sbergman at redhat.com
Tue Sep 8 07:32:56 PDT 2015


 include/svtools/table/tablerenderer.hxx    |   23 -----------------------
 svtools/inc/table/gridtablerenderer.hxx    |    3 ---
 svtools/source/table/gridtablerenderer.cxx |   12 +++---------
 svtools/source/table/tablecontrol_impl.cxx |    2 +-
 svtools/source/table/tabledatawindow.cxx   |    7 ++-----
 5 files changed, 6 insertions(+), 41 deletions(-)

New commits:
commit f19049065aa3e187377ab73cc70b6faa7803102a
Author: Stephan Bergmann <sbergman at redhat.com>
Date:   Tue Sep 8 16:32:18 2015 +0200

    Remove unused function params
    
    Change-Id: I0db4352ab7a14b4a62bebb650d1956c4a177b67f

diff --git a/include/svtools/table/tablerenderer.hxx b/include/svtools/table/tablerenderer.hxx
index a198465..f34e386 100644
--- a/include/svtools/table/tablerenderer.hxx
+++ b/include/svtools/table/tablerenderer.hxx
@@ -221,22 +221,6 @@ namespace svt { namespace table
 
         /** checks whether a given cell content fits into a given target area on a given device.
 
-            @param i_colPos
-                denotes the column which the cell content would be painted into. Your renderer implementation
-                would only need this parameter if rendering is done differently for different columns.
-
-            @param i_rowPos
-                denotes the row which the cell content would be painted into. Your renderer implementation
-                would only need this parameter if rendering is done differently for different rows.
-
-            @param i_active
-                is <TRUE/> if and only if the renderer should assume the cell content would be painted for the active
-                cell.
-
-            @param i_selected
-                is <TRUE/> if and only if the renderer should assume the cell content would be painted for a selected
-                cell.
-
             @param i_targetDevice
                 denotes the target device for the assumed rendering operation
 
@@ -249,8 +233,6 @@ namespace svt { namespace table
         */
         virtual bool    FitsIntoCell(
                             ::com::sun::star::uno::Any const & i_cellContent,
-                            ColPos const i_colPos, RowPos const i_rowPos,
-                            bool const i_active, bool const i_selected,
                             OutputDevice& i_targetDevice, Rectangle const & i_targetArea
                         ) const = 0;
 
@@ -258,10 +240,6 @@ namespace svt { namespace table
 
             @param i_cellValue
                 the value for which an attempt for a string conversion should be made
-            @param  i_colPos
-                the column position of the cell in question
-            @param  i_rowPos
-                the row position of the cell in question
             @param  o_cellString
                 the cell content, formatted as string
             @return
@@ -269,7 +247,6 @@ namespace svt { namespace table
         */
         virtual bool    GetFormattedCellString(
                             ::com::sun::star::uno::Any const & i_cellValue,
-                            ColPos const i_colPos, RowPos const i_rowPos,
                             OUString & o_cellString
                         ) const = 0;
 
diff --git a/svtools/inc/table/gridtablerenderer.hxx b/svtools/inc/table/gridtablerenderer.hxx
index 4df38f8..e3f14cd 100644
--- a/svtools/inc/table/gridtablerenderer.hxx
+++ b/svtools/inc/table/gridtablerenderer.hxx
@@ -87,13 +87,10 @@ namespace svt { namespace table
         virtual void    HideCellCursor( vcl::Window& _rView, const Rectangle& _rCursorRect) SAL_OVERRIDE;
         virtual bool    FitsIntoCell(
                             ::com::sun::star::uno::Any const & i_cellContent,
-                            ColPos const i_colPos, RowPos const i_rowPos,
-                            bool const i_active, bool const i_selected,
                             OutputDevice& i_targetDevice, Rectangle const & i_targetArea
                         ) const SAL_OVERRIDE;
         virtual bool    GetFormattedCellString(
                             ::com::sun::star::uno::Any const & i_cellValue,
-                            ColPos const i_colPos, RowPos const i_rowPos,
                             OUString & o_cellString
                         ) const SAL_OVERRIDE;
 
diff --git a/svtools/source/table/gridtablerenderer.cxx b/svtools/source/table/gridtablerenderer.cxx
index 0074590..a7cce61 100644
--- a/svtools/source/table/gridtablerenderer.cxx
+++ b/svtools/source/table/gridtablerenderer.cxx
@@ -558,8 +558,8 @@ namespace svt { namespace table
     }
 
 
-    bool GridTableRenderer::FitsIntoCell( Any const & i_cellContent, ColPos const i_colPos, RowPos const i_rowPos,
-        bool const i_active, bool const i_selected, OutputDevice& i_targetDevice, Rectangle const & i_targetArea ) const
+    bool GridTableRenderer::FitsIntoCell( Any const & i_cellContent,
+        OutputDevice& i_targetDevice, Rectangle const & i_targetArea ) const
     {
         if ( !i_cellContent.hasValue() )
             return true;
@@ -593,20 +593,14 @@ namespace svt { namespace table
         if ( nTextWidth > aTargetArea.GetWidth() )
             return false;
 
-        OSL_UNUSED( i_active );
-        OSL_UNUSED( i_selected );
-        OSL_UNUSED( i_rowPos );
-        OSL_UNUSED( i_colPos );
         return true;
     }
 
 
-    bool GridTableRenderer::GetFormattedCellString( Any const & i_cellValue, ColPos const i_colPos, RowPos const i_rowPos, OUString & o_cellString ) const
+    bool GridTableRenderer::GetFormattedCellString( Any const & i_cellValue, OUString & o_cellString ) const
     {
         o_cellString = m_pImpl->aStringConverter.convertToString( i_cellValue );
 
-        OSL_UNUSED( i_colPos );
-        OSL_UNUSED( i_rowPos );
         return true;
     }
 
diff --git a/svtools/source/table/tablecontrol_impl.cxx b/svtools/source/table/tablecontrol_impl.cxx
index 94215e2..7100781 100644
--- a/svtools/source/table/tablecontrol_impl.cxx
+++ b/svtools/source/table/tablecontrol_impl.cxx
@@ -2041,7 +2041,7 @@ namespace svt { namespace table
         m_pModel->getCellContent( i_col, i_row, aCellValue );
 
         OUString sCellStringContent;
-        m_pModel->getRenderer()->GetFormattedCellString( aCellValue, i_col, i_row, sCellStringContent );
+        m_pModel->getRenderer()->GetFormattedCellString( aCellValue, sCellStringContent );
 
         return sCellStringContent;
     }
diff --git a/svtools/source/table/tabledatawindow.cxx b/svtools/source/table/tabledatawindow.cxx
index 044f846..49fc160 100644
--- a/svtools/source/table/tabledatawindow.cxx
+++ b/svtools/source/table/tabledatawindow.cxx
@@ -97,19 +97,16 @@ namespace svt { namespace table
                     pTableModel->getCellContent( hitCol, hitRow, aCellToolTip );
 
                     // use the cell content as tool tip only if it doesn't fit into the cell.
-                    bool const activeCell = ( hitRow == m_rTableControl.getCurrentRow() ) && ( hitCol == m_rTableControl.getCurrentColumn() );
-                    bool const selectedCell = m_rTableControl.isRowSelected( hitRow );
-
                     Rectangle const aWindowRect( Point( 0, 0 ), GetOutputSizePixel() );
                     TableCellGeometry const aCell( m_rTableControl, aWindowRect, hitCol, hitRow );
                     Rectangle const aCellRect( aCell.getRect() );
 
                     PTableRenderer const pRenderer = pTableModel->getRenderer();
-                    if ( pRenderer->FitsIntoCell( aCellToolTip, hitCol, hitRow, activeCell, selectedCell, *this, aCellRect ) )
+                    if ( pRenderer->FitsIntoCell( aCellToolTip, *this, aCellRect ) )
                         aCellToolTip.clear();
                 }
 
-                pTableModel->getRenderer()->GetFormattedCellString( aCellToolTip, hitCol, hitRow, sHelpText );
+                pTableModel->getRenderer()->GetFormattedCellString( aCellToolTip, sHelpText );
 
                 if ( sHelpText.indexOf( '\n' ) >= 0 )
                     nHelpStyle = QuickHelpFlags::TipStyleBalloon;


More information about the Libreoffice-commits mailing list