diff options
| author |  Berthold Stoeger <bstoeger@mail.tuwien.ac.at> | 2018-11-23 15:18:59 +0100 |
|---|---|---|
| committer |  Dirk Hohndel <dirk@hohndel.org> | 2019-01-09 20:58:04 -0800 |
| commit | f8327ed51bc2298e2c7ef69a1ef0f82509e8d8d4 (patch) | |
| tree | 932758970f9967a25d7958ad830cd0ab08594a00 /desktop-widgets/mainwindow.cpp | |
| parent | 7923507e76e8d64c6c457664ec59f10c884a9fb0 (diff) | |
| download | subsurface-f8327ed51bc2298e2c7ef69a1ef0f82509e8d8d4.tar.gz | |
Core: move autogroup() into divelist.c
After loading or importing, the caller usually called autogroup()
to autogroup dives if so wished by the user. This has already led
to bugs, when autogroup() was forgotten.
Instead, call autogroup() directly in the process_loaded_dives()
and process_imported_dives() functions. Not only does this prevent
forgetting the call - it also means that autogrouping can be
changed without changing every caller.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to 'desktop-widgets/mainwindow.cpp')
| -rw-r--r-- | desktop-widgets/mainwindow.cpp | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/desktop-widgets/mainwindow.cpp b/desktop-widgets/mainwindow.cpp index d836e8e61..d53d4f97e 100644 --- a/desktop-widgets/mainwindow.cpp +++ b/desktop-widgets/mainwindow.cpp @@ -617,7 +617,6 @@ void MainWindow::on_actionCloudstorageopen_triggered() if (!parse_file(fileNamePtr.data(), &dive_table)) setCurrentFile(fileNamePtr.data()); process_loaded_dives(); - autogroup_dives(); Command::clear(); hideProgressBar(); refreshDisplay(); @@ -1715,7 +1714,6 @@ void MainWindow::importFiles(const QStringList fileNames) parse_file(fileNamePtr.data(), &table); } process_imported_dives(&table, false, false); - autogroup_dives(); Command::clear(); refreshDisplay(); } @@ -1739,7 +1737,6 @@ void MainWindow::loadFiles(const QStringList fileNames) hideProgressBar(); updateRecentFiles(); process_loaded_dives(); - autogroup_dives(); Command::clear(); refreshDisplay(); |