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

Decoder/demuxer options that are kinda useful to expose/change #390

Closed
myrsloik opened this issue Aug 16, 2021 · 2 comments
Closed

Decoder/demuxer options that are kinda useful to expose/change #390

myrsloik opened this issue Aug 16, 2021 · 2 comments
Assignees

Comments

@myrsloik
Copy link
Contributor

myrsloik commented Aug 16, 2021

Demuxer:
mov/mp4/3gp: advanced_editlist (always set to 0)
mov/mp4/3gp: use_absolute_path
mov/mp4/3gp: enable_drefs

Decoder:
ac3: drc_scale

@myrsloik myrsloik self-assigned this Aug 16, 2021
@dwbuiten
Copy link
Member

We really should not hard set advanced_editlist to 0, it has quite some uses, and we even have tests for it - I, at least, use it.

Further, I am not a fan of the new API with specific boolean options... we should really properly expose an API that lets API users set AVOptions themselves, while having some sane defaults. It would let me e.g. get rid of all my networking patches in my local branch, too.

@dwbuiten dwbuiten reopened this Aug 17, 2021
@myrsloik
Copy link
Contributor Author

resolved in a better way

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