Skip to content
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

Segfault in Soloud::stopVoice_internal fixed: mResampleDataOwner arra… #345

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

valid-ptr
Copy link
Contributor

…y size and mMaxActiveVoices was out of sync; this could happen when postinit_internal was not called

…y size and mMaxActiveVoices was out of sync; this could happen when postinit_internal was not called
@konsumer
Copy link

konsumer commented Apr 1, 2023

Hey, @valid-ptr I am trying to make a new fork that has all the PRs, to make an up-to-date and maintained version (here. I notice that you have a bunch of branches:

 * [new branch]      SoLoud_backend_opensles     -> eligovision/SoLoud_backend_opensles
 * [new branch]      master                      -> eligovision/master
 * [new branch]      soloud_backend_pause_resume -> eligovision/soloud_backend_pause_resume
 * [new branch]      soloud_cmake                -> eligovision/soloud_cmake
 * [new branch]      soloud_contrib              -> eligovision/soloud_contrib
 * [new branch]      soloud_contrib_cmake        -> eligovision/soloud_contrib_cmake
 * [new branch]      soloud_encoding             -> eligovision/soloud_encoding
 * [new branch]      soloud_gles                 -> eligovision/soloud_gles
 * [new branch]      soloud_mMaxActiveVoices_fix -> eligovision/soloud_mMaxActiveVoices_fix
 * [new branch]      soloud_macos                -> eligovision/soloud_maco

Which should be merged into my master?

My goal is to merge all these long-pending PRs and add a few new audio-sources.

@valid-ptr
Copy link
Contributor Author

Hi, David!

You can use the master branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants