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

alacconvert does not handle 8-bit wav #11

Open
macosforgebot opened this issue Jul 10, 2012 · 3 comments
Open

alacconvert does not handle 8-bit wav #11

macosforgebot opened this issue Jul 10, 2012 · 3 comments

Comments

@macosforgebot
Copy link

justin.ruggles@… originally submitted this as ticket:12

  • Version: 1.0

Instead of rejecting it as unsupported, the encoder seems to try to encode it, but goes into an infinite loop.

@macosforgebot
Copy link
Author

contact@… originally submitted this as comment:1:⁠ticket:12


I've made changes to check for supported WAV PCM bit depth (i.e., reject anything that's not 16- or 24-bit).

It's my first time submitting changes to MacOSforge — are there any guidelines regarding code review before committing, &cet.? Thanks!

@macosforgebot
Copy link
Author

contact@… originally submitted this as attachment:patch.main.cpp:⁠ticket:12


Patch for convert-utility/main.cpp

@AngeloStavrow
Copy link

AngeloStavrow commented Sep 9, 2016

Should I fork and re-submit the patch as a pull request?

EDIT: Ah, wait. I'm only patching the convert utility, not the encoder itself. Will fix.

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

No branches or pull requests

2 participants