diff options
| author |  Berthold Stoeger <bstoeger@mail.tuwien.ac.at> | 2019-07-03 20:51:24 +0200 |
|---|---|---|
| committer |  Dirk Hohndel <dirk@hohndel.org> | 2019-07-04 10:40:47 +0900 |
| commit | 57c22d3dcc221caad66f12bb01a278c33dca0ab3 (patch) | |
| tree | 642ab351105c90ba505244a087fea91651c183e0 /profile-widget/profilewidget2.cpp | |
| parent | 5c1446a87acbdd8c8859a381eb89a131d49ce49f (diff) | |
| download | subsurface-57c22d3dcc221caad66f12bb01a278c33dca0ab3.tar.gz | |
Cleanup: avoid spurious updateDiveInfo() calls
In 2e230da3610dd1fc61badaf328a084512895fb90 the dive-selection signals
were unified. Sadly, this was done in a suboptimal way resulting in
numerous calls to updateDiveInfo(), which refreshes the main-tab.
Firstly, the MainWindow connected to selection changes from both,
the undo-command and the divelist. Secondly, every selected dive
in the divelist caused a single signal.
Thus, connect only to the divelist (this is necessary for user-initiated
selection changes) and only send a single signal in the divelist
per selection-reset.
This is still less than perfect as updateDiveInfo() is called even
if the current dive doesn't change.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to 'profile-widget/profilewidget2.cpp')
0 files changed, 0 insertions, 0 deletions