diff options
| author |  Berthold Stoeger <bstoeger@mail.tuwien.ac.at> | 2019-03-19 20:05:39 +0100 |
|---|---|---|
| committer |  Dirk Hohndel <dirk@hohndel.org> | 2019-03-19 16:06:14 -0700 |
| commit | e19b71709de18bbd21e6567ebd1e2e44d6539fa4 (patch) | |
| tree | a1b3bcca450c52e22c5ba312a6cd6c8be2ecf2ff /profile-widget/ruleritem.cpp | |
| parent | dc254716cf3a7e1289f6c72f7b3b90277d18359e (diff) | |
| download | subsurface-e19b71709de18bbd21e6567ebd1e2e44d6539fa4.tar.gz | |
Cleanup: pass const-reference to RulerItem2::setPlotInfo()
Instead of passing a pointer, pass a cons reference. This is more
idiomatic and consistent with RulerNodeItem2::setPlotInfo().
Also make the reference passed to RulerNodeItem2::setPlotInfo()
const, to make clear that the argument is copied.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to 'profile-widget/ruleritem.cpp')
| -rw-r--r-- | profile-widget/ruleritem.cpp | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/profile-widget/ruleritem.cpp b/profile-widget/ruleritem.cpp index c7b82a3c6..f489aa9dd 100644 --- a/profile-widget/ruleritem.cpp +++ b/profile-widget/ruleritem.cpp @@ -26,7 +26,7 @@ RulerNodeItem2::RulerNodeItem2() : setFlag(ItemIgnoresTransformations); } -void RulerNodeItem2::setPlotInfo(plot_info &info) +void RulerNodeItem2::setPlotInfo(const plot_info &info) { pInfo = info; entry = pInfo.entry; @@ -152,11 +152,11 @@ RulerNodeItem2 *RulerItem2::destNode() const return dest; } -void RulerItem2::setPlotInfo(plot_info *info) +void RulerItem2::setPlotInfo(const plot_info &info) { - pInfo = *info; - dest->setPlotInfo(*info); - source->setPlotInfo(*info); + pInfo = info; + dest->setPlotInfo(info); + source->setPlotInfo(info); dest->recalculate(); source->recalculate(); recalculate(); |