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

Mike Kaganski mike.kaganski at collabora.com
Fri Jan 19 09:18:33 UTC 2018


 sw/source/core/frmedt/fetab.cxx |    3 ---
 1 file changed, 3 deletions(-)

New commits:
commit 0f46830130bfd139af35a787133b778b85c96b89
Author: Mike Kaganski <mike.kaganski at collabora.com>
Date:   Thu Jan 18 18:05:59 2018 +0100

    Remove noop
    
    IsTableNode() is true when m_pTableCursor is not nullptr.
    So, GetCursor() is only called when it is nullptr.
    In turn, GetCursor() only does something if m_pTableCursor is not nullptr.
    
    This is so from the beginning (commit 84a3db80 from 2000 already has it).
    
    Change-Id: I258129fc1bab0e3e5591702af1158c1ec2579d8d
    Reviewed-on: https://gerrit.libreoffice.org/48144
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Miklos Vajna <vmiklos at collabora.co.uk>

diff --git a/sw/source/core/frmedt/fetab.cxx b/sw/source/core/frmedt/fetab.cxx
index 3de36c61ad2d..e5a7728c9f19 100644
--- a/sw/source/core/frmedt/fetab.cxx
+++ b/sw/source/core/frmedt/fetab.cxx
@@ -1215,9 +1215,6 @@ bool SwFEShell::UpdateTableStyleFormatting(SwTableNode *pTableNode,
 
     SwSelBoxes aBoxes;
 
-    if ( !IsTableMode() )       // if cursors are not current
-        GetCursor();
-
     // whole table or only current selection
     if( IsTableMode() )
         ::GetTableSelCrs( *this, aBoxes );


More information about the Libreoffice-commits mailing list