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

Add a note about return values of snd_pcm_state() #162

Closed
wants to merge 1 commit into from

Conversation

HEnquist
Copy link
Contributor

@HEnquist HEnquist commented Jul 7, 2021

This is a suggested improvement to the documentation of snd_pcm_state(). There is some confusion, probably caused by old bugs that could made it return a negative error code. This stuff is still floating around the internet and sometimes shows up in searches. This PR adds a comment intended to make it absolutely clear that there is never supposed to be a negative error code.

@HEnquist
Copy link
Contributor Author

HEnquist commented Sep 2, 2021

Very quiet here.. The readme states "Patches are also accepted as GitHub pull requests", is that no longer true?

@perexg perexg closed this in c4972cb Sep 3, 2021
@perexg
Copy link
Member

perexg commented Sep 3, 2021

Thank you for your contribution.

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