smokephil/ardour
1
0
Fork 0
mirror of https://github.com/Ardour/ardour.git synced 2025-12-21 22:26:29 +01:00
Code Issues Projects Releases Wiki Activity

remove muted-by-others, a concept that turns out to have been a red herring; add auditioning alert button to monitor section; start adding exclusive solo button to monitor section

Browse source 
git-svn-id: svn://localhost/ardour2/branches/3.0@7052 d708f5d6-7413-0410-9779-e7cbd77b26cf
This commit is contained in:
Paul Davis 2010-05-04 16:39:03 +00:00
parent 823bad3049
commit bf6de6da77
10 changed files with 117 additions and 201 deletions
Download patch file Download diff file Expand all files Collapse all files

19
libs/ardour/ardour/mute_master.h
View file

@ -46,12 +46,8 @@ class MuteMaster : public SessionHandleRef, public PBD::Stateful
MuteMaster (Session& s, const std::string& name);
~MuteMaster() {}
bool self_muted() const { return _self_muted && (_mute_point != MutePoint (0)); }
bool muted_by_others() const { return _muted_by_others && (_mute_point != MutePoint (0)); }
bool muted() const { return (_self_muted || (_muted_by_others > 0)) && (_mute_point != MutePoint (0)); }
bool muted_at (MutePoint mp) const { return (_self_muted || (_muted_by_others > 0)) && (_mute_point & mp); }
bool self_muted_at (MutePoint mp) const { return _self_muted && (_mute_point & mp); }
bool muted_by_others_at (MutePoint mp) const { return (_muted_by_others > 0) && (_mute_point & mp); }
bool muted() const { return _muted && (_mute_point != MutePoint (0)); }
bool muted_at (MutePoint mp) const { return _muted && (_mute_point & mp); }
bool muted_pre_fader() const { return muted_at (PreFader); }
bool muted_post_fader() const { return muted_at (PostFader); }
@ -60,9 +56,7 @@ class MuteMaster : public SessionHandleRef, public PBD::Stateful
gain_t mute_gain_at (MutePoint) const;
void set_self_muted (bool yn) { _self_muted = yn; }
void mod_muted_by_others (int delta);
void clear_muted_by_others ();
void set_muted (bool yn) { _muted = yn; }
void mute_at (MutePoint);
void unmute_at (MutePoint);
@ -71,7 +65,7 @@ class MuteMaster : public SessionHandleRef, public PBD::Stateful
void set_mute_points (MutePoint);
MutePoint mute_points() const { return _mute_point; }
void set_solo_level (SoloLevel);
void set_soloed (bool);
void set_solo_ignore (bool yn) { _solo_ignore = yn; }
PBD::Signal0<void> MutePointChanged;
@ -81,9 +75,8 @@ class MuteMaster : public SessionHandleRef, public PBD::Stateful
private:
volatile MutePoint _mute_point;
volatile bool _self_muted;
volatile uint32_t _muted_by_others;
volatile SoloLevel _solo_level;
volatile bool _muted;
volatile bool _soloed;
volatile bool _solo_ignore;
};

8
libs/ardour/ardour/route.h
View file

@ -126,14 +126,7 @@ class Route : public SessionObject, public AutomatableControls, public RouteGrou
MuteMaster::MutePoint mute_points() const { return _mute_points; }
bool muted () const;
bool self_muted () const;
bool muted_by_others () const;
bool path_muted_by_others() const { return _path_muted_by_others > 0; }
void mod_path_muted_by_others (int delta);
void set_mute (bool yn, void* src);
void mod_muted_by_others (int delta);
/* controls use set_solo() to modify this route's solo state
*/
@ -437,7 +430,6 @@ class Route : public SessionObject, public AutomatableControls, public RouteGrou
boost::shared_ptr<MuteControllable> _mute_control;
boost::shared_ptr<MuteMaster> _mute_master;
MuteMaster::MutePoint _mute_points;
volatile uint32_t _path_muted_by_others;
std::string _comment;
bool _have_internal_generator;

7
libs/ardour/ardour/types.h
View file

@ -306,13 +306,6 @@ namespace ARDOUR {
PreFaderListen
};
enum SoloLevel {
NotSoloed,
DownstreamSoloed,
UpstreamSoloed,
SelfSoloed
};
enum AutoConnectOption {
ManualConnect = 0x0,
AutoConnectPhysical = 0x1,

73
libs/ardour/mute_master.cc
View file

@ -38,8 +38,8 @@ const MuteMaster::MutePoint MuteMaster::AllPoints = MutePoint (MuteMaster::PreFa
MuteMaster::MuteMaster (Session& s, const std::string&)
: SessionHandleRef (s)
, _mute_point (AllPoints)
, _self_muted (false)
, _muted_by_others (0)
, _muted (false)
, _soloed (false)
, _solo_ignore (false)
{
}
@ -49,7 +49,6 @@ MuteMaster::mute_at (MutePoint mp)
{
if ((_mute_point & mp) != mp) {
_mute_point = MutePoint (_mute_point | mp);
cerr << "Mute point set, now " << _mute_point << endl;
MutePointChanged (); // EMIT SIGNAL
}
}
@ -59,53 +58,25 @@ MuteMaster::unmute_at (MutePoint mp)
{
if ((_mute_point & mp) == mp) {
_mute_point = MutePoint (_mute_point & ~mp);
cerr << "Mute point unset, now " << _mute_point << endl;
MutePointChanged (); // EMIT SIGNAL
}
}
void
MuteMaster::clear_muted_by_others ()
MuteMaster::set_soloed (bool yn)
{
_muted_by_others = 0;
}
void
MuteMaster::mod_muted_by_others (int32_t delta)
{
if (delta < 0) {
if (_muted_by_others >= (uint32_t) abs (delta)) {
_muted_by_others += delta;
} else {
_muted_by_others = 0;
}
} else {
_muted_by_others += delta;
}
}
void
MuteMaster::set_solo_level (SoloLevel l)
{
_solo_level = l;
_soloed = yn;
}
gain_t
MuteMaster::mute_gain_at (MutePoint mp) const
{
gain_t gain;
const SoloLevel l = _solo_level;
// cerr << "solo level = " << _solo_level << " selfmuted " << self_muted_at (mp) << " omute " << muted_by_others_at (mp) << endl;
if (Config->get_solo_mute_override()) {
if ((l == SelfSoloed) || (l == UpstreamSoloed)) {
if (_soloed) {
gain = 1.0;
} else if (self_muted_at (mp)) { // self-muted
gain = Config->get_solo_mute_gain ();
} else if (l == DownstreamSoloed) {
gain = 1.0;
} else if (muted_by_others_at (mp)) { // muted by others
} else if (muted_at (mp)) { // self-muted
gain = Config->get_solo_mute_gain ();
} else {
if (!_solo_ignore && _session.soloing()) {
@ -115,13 +86,9 @@ MuteMaster::mute_gain_at (MutePoint mp) const
}
}
} else {
if (self_muted_at (mp)) { // self-muted
if (muted_at (mp)) { // self-muted
gain = Config->get_solo_mute_gain ();
} else if ((l == SelfSoloed) || (l == UpstreamSoloed)) {
gain = 1.0;
} else if (muted_by_others_at (mp)) { // muted by others
gain = Config->get_solo_mute_gain ();
} else if (l == DownstreamSoloed) { // soloed by others
} else if (_soloed) {
gain = 1.0;
} else {
if (!_solo_ignore && _session.soloing()) {
@ -132,8 +99,6 @@ MuteMaster::mute_gain_at (MutePoint mp) const
}
}
// cerr << "\tgain = " << gain << endl;
return gain;
}
@ -143,7 +108,6 @@ MuteMaster::set_mute_points (const std::string& mute_point)
MutePoint old = _mute_point;
_mute_point = (MutePoint) string_2_enum (mute_point, _mute_point);
cerr << "Mute point set from string, now " << _mute_point << endl;
if (old != _mute_point) {
MutePointChanged(); /* EMIT SIGNAL */
@ -155,7 +119,6 @@ MuteMaster::set_mute_points (MutePoint mp)
{
if (_mute_point != mp) {
_mute_point = mp;
cerr << "Mute point set from mp, now " << _mute_point << endl;
MutePointChanged (); /* EMIT SIGNAL */
}
}
@ -167,21 +130,12 @@ MuteMaster::set_state (const XMLNode& node, int /*version*/)
if ((prop = node.property ("mute-point")) != 0) {
_mute_point = (MutePoint) string_2_enum (prop->value(), _mute_point);
cerr << "Mute point set from STATE string, now " << _mute_point << endl;
}
if ((prop = node.property ("muted")) != 0) {
_self_muted = string_is_affirmative (prop->value());
_muted = string_is_affirmative (prop->value());
} else {
_self_muted = (_mute_point != MutePoint (0));
}
if ((prop = node.property ("muted-by-others")) != 0) {
if (sscanf (prop->value().c_str(), "%u", &_muted_by_others) != 1) {
_muted_by_others = 0;
}
} else {
_muted_by_others = 0;
_muted = (_mute_point != MutePoint (0));
}
return 0;
@ -192,11 +146,6 @@ MuteMaster::get_state()
{
XMLNode* node = new XMLNode (X_("MuteMaster"));
node->add_property ("mute-point", enum_2_string (_mute_point));
node->add_property ("muted", (_self_muted ? X_("yes") : X_("no")));
char buf[32];
snprintf (buf, sizeof (buf), "%u", _muted_by_others);
node->add_property ("muted-by-others", buf);
node->add_property ("muted", (_muted ? X_("yes") : X_("no")));
return *node;
}

84
libs/ardour/route.cc
View file

@ -91,7 +91,6 @@ Route::Route (Session& sess, string name, Flag flg, DataType default_type)
, _mute_control (new MuteControllable (X_("mute"), *this))
, _mute_master (new MuteMaster (sess, name))
, _mute_points (MuteMaster::AllPoints)
, _path_muted_by_others (false)
, _have_internal_generator (false)
, _physically_connected (false)
, _graph_level (-1)
@ -617,13 +616,26 @@ Route::mod_solo_by_others_upstream (int32_t delta)
_soloed_by_others_upstream += delta;
}
/* push the inverse solo change to everything that feeds us.
This is important for solo-within-group. When we solo 1 track out of N that
feed a bus, that track will cause mod_solo_by_upstream (+1) to be called
on the bus. The bus then needs to call mod_solo_by_downstream (-1) on all
tracks that feed it. This will silence them if they were audible because
of a bus solo, but the newly soloed track will still be audible (because
it is self-soloed).
but .. do this only when we are being told to solo-by-upstream (i.e delta = +1),
not in reverse.
*/
for (FedBy::iterator i = _fed_by.begin(); i != _fed_by.end(); ++i) {
boost::shared_ptr<Route> sr = i->r.lock();
if (sr) {
sr->mod_solo_by_others_downstream (-delta);
if (delta > 0) {
for (FedBy::iterator i = _fed_by.begin(); i != _fed_by.end(); ++i) {
boost::shared_ptr<Route> sr = i->r.lock();
if (sr) {
sr->mod_solo_by_others_downstream (-delta);
}
}
}
@ -655,19 +667,7 @@ Route::mod_solo_by_others_downstream (int32_t delta)
void
Route::set_mute_master_solo ()
{
SoloLevel level;
if (self_soloed()) {
level = SelfSoloed;
} else if (soloed_by_others_upstream()) {
level = UpstreamSoloed;
} else if (soloed_by_others_downstream()) {
level = DownstreamSoloed;
} else {
level = NotSoloed;
}
_mute_master->set_solo_level (level);
_mute_master->set_soloed (self_soloed() || soloed_by_others_downstream() || soloed_by_others_upstream());
}
void
@ -745,8 +745,8 @@ Route::set_mute (bool yn, void *src)
return;
}
if (self_muted() != yn) {
_mute_master->set_self_muted (yn);
if (muted() != yn) {
_mute_master->set_muted (yn);
mute_changed (src); /* EMIT SIGNAL */
}
}
@ -754,47 +754,7 @@ Route::set_mute (bool yn, void *src)
bool
Route::muted () const
{
return self_muted() || muted_by_others();
}
bool
Route::self_muted() const
{
return _mute_master->self_muted ();
}
bool
Route::muted_by_others() const
{
return _mute_master->muted_by_others ();
}
void
Route::mod_muted_by_others (int delta)
{
if (_solo_isolated) {
return;
}
bool old = muted ();
_mute_master->mod_muted_by_others (delta);
if (old != muted()) {
mute_changed (this);
}
}
void
Route::mod_path_muted_by_others (int32_t delta)
{
if (delta < 0) {
if (_path_muted_by_others >= (uint32_t) abs (delta)) {
_path_muted_by_others += delta;
} else {
_path_muted_by_others = 0;
}
} else {
_path_muted_by_others += delta;
}
return _mute_master->muted();
}
#if 0
@ -3069,7 +3029,7 @@ Route::MuteControllable::set_value (float val)
float
Route::MuteControllable::get_value (void) const
{
return route.self_muted() ? 1.0f : 0.0f;
return route.muted() ? 1.0f : 0.0f;
}
void

43
libs/ardour/session.cc
View file

@ -2251,7 +2251,7 @@ Session::route_solo_changed (bool self_solo_change, void* /*src*/, boost::weak_p
error << string_compose (_("programming error: %1"), X_("invalid route weak ptr passed to route_solo_changed")) << endmsg;
return;
}
shared_ptr<RouteList> r = routes.reader ();
int32_t delta;
@ -2260,21 +2260,19 @@ Session::route_solo_changed (bool self_solo_change, void* /*src*/, boost::weak_p
} else {
delta = -1;
}
if (delta == 1 && !Config->get_solo_latched()) {
/* new solo: disable all other solos */
for (RouteList::iterator i = r->begin(); i != r->end(); ++i) {
if ((*i) == route || (*i)->solo_isolated() || (*i)->is_master() || (*i)->is_monitor() || (*i)->is_hidden()) {
continue;
}
(*i)->set_solo (false, this);
}
}
solo_update_disabled = true;
/*
solo a route:
for anything in the signal path for this route, increment its soloed-by-other count
for anything not in the signal path for this route, increment its muted-by-other count
unsolo a route:
for anything in the signal path for this route, decrement its soloed-by-other count
for anything not in the signal path for this route, decrement its muted-by-other count
*/
RouteList uninvolved;
for (RouteList::iterator i = r->begin(); i != r->end(); ++i) {
@ -2287,15 +2285,6 @@ Session::route_solo_changed (bool self_solo_change, void* /*src*/, boost::weak_p
in_signal_flow = false;
/* feed-backwards (other route to solo change route):
if (*i) feeds the one whose solo status changed
it should be soloed by other if the change was -> solo OR de-soloed by other if change was -> !solo
else
do nothing
*/
if ((*i)->feeds (route, &via_sends_only)) {
if (!via_sends_only) {
(*i)->mod_solo_by_others_downstream (delta);
@ -2303,14 +2292,6 @@ Session::route_solo_changed (bool self_solo_change, void* /*src*/, boost::weak_p
}
}
/* feed-forward (solo change route to other routes):
if the route whose solo status changed feeds (*i)
do nothing
else
mute if the change was -> solo OR demute if change was -> !solo
*/
if (route->feeds (*i, &via_sends_only)) {
(*i)->mod_solo_by_others_upstream (delta);
in_signal_flow = true;