diff options
| author | Mike Kaganski <mike.kaganski@collabora.com> | 2018-01-18 18:05:59 +0100 |
|---|---|---|
| committer | Miklos Vajna <vmiklos@collabora.co.uk> | 2018-01-19 10:18:08 +0100 |
| commit | 0f46830130bfd139af35a787133b778b85c96b89 (patch) | |
| tree | a4bcb324a67c4f236ffc66a6c711af6fb6c0241c | |
| parent | a86db7b2b6f7bc8125db5d9c87348d6a9591c872 (diff) | |
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@libreoffice.org>
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
| -rw-r--r-- | sw/source/core/frmedt/fetab.cxx | 3 |
1 files changed, 0 insertions, 3 deletions
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 ); |