fix bypassing plugins with sidechain i/o

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

diff --git a/libs/ardour/beats_frames_converter.cc b/libs/ardour/beats_frames_converter.cc
index 584732ff03e87029e8dfd8eb2fefa44d770da718..54db8008da2c4fcabacaba0d0da64963952da040 100644 (file)
--- a/libs/ardour/beats_frames_converter.cc
+++ b/libs/ardour/beats_frames_converter.cc
@@ -19,6 +19,8 @@
     $Id: midiregion.h 733 2006-08-01 17:19:38Z drobilla $
 */
 
+#include "pbd/stacktrace.h"
+
 #include "ardour/beats_frames_converter.h"
 #include "ardour/tempo.h"
 
@@ -28,23 +30,44 @@ namespace ARDOUR {
  *  supplied to the constructor.  Returns the equivalent number of frames,
  *  taking tempo changes into account.
  */
-framecnt_t
-BeatsFramesConverter::to (double beats) const
+framepos_t
+BeatsFramesConverter::to (Evoral::Beats beats) const
 {
-       assert (beats >= 0);
-       framecnt_t r = _tempo_map.framepos_plus_beats (_origin_b, beats) - _origin_b;
-       return r;
+       if (beats < Evoral::Beats()) {
+               std::cerr << "negative beats passed to BFC: " << beats << std::endl;
+               PBD::stacktrace (std::cerr, 30);
+               return 0;
+       }
+       return _tempo_map.framepos_plus_beats (_origin_b, beats) - _origin_b;
 }
 
 /** Takes a duration in frames and considers it as a distance from the origin
  *  supplied to the constructor.  Returns the equivalent number of beats,
  *  taking tempo changes into account.
  */
+Evoral::Beats
+BeatsFramesConverter::from (framepos_t frames) const
+{
+       return _tempo_map.framewalk_to_beats (_origin_b, frames);
+}
+
+/** As above, but with beats in double instead (for GUI). */
+framepos_t
+DoubleBeatsFramesConverter::to (double beats) const
+{
+       if (beats < 0.0) {
+               std::cerr << "negative beats passed to BFC: " << beats << std::endl;
+               PBD::stacktrace (std::cerr, 30);
+               return 0;
+       }
+       return _tempo_map.framepos_plus_beats (_origin_b, Evoral::Beats(beats)) - _origin_b;
+}
+
+/** As above, but with beats in double instead (for GUI). */
 double
-BeatsFramesConverter::from (framecnt_t frames) const
+DoubleBeatsFramesConverter::from (framepos_t frames) const
 {
-       double b = _tempo_map.framewalk_to_beats (_origin_b, frames);
-       return b;
+       return _tempo_map.framewalk_to_beats (_origin_b, frames).to_double();
 }
 
 } /* namespace ARDOUR */