From 79ccabdc93aacb7d9299963118f305f9ea26110d Mon Sep 17 00:00:00 2001 From: Tim Mayberry Date: Mon, 12 Jun 2017 21:19:25 +1000 Subject: [PATCH] Remove unnecessary temporary variables in MasterRecord::set_state XMLNode::get_property only modifies the argument if the property is found and conversion is successful. --- libs/ardour/slavable_automation_control.cc | 14 +++----------- 1 file changed, 3 insertions(+), 11 deletions(-) diff --git a/libs/ardour/slavable_automation_control.cc b/libs/ardour/slavable_automation_control.cc index adde7cdac3..9c2ce14ae7 100644 --- a/libs/ardour/slavable_automation_control.cc +++ b/libs/ardour/slavable_automation_control.cc @@ -505,17 +505,9 @@ SlavableAutomationControl::slaved () const int SlavableAutomationControl::MasterRecord::set_state (XMLNode const& n, int) { - bool yn; - double v; - if (n.get_property (X_("yn"), yn)) { - _yn = yn; - } - if (n.get_property (X_("val-ctrl"), v)) { - _val_ctrl = v; - } - if (n.get_property (X_("val-master"), v)) { - _val_master = v; - } + n.get_property (X_("yn"), _yn); + n.get_property (X_("val-ctrl"), _val_ctrl); + n.get_property (X_("val-master"), _val_master); return 0; } -- 2.30.2