From 5b71470449ca55955ee96d141af17fd5ba9dd823 Mon Sep 17 00:00:00 2001 From: Tim Mayberry Date: Tue, 30 Aug 2016 22:31:25 +1000 Subject: [PATCH] Use XMLNode::get_property API in AutomationTimeAxis class --- gtk2_ardour/automation_time_axis.cc | 18 ++++++++---------- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/gtk2_ardour/automation_time_axis.cc b/gtk2_ardour/automation_time_axis.cc index 2634b87891..778c57cdba 100644 --- a/gtk2_ardour/automation_time_axis.cc +++ b/gtk2_ardour/automation_time_axis.cc @@ -828,13 +828,13 @@ int AutomationTimeAxisView::set_state_2X (const XMLNode& node, int /*version*/) { if (node.name() == X_("gain") && _parameter == Evoral::Parameter (GainAutomation)) { - XMLProperty const * shown = node.property (X_("shown")); - if (shown) { - bool yn = string_is_affirmative (shown->value ()); - if (yn) { + + bool shown; + if (node.get_property (X_("shown"), shown)) { + if (shown) { _canvas_display->show (); /* FIXME: necessary? show_at? */ + set_gui_property ("visible", shown); } - set_gui_property ("visible", yn); } else { set_gui_property ("visible", false); } @@ -869,11 +869,9 @@ AutomationTimeAxisView::what_has_visible_automation (const boost::shared_ptrextra_xml ("GUI"); if (gui_node) { - XMLProperty const * prop = gui_node->property ("shown"); - if (prop) { - if (string_is_affirmative (prop->value())) { - visible.insert (i->first); - } + bool shown; + if (gui_node->get_property ("shown", shown) && shown) { + visible.insert (i->first); } } }