Tonesweep ignores t_time if t_lo = t_hi

[waltsatan]

Description

Tonesweep ignores t_time if t_lo = t_hi. If both values are the same, the tone plays continually.

Hardware & Software

Teensy 3.2
Arduino IDE 1.85
Teensyduino 1.45

Arduino Sketch

#include <Audio.h>

AudioSynthToneSweep      tonesweep;
AudioOutputAnalog        dac;
AudioConnection          patchCord1(tonesweep1, dac);

void setup() {
	AudioMemory(16);
}

unsigned long lastTone;

void loop() {
	if ( millis() - lastTone > 2000 ) {
		tonesweep.play(1.0, 659, 659, 0.5);
		lastTone=millis();
	}
}

[waltsatan]

I was just inspecting synth_tonesweep.cpp and sweep_busy requires an incremental value derived from the hi-low value for it to stop playback (line 109). I think storing the start play time and stopping it if the current millis() - start_time is > t_time would be the only fix.