summaryrefslogtreecommitdiffstats
path: root/packaging/OBS
diff options
context:
space:
mode:
authorGravatar Berthold Stoeger <bstoeger@mail.tuwien.ac.at>2020-08-20 07:31:04 +0200
committerGravatar Dirk Hohndel <dirk@hohndel.org>2020-08-21 08:48:22 -0700
commitfe3021b88a414b6963891a65b738f52bc41e36a8 (patch)
tree390d1dc9e2fb95171a43b14932a82ea1200578c0 /packaging/OBS
parentefc1b4f31ac64d4815d1ab49d9da90dd37c7f3b8 (diff)
downloadsubsurface-fe3021b88a414b6963891a65b738f52bc41e36a8.tar.gz
cleanup: consistently use get_cylinder() accessor
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>
Diffstat (limited to 'packaging/OBS')
0 files changed, 0 insertions, 0 deletions