diff options
| author |  Berthold Stoeger <bstoeger@mail.tuwien.ac.at> | 2019-07-06 15:34:31 +0200 |
|---|---|---|
| committer |  Dirk Hohndel <dirk@hohndel.org> | 2019-07-06 11:23:35 -0700 |
| commit | ae60fdf815d1febe36b5f43642a28afbaf84362b (patch) | |
| tree | 20e33d82b358bab3fb582969c455e188683c1cf4 /profile-widget/profilewidget2.cpp | |
| parent | 69be1e23f231081e6bad98903c10782faf5104a7 (diff) | |
| download | subsurface-ae60fdf815d1febe36b5f43642a28afbaf84362b.tar.gz | |
Cleanup: call calculate_max_limits_new() in create_plot_info_new()
All callers of create_plot_info_new() called calculate_max_limits_new()
a line before. Thus, simply call the latter in the former.
This allows us to automatically free the plot data in create_plot_info_new().
The old code overwrote the corresponding field with NULL.
As a side-effect, this removes a bogus static variable.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to 'profile-widget/profilewidget2.cpp')
| -rw-r--r-- | profile-widget/profilewidget2.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/profile-widget/profilewidget2.cpp b/profile-widget/profilewidget2.cpp index f377eeb8e..ca6ce3950 100644 --- a/profile-widget/profilewidget2.cpp +++ b/profile-widget/profilewidget2.cpp @@ -636,8 +636,7 @@ void ProfileWidget2::plotDive(const struct dive *d, bool force, bool doClearPict * shown. */ - free_plot_info_data(&plotInfo); - plotInfo = calculate_max_limits_new(&displayed_dive, currentdc); + // create_plot_info_new() automatically frees old plot data #ifndef SUBSURFACE_MOBILE create_plot_info_new(&displayed_dive, currentdc, &plotInfo, !shouldCalculateMaxDepth, &DivePlannerPointsModel::instance()->final_deco_state); #else |