sam/sfizz
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Correct the sostenuto/sustain logic

Browse source 
Only release if the received CC is one of the above AND if the region is not a one-shot
This commit is contained in:
Paul Fd 2021-04-04 21:23:05 +02:00
parent b3ff1469ed
commit 7f02c413c1
2 changed files with 59 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
src/sfizz/Voice.cpp
View file

@ -614,32 +614,38 @@ void Voice::registerCC(int delay, int ccNumber, float ccValue) noexcept
if (impl.region_ == nullptr)
return;
const Region& region = *impl.region_;
if (impl.state_ != State::playing)
return;
if (impl.region_->checkSustain && (ccNumber == impl.region_->sostenutoCC)) {
if (ccValue < impl.region_->sostenutoThreshold) {
if (ccNumber != region.sustainCC && ccNumber != region.sostenutoCC)
return;
if (region.checkSustain && (ccNumber == region.sostenutoCC)) {
if (ccValue < region.sostenutoThreshold) {
impl.sostenutoState_ = Impl::SostenutoState::Up;
} else if (impl.sostenutoState_ == Impl::SostenutoState::Up) {
impl.sostenutoState_ = Impl::SostenutoState::Sustaining;
}
}
if (impl.region_->checkSostenuto && (ccNumber == impl.region_->sustainCC)) {
if (ccValue < impl.region_->sustainThreshold) {
if (region.checkSostenuto && (ccNumber == region.sustainCC)) {
if (ccValue < region.sustainThreshold) {
impl.sustainState_ = Impl::SustainState::Up;
} else {
impl.sustainState_ = Impl::SustainState::Sustaining;
}
}
const bool sustainPedalReleaseCondition = !impl.region_->checkSustain
const bool sustainPedalReleaseCondition = !region.checkSustain
|| (impl.sustainState_ != Impl::SustainState::Sustaining);
const bool sostenutoPedalReleaseCondition = !impl.region_->checkSostenuto
const bool sostenutoPedalReleaseCondition = !region.checkSostenuto
|| (impl.sostenutoState_ != Impl::SostenutoState::Sustaining);
if (impl.noteIsOff_ && sostenutoPedalReleaseCondition && sustainPedalReleaseCondition)
if (impl.noteIsOff_ && region.loopMode != LoopMode::one_shot
&& sostenutoPedalReleaseCondition && sustainPedalReleaseCondition)
release(delay);
}

46
tests/SynthT.cpp
View file

@ -926,6 +926,52 @@ TEST_CASE("[Synth] Release (sustain + sostenuto)")
}
}
TEST_CASE("[Synth] One shot regions with sustain + sostenuto")
{
sfz::Synth synth;
sfz::AudioBuffer<float> buffer { 2, static_cast<unsigned>(synth.getSamplesPerBlock()) };
synth.loadSfzString(fs::current_path() / "tests/TestFiles/one_shot_sustain.sfz", R"(
<region> key=60 sample=kick.wav loop_mode=one_shot
)");
SECTION("Sustain")
{
synth.noteOn(0, 60, 85);
synth.cc(1, 64, 127);
synth.noteOff(2, 60, 85);
synth.renderBlock(buffer);
REQUIRE( numPlayingVoices(synth) == 1 );
synth.cc(1, 64, 0);
synth.renderBlock(buffer);
REQUIRE( numPlayingVoices(synth) == 1 );
}
SECTION("Sostenuto")
{
synth.noteOn(0, 60, 85);
synth.cc(1, 66, 127);
synth.noteOff(2, 60, 85);
synth.renderBlock(buffer);
REQUIRE( numPlayingVoices(synth) == 1 );
synth.cc(1, 66, 0);
synth.renderBlock(buffer);
REQUIRE( numPlayingVoices(synth) == 1 );
}
SECTION("Sostenuto up first")
{
synth.noteOn(0, 60, 85);
synth.cc(1, 66, 127);
synth.cc(1, 64, 127);
synth.noteOff(2, 60, 85);
synth.renderBlock(buffer);
REQUIRE( numPlayingVoices(synth) == 1 );
synth.cc(3, 66, 0);
synth.renderBlock(buffer);
REQUIRE( numPlayingVoices(synth) == 1 );
synth.cc(4, 64, 0);
synth.renderBlock(buffer);
REQUIRE( numPlayingVoices(synth) == 1 );
}
}
TEST_CASE("[Synth] Sustain threshold default")
{
sfz::Synth synth;