-
Notifications
You must be signed in to change notification settings - Fork 40
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
Why did you integrate it into FFmpeg and drop the standalone decoder? #59
Comments
ffmpeg can also output to a pipe, so any usage scenarios you used before can be done after as well. |
Technically, but it's still against the unix philosophy. |
Big deal. The "unix philosophy" is not a religious dogma. |
Neither is FFmpeg. in fact it's code base is horrific. I mean seriously who the fuck iterates over command line options by using pointer math? |
How else would you do that, given that C provides command line options as array, which always require pointer math for accessing them? |
Uh, index it as an array and let the compiler do the pointer math? |
That's the same. I agree that FFmpeg does some strange things, but dismissing it just because of some minor stylistic issues seems a bit strange. There's much more valid criticism you could make. Besides I looked at some of your code and I found strange or broken things too. |
But it's less confusing. I don't dismiss FFmpeg because of a few minor stylistic issues, their entire approach is wrong, tools should work together, not be built up together containing everything monolithically IMO. Not to menton is kind of a huge mess, how many different functions do you need to read a variable amount of bits? I mean seriously. AFAIK there's at least 3, tho it's been a while so I may be misremembering their names. One reads one bit, one variant reads I believe up to 25 bits? and another that's more general, and can read up to 64 bits, all in use at the same time, without buffering or anything, sorry dude that's just a badly designed API. |
You should ask the people that make audio/video formats how many different ways to encode variable length data you need. Its probably one more then the number of people you asked. |
Hey! that's me! turns out the answer is that it's really not that difficult. |
Does your code support as many media formats at at least the same performance? Link to your library? |
Which one? I haven't had a decade and dozens of volunteers writing code for me, of course I don't support as many formats as ffmpeg, and that's before you factor in the fact that about a quarter of the formats it supports are entirely useless (one off game specific formats are grody). Feel free to peruse my profile. |
You're tying your wagon to a less-than-healthy horse for no real benefit that I can see?
Also, FFmpeg in general clashes with the unix philosophy, and makes it harder to get what you need done.
If it was a standalone decoder, I could pipe the output PCM into any encoder I wanted, but now I either have to deal with what ffmpeg provides, OR output to a wave file on disk.
But hey, whateve. I've learned a TON about programming my projects, BitIO, etc, so it's not a huge deal.
The text was updated successfully, but these errors were encountered: