make auto track naming make a bit more sense
authorLen Ovens <len@ovenwerks.net>
Mon, 7 Nov 2016 19:08:00 +0000 (11:08 -0800)
committerLen Ovens <len@ovenwerks.net>
Mon, 7 Nov 2016 19:08:00 +0000 (11:08 -0800)
libs/ardour/session.cc

index 51bef1202b91ddd3686526d1bf47d2f465d0f79d..7e96cefbf6263fc959874b9b98e80edb2439dcbf 100644 (file)
@@ -2380,8 +2380,11 @@ Session::find_route_name (string const & base, uint32_t& id, string& name, bool
                }
        }
 
-       if (!definitely_add_number && route_by_name (base) == 0) {
-               /* juse use the base */
+       /* if we have "base 1" already, it doesn't make sense to add "base"
+        * if "base 1" has been deleted, adding "base" is no worse than "base 1"
+        */
+       if (!definitely_add_number && route_by_name (base) == 0 && (route_by_name (string_compose("%1 1", base)) == 0)) {
+               /* just use the base */
                name = base;
                return true;
        }