-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Add end() method to MixerVoice to allow samples to finish playing bef… #10309
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the PR! One suggestion to fix the compile and you'll want to tweak the formatting to get a minimal diff. Thanks!
Co-authored-by: Scott Shawcroft <scott@tannewt.org>
Co-authored-by: Mark <56205165+gamblor21@users.noreply.github.com>
Co-authored-by: Mark <56205165+gamblor21@users.noreply.github.com>
Co-authored-by: Mark <56205165+gamblor21@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me! Thank you!
Added an
end
method to the MixerVoice class within the auidomixer library.Calling this method will set the object's
loop
flag toFalse
. This is different than thestop
method, which ends the sample immediately, and instead allows any looping sample to play thorugh its entirety before ending.This feature was added to fix a problem where samples ended via
stop
created a loud clicking sound from the connected speaker. By allowing the samples to play out fully, the clicking does not exist.I have tested this on multiple looping audio samples.