allanger/ardour
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Activity

Remove unnecessary temporary variables in MasterRecord::set_state

Browse Source 
XMLNode::get_property only modifies the argument if the property is found and
conversion is successful.
This commit is contained in:
Tim Mayberry
2017-06-12 21:19:25 +10:00
parent 333e20a3a6
commit 79ccabdc93
1 changed files with 3 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
libs/ardour/slavable_automation_control.cc
View 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;
}