use const static member instead of a magic number

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

diff --git a/libs/ardour/amp.cc b/libs/ardour/amp.cc
index 80dd7422e9a89d70f1ba4bc1f080235f12cd4db2..f60240fd57c128dc6ca97a74364eecbb90b56912 100644 (file)
--- a/libs/ardour/amp.cc
+++ b/libs/ardour/amp.cc
@@ -33,6 +33,10 @@
 
 using namespace ARDOUR;
 using namespace PBD;
+using std::min;
+
+/* gain range of -inf to +6dB, default 0dB */
+const float Amp::max_gain_coefficient = 1.99526231f;
 
 Amp::Amp (Session& s)
        : Processor(s, "Amp")
@@ -42,8 +46,7 @@ Amp::Amp (Session& s)
        , _gain_automation_buffer(0)
 {
        Evoral::Parameter p (GainAutomation);
-       /* gain range of -inf to +6dB, default 0dB */
-       p.set_range (0, 1.99526231f, 1, false);
+       p.set_range (0, max_gain_coefficient, 1, false);
        boost::shared_ptr<AutomationList> gl (new AutomationList (p));
        _gain_control = boost::shared_ptr<GainControl> (new GainControl (X_("gaincontrol"), s, this, p, gl));
        _gain_control->set_flags (Controllable::GainLike);
@@ -114,7 +117,7 @@ Amp::run (BufferSet& bufs, framepos_t /*start_frame*/, framepos_t /*end_frame*/,
                                for (BufferSet::midi_iterator i = bufs.midi_begin(); i != bufs.midi_end(); ++i) {
 
                                        MidiBuffer& mb (*i);
-
+                                       
                                        for (MidiBuffer::iterator m = mb.begin(); m != mb.end(); ++m) {
                                                Evoral::MIDIEvent<MidiBuffer::TimeType> ev = *m;
                                                if (ev.is_note_on()) {
@@ -172,7 +175,7 @@ Amp::apply_gain (BufferSet& bufs, framecnt_t nframes, gain_t initial, gain_t tar
                        Evoral::MIDIEvent<MidiBuffer::TimeType> ev = *m;
 
                        if (ev.is_note_on()) {
-                               gain_t scale = delta * (ev.time()/(double) nframes);
+                               const gain_t scale = delta * (ev.time()/(double) nframes);
                                ev.scale_velocity (initial+scale);
                        }
                }
@@ -371,10 +374,7 @@ Amp::inc_gain (gain_t factor, void *src)
 void
 Amp::set_gain (gain_t val, void *src)
 {
-       // max gain at about +6dB (10.0 ^ ( 6 dB * 0.05))
-       if (val > 1.99526231f) {
-               val = 1.99526231f;
-       }
+       val = min (val, max_gain_coefficient);
 
        if (src != _gain_control.get()) {
                _gain_control->set_value (val);
@@ -414,9 +414,8 @@ Amp::set_state (const XMLNode& node, int version)
 void
 Amp::GainControl::set_value (double val)
 {
-       // max gain at about +6dB (10.0 ^ ( 6 dB * 0.05))
-       if (val > 1.99526231) {
-               val = 1.99526231;
+       if (val > max_gain_coefficient) {
+               val = max_gain_coefficient;
        }
 
        _amp->set_gain (val, this);
@@ -449,7 +448,7 @@ Amp::GainControl::internal_to_user (double v) const
 void
 Amp::setup_gain_automation (framepos_t start_frame, framepos_t end_frame, framecnt_t nframes)
 {
-       Glib::Mutex::Lock am (control_lock(), Glib::TRY_LOCK);
+       Glib::Threads::Mutex::Lock am (control_lock(), Glib::Threads::TRY_LOCK);
 
        if (am.locked() && _session.transport_rolling() && _gain_control->automation_playback()) {
                assert (_gain_automation_buffer);