ardour/livetrax
13
0
Fork 0
Code

triggerbox: remove static signal added in dd842fdc5c

Browse Source 
Bad design idea. Signal would be emitted from the triggerbox thread, and cannot
return a value from the GUI. Would also be problematic for clip loading
initiated from a control surface
This commit is contained in:
Paul Davis 2021-12-13 15:06:57 -07:00
parent 5e38da36c6
commit 087bdaa87b
2 changed files with 0 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
libs/ardour/ardour/triggerbox.h
View File

@ -293,8 +293,6 @@ class LIBARDOUR_API AudioTrigger : public Trigger {
SegmentDescriptor get_segment_descriptor () const; SegmentDescriptor get_segment_descriptor () const;
static PBD::Signal1<void,std::string> CannotDetermineTempo;
protected: protected:
void retrigger (); void retrigger ();
void set_usable_length (); void set_usable_length ();

3
libs/ardour/triggerbox.cc
View File

@ -524,8 +524,6 @@ Trigger::maybe_compute_next_transition (samplepos_t start_sample, Temporal::Beat
/*--------------------*/ /*--------------------*/
PBD::Signal1<void,std::string> AudioTrigger::CannotDetermineTempo;
AudioTrigger::AudioTrigger (uint64_t n, TriggerBox& b) AudioTrigger::AudioTrigger (uint64_t n, TriggerBox& b)
: Trigger (n, b) : Trigger (n, b)
, data (0) , data (0)
@ -791,7 +789,6 @@ AudioTrigger::determine_tempo ()
if (_apparent_tempo == 0.0) { if (_apparent_tempo == 0.0) {
/* no apparent tempo, just return since we'll use it as-is */ /* no apparent tempo, just return since we'll use it as-is */
std::cerr << "Could not determine tempo for " << name() << std::endl; std::cerr << "Could not determine tempo for " << name() << std::endl;
CannotDetermineTempo (_region->source (0)->name());
return; return;
} }