summaryrefslogtreecommitdiffstats
path: root/src/ui/widget
diff options
context:
space:
mode:
authorEmmanuel Gil Peyrot <linkmauve@linkmauve.fr>2018-06-18 16:54:54 +0000
committerEmmanuel Gil Peyrot <linkmauve@linkmauve.fr>2018-06-18 18:27:11 +0000
commit571f36f1b61d316a2f2ace00fa94ba83ab1ac0a0 (patch)
tree95696a57d31908e2d5b5853b4c84e3d53c700db1 /src/ui/widget
parentUpdate pdf-parser.cpp (diff)
downloadinkscape-571f36f1b61d316a2f2ace00fa94ba83ab1ac0a0.tar.gz
inkscape-571f36f1b61d316a2f2ace00fa94ba83ab1ac0a0.zip
Run clang-tidy’s modernize-pass-by-value pass.
This avoids having to pass variables by reference before copying them when calling a constructor.
Diffstat (limited to 'src/ui/widget')
-rw-r--r--src/ui/widget/color-icc-selector.cpp11
-rw-r--r--src/ui/widget/ink-select-one-action.cpp3
-rw-r--r--src/ui/widget/ink-spinscale.cpp3
3 files changed, 10 insertions, 7 deletions
diff --git a/src/ui/widget/color-icc-selector.cpp b/src/ui/widget/color-icc-selector.cpp
index bd1cfde1b..9e1a0c05b 100644
--- a/src/ui/widget/color-icc-selector.cpp
+++ b/src/ui/widget/color-icc-selector.cpp
@@ -3,6 +3,7 @@
#endif
#include <set>
+#include <utility>
#include <gtkmm/adjustment.h>
#include <glibmm/i18n.h>
@@ -115,9 +116,9 @@ colorspace::Component::Component()
{
}
-colorspace::Component::Component(std::string const &name, std::string const &tip, guint scale)
- : name(name)
- , tip(tip)
+colorspace::Component::Component(std::string name, std::string tip, guint scale)
+ : name(std::move(name))
+ , tip(std::move(tip))
, scale(scale)
{
}
@@ -217,8 +218,8 @@ class ComponentUI {
{
}
- ComponentUI(colorspace::Component const &component)
- : _component(component)
+ ComponentUI(colorspace::Component component)
+ : _component(std::move(component))
, _adj(nullptr)
, _slider(nullptr)
, _btn(nullptr)
diff --git a/src/ui/widget/ink-select-one-action.cpp b/src/ui/widget/ink-select-one-action.cpp
index 9ab4a7f45..b85e9c189 100644
--- a/src/ui/widget/ink-select-one-action.cpp
+++ b/src/ui/widget/ink-select-one-action.cpp
@@ -20,6 +20,7 @@
#include "ink-select-one-action.h"
#include <iostream>
+#include <utility>
#include <gtkmm/toolitem.h>
#include <gtkmm/menuitem.h>
#include <gtkmm/radioaction.h>
@@ -49,7 +50,7 @@ InkSelectOneAction::InkSelectOneAction (const Glib::ustring &name,
_group_label( group_label ),
_tooltip( tooltip ),
_stock_id( stock_id ),
- _store (store),
+ _store (std::move(store)),
_use_radio (true),
_use_label (true),
_use_icon (true),
diff --git a/src/ui/widget/ink-spinscale.cpp b/src/ui/widget/ink-spinscale.cpp
index 87a2bd11c..3e075978b 100644
--- a/src/ui/widget/ink-spinscale.cpp
+++ b/src/ui/widget/ink-spinscale.cpp
@@ -25,6 +25,7 @@
#include <gdk/gdk.h>
#include <iostream>
+#include <utility>
InkScale::InkScale(Glib::RefPtr<Gtk::Adjustment> adjustment, Gtk::SpinButton* spinbutton)
: Glib::ObjectBase("InkScale")
@@ -224,7 +225,7 @@ InkSpinScale::InkSpinScale(double value, double lower,
}
InkSpinScale::InkSpinScale(Glib::RefPtr<Gtk::Adjustment> adjustment)
- : _adjustment(adjustment)
+ : _adjustment(std::move(adjustment))
{
set_name("InkSpinScale");