diff options
author | Robin Gareus <robin@gareus.org> | 2017-10-04 00:35:29 +0200 |
---|---|---|
committer | Robin Gareus <robin@gareus.org> | 2017-10-04 00:54:55 +0200 |
commit | e31f5d999837b0d0d63dc824c50e9da4acca7244 (patch) | |
tree | ef4712ccd25e8cc651ffabcd16e120571c21921d /libs/ardour/processor.cc | |
parent | 5dd9acf9ab39306277652c28fb5ac4af330e0af0 (diff) |
Clean up State API:
* Processor implement get_state(), classes derived from Processor
implement protected ::state() -- as documented in processor.h
* likewise for Route, Track: make ::state() a protected interface
* removal of "full_state", use explicit "template_save"
* use RAII/Unwind to skip saving automation-state
Diffstat (limited to 'libs/ardour/processor.cc')
-rw-r--r-- | libs/ardour/processor.cc | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/libs/ardour/processor.cc b/libs/ardour/processor.cc index 03510130a2..c6e6c21360 100644 --- a/libs/ardour/processor.cc +++ b/libs/ardour/processor.cc @@ -102,7 +102,7 @@ Processor::~Processor () XMLNode& Processor::get_state (void) { - return state (true); + return state (); } /* NODE STRUCTURE @@ -120,7 +120,7 @@ Processor::get_state (void) */ XMLNode& -Processor::state (bool full_state) +Processor::state () { XMLNode* node = new XMLNode (state_node_name); @@ -132,7 +132,7 @@ Processor::state (bool full_state) node->add_child_copy (*_extra_xml); } - if (full_state) { + if (!skip_saving_automation) { XMLNode& automation = Automatable::get_automation_xml_state(); if (!automation.children().empty() || !automation.properties().empty()) { node->add_child_nocopy (automation); |