From: Tim Mayberry <mojofunk@gmail.com>
Date: Tue, 30 Aug 2016 12:31:25 +0000 (+1000)
Subject: Use XMLNode::get_property API in AutomationTimeAxis class
X-Git-Tag: 5.9~287
X-Git-Url: https://main.carlh.net/gitweb/?a=commitdiff_plain;h=5b71470449ca55955ee96d141af17fd5ba9dd823;p=ardour.git

Use XMLNode::get_property API in AutomationTimeAxis class
---

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_ptr<Aut
 			const XMLNode* gui_node = ac->extra_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);
 				}
 			}
 		}