Remove unnecessary temporary variables in MasterRecord::set_state
authorTim Mayberry <mojofunk@gmail.com>
Mon, 12 Jun 2017 11:19:25 +0000 (21:19 +1000)
committerTim Mayberry <mojofunk@gmail.com>
Mon, 12 Jun 2017 11:19:25 +0000 (21:19 +1000)
XMLNode::get_property only modifies the argument if the property is found and
conversion is successful.

libs/ardour/slavable_automation_control.cc

index adde7cdac34bc45653d00a27564f08d6e4878f0d..9c2ce14ae7593b4d2b9756eb4acd3c2ad75b155d 100644 (file)
@@ -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;
 }