From 40a3e562b01de2827fa6c2c5fd9d46631f91dc36 Mon Sep 17 00:00:00 2001 From: Berthold Stoeger Date: Mon, 25 Mar 2019 09:05:47 +0100 Subject: Cleanup: provide printGPSCoords in C and C++ versions printGPSCoords() returned a newly allocated C-style string. Most callers simply made a QString out of it and freed the C-style string. This is paradoxical, as printGPSCoords internally works with QStrings and converts them to C-style on return. Therefore, let printGPSCoords() return a QString and create a printGPSCoordsC() wrapper for the two C-callers. Signed-off-by: Berthold Stoeger --- mobile-widgets/qmlmanager.cpp | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) (limited to 'mobile-widgets') diff --git a/mobile-widgets/qmlmanager.cpp b/mobile-widgets/qmlmanager.cpp index d9da5e562..1e2b6e132 100644 --- a/mobile-widgets/qmlmanager.cpp +++ b/mobile-widgets/qmlmanager.cpp @@ -1604,10 +1604,7 @@ QString QMLManager::getGpsFromSiteName(const QString& siteName) ds = get_dive_site_by_name(qPrintable(siteName)); if (!ds) return QString(); - char *gps = printGPSCoords(&ds->location); - QString res = gps; - free(gps); - return res; + return printGPSCoords(&ds->location); } void QMLManager::setNotificationText(QString text) -- cgit v1.2.3-70-g09d2