Skip to content

Add AudioTransform: convert samples #6

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

Merged

Conversation

cchampet
Copy link
Member

@cchampet cchampet commented Jun 4, 2014

  • AudioTransform: does the same process as ColorTransform, but applied
    to audio.
  • AudioDesc: add a function to get a FrameDesc.
  • AudioRewrapper: use AudioTransform.

* AudioTransform: does the same process as ColorTransform, but applied
to audio.
* AudioDesc: add a function to get a FrameDesc.
* AudioRewrapper: use AudioTransform.
@@ -18,6 +18,8 @@ extern "C" {
namespace avtranscoder
{

class AudioFrameDesc;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you use an include instead?

Not a good thing to put forward declaration, when it's used in the API, here: "AudioFrameDesc getFrameDesc()".

Clement Champetier added 4 commits June 5, 2014 12:12
AudioFrameDesc can be used in the public API.
Use swr_alloc_set_opts() instead of setting options one by one, with a
risk of runtime error if an option we want to set doesn't exist.
* Rename audioODesc to audioOutputDesc.
* Explain the magick numbers ;)
MarcAntoine-Arnaud added a commit that referenced this pull request Jun 5, 2014
Add AudioTransform: convert samples
@MarcAntoine-Arnaud MarcAntoine-Arnaud merged commit fe1a073 into avTranscoder:master Jun 5, 2014
valnoel added a commit to valnoel/avTranscoder that referenced this pull request Dec 3, 2014
…Formats

Supported chroma subsampling :
* 4:4:4
* 4:4:0
* 4:2:2
* 4:2:0
* 4:1:1
* 4:1:0
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.

4 participants