add wrapper for ChanMapping::get()

The boolean "valid" is implicit, get() returns -1 (really UINT32_MAX)
This simplifies upcoming lua bindings

diff --git a/libs/ardour/ardour/chan_mapping.h b/libs/ardour/ardour/chan_mapping.h
index ebb7be402a2801d9bebe6259e160c8187df9155b..7b9f81976d52338a3a711847255dab63cfd5e5da 100644 (file)
--- a/libs/ardour/ardour/chan_mapping.h
+++ b/libs/ardour/ardour/chan_mapping.h
@@ -40,6 +40,7 @@ public:
        ChanMapping(ARDOUR::ChanCount identity);
 
        uint32_t get(DataType t, uint32_t from, bool* valid);
+       uint32_t get(DataType t, uint32_t from) { return get (t, from, NULL); }
        void     set(DataType t, uint32_t from, uint32_t to);
        void     offset_from(DataType t, int32_t delta);
        void     offset_to(DataType t, int32_t delta);

diff --git a/libs/ardour/chan_mapping.cc b/libs/ardour/chan_mapping.cc
index 5c5bb17de67a292c3094eb4584dbf25e84779603..57734803d2cd5b8751265872f99563b341676800 100644 (file)
--- a/libs/ardour/chan_mapping.cc
+++ b/libs/ardour/chan_mapping.cc
@@ -45,15 +45,15 @@ ChanMapping::get(DataType t, uint32_t from, bool* valid)
 {
        Mappings::iterator tm = _mappings.find(t);
        if (tm == _mappings.end()) {
-               *valid = false;
+               if (valid) { *valid = false; }
                return -1;
        }
        TypeMapping::iterator m = tm->second.find(from);
        if (m == tm->second.end()) {
-               *valid = false;
+               if (valid) { *valid = false; }
                return -1;
        }
-       *valid = true;
+       if (valid) { *valid = true; }
        return m->second;
 }