diff options
| author | Emmanuel Gil Peyrot <linkmauve@linkmauve.fr> | 2018-06-18 16:54:54 +0000 |
|---|---|---|
| committer | Emmanuel Gil Peyrot <linkmauve@linkmauve.fr> | 2018-06-18 18:27:11 +0000 |
| commit | 571f36f1b61d316a2f2ace00fa94ba83ab1ac0a0 (patch) | |
| tree | 95696a57d31908e2d5b5853b4c84e3d53c700db1 /src/ui/widget | |
| parent | Update pdf-parser.cpp (diff) | |
| download | inkscape-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.cpp | 11 | ||||
| -rw-r--r-- | src/ui/widget/ink-select-one-action.cpp | 3 | ||||
| -rw-r--r-- | src/ui/widget/ink-spinscale.cpp | 3 |
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"); |
