diff options
author | Berthold Stoeger <bstoeger@mail.tuwien.ac.at> | 2019-03-24 21:50:01 +0100 |
---|---|---|
committer | Dirk Hohndel <dirk@hohndel.org> | 2019-03-24 21:57:13 -0700 |
commit | 04593e8ec4bac2606dec54605c72a6a49cc83f9b (patch) | |
tree | 030da73608ad9bf1653b5018404b3a5514e6c7bd /desktop-widgets/modeldelegates.cpp | |
parent | 8facd937f86d913d94275abb80bfa4d2b181bbc6 (diff) | |
download | subsurface-04593e8ec4bac2606dec54605c72a6a49cc83f9b.tar.gz |
Cleanup: fix printGPSCoords signature and leaks
The printGPSCoords() function returns a copied C-style string. Since
the owndership is transferred to the caller, the correct return type
is "char *" instead of "const char *".
Thus a number of casts when calling free can be removed.
Moreover a number of callers didn't free the string and thus were
leaking memory. Fix them. Ultimately we might want two versions
of the function: one for QString, one for C-style strings.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to 'desktop-widgets/modeldelegates.cpp')
-rw-r--r-- | desktop-widgets/modeldelegates.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/desktop-widgets/modeldelegates.cpp b/desktop-widgets/modeldelegates.cpp index a237c1306..c6405a433 100644 --- a/desktop-widgets/modeldelegates.cpp +++ b/desktop-widgets/modeldelegates.cpp @@ -484,9 +484,9 @@ void LocationFilterDelegate::paint(QPainter *painter, const QStyleOptionViewItem } if (bottomText.isEmpty()) { - const char *gpsCoords = printGPSCoords(&ds->location); + char *gpsCoords = printGPSCoords(&ds->location); bottomText = QString(gpsCoords); - free( (void*) gpsCoords); + free(gpsCoords); } if (dive_site_has_gps_location(ds) && currentDiveSiteHasGPS) { |