Remove unnecessary 0 checks before delete; see http://www.parashift.com/c++-faq-lite...

[ardour.git] / libs / ardour / route_group.cc

diff --git a/libs/ardour/route_group.cc b/libs/ardour/route_group.cc
index 57306237427ff581a17e352b615d29516ccf3bc0..f22deb5dbfd2c6b6420b7ae54f5339884c7856f2 100644 (file)
--- a/libs/ardour/route_group.cc
+++ b/libs/ardour/route_group.cc
@@ -15,7 +15,6 @@
     along with this program; if not, write to the Free Software
     Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
 
-    $Id$
 */
 
 #define __STDC_FORMAT_MACROS
@@ -26,6 +25,7 @@
 #include <sigc++/bind.h>
 
 #include <pbd/error.h>
+#include <pbd/enumwriter.h>
 
 #include <ardour/route_group.h>
 #include <ardour/audio_track.h>
@@ -125,11 +125,9 @@ RouteGroup::get_max_factor(gain_t factor)
 XMLNode&
 RouteGroup::get_state (void)
 {
-       char buf[32];
        XMLNode *node = new XMLNode ("RouteGroup");
        node->add_property ("name", _name);
-       snprintf (buf, sizeof (buf), "%" PRIu32, (uint32_t) _flags);
-       node->add_property ("flags", buf);
+       node->add_property ("flags", enum_2_string (_flags));
        return *node;
 }
 
@@ -143,7 +141,7 @@ RouteGroup::set_state (const XMLNode& node)
        }
 
        if ((prop = node.property ("flags")) != 0) {
-               _flags = atoi (prop->value().c_str());
+               _flags = Flag (string_2_enum (prop->value(), _flags));
        }
 
        return 0;
@@ -151,15 +149,14 @@ RouteGroup::set_state (const XMLNode& node)
 
 void
 RouteGroup::set_active (bool yn, void *src)
-
 {
        if (is_active() == yn) {
                return;
        }
        if (yn) {
-               _flags |= Active;
+               _flags = Flag (_flags | Active);
        } else {
-               _flags &= ~Active;
+               _flags = Flag (_flags & ~Active);
        }
        _session.set_dirty ();
        FlagsChanged (src); /* EMIT SIGNAL */
@@ -173,9 +170,9 @@ RouteGroup::set_relative (bool yn, void *src)
                return;
        }
        if (yn) {
-               _flags |= Relative;
+               _flags = Flag (_flags | Relative);
        } else {
-               _flags &= ~Relative;
+               _flags = Flag (_flags & ~Relative);
        }
        _session.set_dirty ();
        FlagsChanged (src); /* EMIT SIGNAL */
@@ -189,14 +186,14 @@ RouteGroup::set_hidden (bool yn, void *src)
                return;
        }
        if (yn) {
-               _flags |= Hidden;
+               _flags = Flag (_flags | Hidden);
                if (Config->get_hiding_groups_deactivates_groups()) {
-                       _flags &= ~Active;
+                       _flags = Flag (_flags & ~Active);
                }
        } else {
-               _flags &= ~Hidden;
+               _flags = Flag (_flags & ~Hidden);
                if (Config->get_hiding_groups_deactivates_groups()) {
-                       _flags |= Active;
+                       _flags = Flag (_flags | Active);
                }
        }
        _session.set_dirty ();