mifr/subsurface
1
0
Fork 0
mirror of https://github.com/subsurface/subsurface.git synced 2025-02-19 22:16:15 +00:00
Code Issues Projects Releases Packages Wiki Activity

cleanup: consistently use get_cylinder() accessor

Browse source 
get_cylinder(d, i) is more readable than d->cylinders.cylinders[i].
Moreover, it does bound checking and is more flexible with respect to
changing the core data structures. Most places already used this accessor,
but some still accessed the cylinders directly.

This patch unifies the accesses by consistently switching to get_cylinder().
The affected code is in C++ and accesses the cylinder as reference or
object, whereas the get_cylinder() function is C and returns a pointer.
This results in funky looking "*get_cylinder(d, i)" expressions.
Arguably still better than the original.

Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
This commit is contained in:
Berthold Stoeger 2020-08-20 07:31:04 +02:00 • committed by Dirk Hohndel
parent efc1b4f31a
commit fe3021b88a
4 changed files with 9 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

4
qt-models/cylindermodel.cpp
View file

@ -357,7 +357,7 @@ bool CylindersModel::setData(const QModelIndex &index, const QVariant &value, in
// First, we make a shallow copy of the old cylinder. Then we modify the fields inside that copy.
// At the end, we either place an EditCylinder undo command (EquipmentTab) or copy the cylinder back (planner).
// Yes, this is not ideal, but the pragmatic thing to do for now.
cylinder_t cyl = d->cylinders.cylinders[row];
cylinder_t cyl = *get_cylinder(d, row);
if (index.column() != TYPE && !changed)
return false;
@ -467,7 +467,7 @@ bool CylindersModel::setData(const QModelIndex &index, const QVariant &value, in
// In the planner - simply overwrite the cylinder in the dive with the modified cylinder.
// We have only made a shallow copy, therefore copy the new cylinder first.
cylinder_t copy = clone_cylinder(cyl);
std::swap(copy, d->cylinders.cylinders[row]);
std::swap(copy, *get_cylinder(d, row));
free_cylinder(copy);
dataChanged(index, index);
} else {

2
qt-models/divesummarymodel.cpp
View file

@ -148,7 +148,7 @@ static void calculateDive(struct dive *dive, Stats &stats)
// EAN dive ?
for (int j = 0; j < dive->cylinders.nr; ++j) {
if (dive->cylinders.cylinders[j].gasmix.o2.permille > 210) {
if (get_cylinder(dive, j)->gasmix.o2.permille > 210) {
stats.divesEAN++;
break;
}