diff options
author | Mike Kaganski <mike.kaganski@collabora.com> | 2022-07-21 09:29:32 +0200 |
---|---|---|
committer | Mike Kaganski <mike.kaganski@collabora.com> | 2022-07-21 12:10:56 +0200 |
commit | 9426e90c2c339b16487c48cd885a3e4586e1e1f3 (patch) | |
tree | 0386ac52758058c39a461a2628a5f05fb8963e72 /svtools/inc/mousefunction.hxx | |
parent | f319363b0e07010eea806c723254f17dacdf1b06 (diff) |
Cleanup restoring last position
The checks of valid bottom were introduced in commit
f50ec0bda2a4a6d55217bbc8e82e1a9ab3daaa9f
Author Martin Gallwey <mtg@openoffice.org>
Date Wed May 16 16:08:38 2001 +0000
#86580#, #86953#, #86294#, make ReadUserDataSequence bullet-proof
Possibly initially, bBrowse passed to SwView::ReadUserData was
sometimes false; but at some point it became always true, making
the code using right and bottom (stored in aVis) unreachable. That
became evident after commit db93b92d78714ed193b3cbe52b18e3cfd9da99b8
Author Noel Grandin <noel@peralex.com>
Date Thu Mar 17 11:21:56 2016 +0200
loplugin:constantparam in sfx2
So just drop the unused pieces, that for large documents often resulted
in last position not being restored, because the code exited early at
nBottom > (m_pWrtShell->GetDocSize().Height()+nAdd) check.
Change-Id: If9b4e6faa2754b94cb1298a7221e70736c94f27e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/137186
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Diffstat (limited to 'svtools/inc/mousefunction.hxx')
0 files changed, 0 insertions, 0 deletions