aboutsummaryrefslogtreecommitdiffstats
path: root/core/subsurface-qt
diff options
context:
space:
mode:
authorGravatar Berthold Stoeger <bstoeger@mail.tuwien.ac.at>2020-12-15 15:33:39 +0100
committerGravatar Dirk Hohndel <dirk@hohndel.org>2020-12-17 13:03:56 -0800
commitae182c386b0b1b58ab25fc1fbc18c3618062e57f (patch)
tree87c5d94a0f795b969233750b83fb22320ff53389 /core/subsurface-qt
parentd9942269a9b61bc35bd939b23858a7cb4a2515c3 (diff)
downloadsubsurface-ae182c386b0b1b58ab25fc1fbc18c3618062e57f.tar.gz
printing: remove DiveObjectHelper from printing code
At this point (post grantlee), DiveObjectHelper is just pointless glue code. Let's remove it from the printing code and call the formatting functions directly. If necessary, move these functions to core/string-format.cpp. Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to 'core/subsurface-qt')
-rw-r--r--core/subsurface-qt/diveobjecthelper.cpp66
1 files changed, 4 insertions, 62 deletions
diff --git a/core/subsurface-qt/diveobjecthelper.cpp b/core/subsurface-qt/diveobjecthelper.cpp
index 86dda1831..aa60355b2 100644
--- a/core/subsurface-qt/diveobjecthelper.cpp
+++ b/core/subsurface-qt/diveobjecthelper.cpp
@@ -19,64 +19,6 @@
static int callCounter = 0;
#endif /* defined(DEBUG_DOH) */
-
-static QString getFormattedWeight(const struct dive *dive, int idx)
-{
- const weightsystem_t *weight = &dive->weightsystems.weightsystems[idx];
- if (!weight->description)
- return QString();
- QString fmt = QString(weight->description);
- fmt += ", " + get_weight_string(weight->weight, true);
- return fmt;
-}
-
-static QString formatGas(const dive *d)
-{
- /*WARNING: here should be the gastlist, returned
- * from the get_gas_string function or this is correct?
- */
- QString gas, gases;
- for (int i = 0; i < d->cylinders.nr; i++) {
- if (!is_cylinder_used(d, i))
- continue;
- gas = get_cylinder(d, i)->type.description;
- if (!gas.isEmpty())
- gas += QChar(' ');
- gas += gasname(get_cylinder(d, i)->gasmix);
- // if has a description and if such gas is not already present
- if (!gas.isEmpty() && gases.indexOf(gas) == -1) {
- if (!gases.isEmpty())
- gases += QString(" / ");
- gases += gas;
- }
- }
- return gases;
-}
-
-static QString formatWeightList(const dive *d)
-{
- QString weights;
- for (int i = 0; i < d->weightsystems.nr; i++) {
- QString w = getFormattedWeight(d, i);
- if (w.isEmpty())
- continue;
- weights += w + "; ";
- }
- return weights;
-}
-
-static QStringList formatWeights(const dive *d)
-{
- QStringList weights;
- for (int i = 0; i < d->weightsystems.nr; i++) {
- QString w = getFormattedWeight(d, i);
- if (w.isEmpty())
- continue;
- weights << w;
- }
- return weights;
-}
-
QString formatDiveSalinity(const dive *d)
{
int salinity = get_dive_salinity(d);
@@ -105,11 +47,11 @@ DiveObjectHelper::DiveObjectHelper(const struct dive *d) :
surge(d->surge),
chill(d->chill),
timestamp(d->when),
- location(get_dive_location(d) ? QString::fromUtf8(get_dive_location(d)) : QString()),
- gps(d->dive_site ? printGPSCoords(&d->dive_site->location) : QString()),
+ location(get_dive_location(d)),
+ gps(formatDiveGPS(d)),
gps_decimal(format_gps_decimal(d)),
dive_site(QVariant::fromValue(d->dive_site)),
- duration(get_dive_duration_string(d->duration.seconds, gettextFromC::tr("h"), gettextFromC::tr("min"))),
+ duration(formatDiveDuration(d)),
noDive(d->duration.seconds == 0 && d->dc.duration.seconds == 0),
depth(get_depth_string(d->dc.maxdepth.mm, true, true)),
divemaster(d->divemaster ? d->divemaster : QString()),
@@ -127,7 +69,7 @@ DiveObjectHelper::DiveObjectHelper(const struct dive *d) :
cylinders(formatCylinders(d)),
maxcns(d->maxcns),
otu(d->otu),
- sumWeight(get_weight_string(weight_t { total_weight(d) }, true)),
+ sumWeight(formatSumWeight(d)),
getCylinder(formatGetCylinder(d)),
startPressure(formatStartPressure(d)),
endPressure(formatEndPressure(d)),