mifr/subsurface
1
0
Fork 0
mirror of https://github.com/subsurface/subsurface.git synced 2025-01-20 06:45:27 +00:00
Code Issues Projects Releases Packages Wiki Activity

Make 'string_to_grams()' use proper type safe types

Browse source 
Make it use 'weight_t' and hide the "grams" part inside the type.  That
was the whole point of the weight_t type, after all.  Returning a
"double" was always bogus, since we internally always do integer grams
(and the function actually used "rint()" to get all the rounding right
anyway).

As a result, it's now called "string_to_weight()".

Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
This commit is contained in:
Linus Torvalds 2014-01-08 14:24:35 +08:00 • committed by Dirk Hohndel
parent 88fdf1b138
commit 19b982d3df
1 changed files with 7 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
qt-ui/models.cpp
View file

@ -464,13 +464,14 @@ void WeightModel::passInData(const QModelIndex& index, const QVariant& value)
}
}
double string_to_grams(char *str)
weight_t string_to_weight(char *str)
{
char *end;
double value = strtod_flags(str, &end, 0);
QString rest = QString(end).trimmed();
QString local_kg = WeightModel::tr("kg");
QString local_lbs = WeightModel::tr("lbs");
weight_t weight;
if (rest.startsWith("kg") || rest.startsWith(local_kg))
goto kg;
@ -480,9 +481,11 @@ double string_to_grams(char *str)
if (prefs.units.weight == prefs.units.LBS)
goto lbs;
kg:
return rint(value * 1000);
weight.grams = rint(value * 1000);
return weight;
lbs:
return lbs_to_grams(value);
weight.grams = lbs_to_grams(value);
return weight;
}
bool WeightModel::setData(const QModelIndex& index, const QVariant& value, int role)
@ -509,7 +512,7 @@ bool WeightModel::setData(const QModelIndex& index, const QVariant& value, int r
break;
case WEIGHT:
if (CHANGED(data, "", "")) {
ws->weight.grams = string_to_grams(vString.toUtf8().data());
ws->weight = string_to_weight(vString.toUtf8().data());
// now update the ws_info
changed = true;
WSInfoModel *wsim = WSInfoModel::instance();