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

Initial support for libsoundio #433

Merged
merged 1 commit into from
Jan 21, 2017
Merged

Initial support for libsoundio #433

merged 1 commit into from
Jan 21, 2017

Conversation

shtirlic
Copy link

Support for C library for cross-platform real-time audio input and output
http://libsound.io/
https://github.com/andrewrk/libsoundio

Everything works, please try, need suggestion about settings for software latency.

@mikebrady
Copy link
Owner

Hi Serg. This looks exciting. However, I wonder if it wouldn't be better targeted to the development branch. What do you think? It'll have to go there anyway to have a future. Would there be a lot of work in it?

@shtirlic shtirlic changed the base branch from master to development December 3, 2016 22:03
@shtirlic
Copy link
Author

shtirlic commented Dec 3, 2016

Hi Mike, changed base branch and updated some code, I see that there is a new format in play method, should it be mapped to the target lib formats or we are using default short 2bytes ?

@mikebrady
Copy link
Owner

Many thanks for this Serg.

I'm not sure what you are referring to.

Do you mean that there is a format parameter in the init() function? If so, that should use the enumeration defines in common.h. These are different, unfortunately, to the ALSA enumeration values, so they are translated into ALSA enumerations in audio_alsa.c. Obviously, you don't have to use ALSA stuff in your libsoundio stuff.

The format and the rate default to 16-bit interleaved stereo at 44,100 frames per second anyway, so you may not have to worry.

Sorry if this is not relevant to your question...

@mikebrady mikebrady merged commit 77f700e into mikebrady:development Jan 21, 2017
@mikebrady
Copy link
Owner

Hi Serg. I'm merging the pull request, even though I don't know whether you message of December 3 means it doesn't work. Let us travel hopefully :)

@shtirlic
Copy link
Author

shtirlic commented Jan 25, 2017

Hi Mike, I mixed up the method name it was start 's new format param, but it's ok since I use 16-bit interleaved stereo at 44,100 by default, I will add correct mappings if anyone will encounter issues with custom formats.

@shtirlic shtirlic deleted the libsoundio branch January 25, 2017 07:05
@mikebrady
Copy link
Owner

Great, thanks.

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