Skip to content

Dummy stream: new way to add #28

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
merged 7 commits into from
Jul 21, 2014

Conversation

cchampet
Copy link
Member

  • Add EssenceDesc (based class of VideoDesc and AudioDesc).
  • Transcoder: 2 new "add(...)" functions.
  • Refactoring (add const).

Clement Champetier added 7 commits July 21, 2014 14:47
* AudioDesc and VideoDesc inherit EssenceDesc.
* Rename getAudio/VideoCodec to getCodec.
* Rename setAudio/VideoCodec to setCodec.
* remove unused includes.
* Use "const ProfileDesc" instead of "ProfileDesc" when instanciate a
StreamTranscoder.
* Consequences:
     * OutputAudio: setProfile with a const ProfileDesc.
     * OutputVideo: setProfile with a const ProfileDesc.
     * use find instead of [] to access the elements of ProfileDesc.
* 2 new "add(...)" functions, the only ones which can create dummy
streams (from a profileDesc and an EssenceDesc). When the user tries to
create a dummy stream with an empty file name and no EssenceDesc
indicated, there is no add.
* Update comments to fit to this new behavior.
* Use "const EssenceDesc" instead of "EssenceDesc" when add a
DummyStream.
* Consequences:
     * AudioDesc: copy constructor with const EssenceDesc.
     * VideoDesc: copy constructor with const EssenceDesc.
     * DummyVideo: setVideoDesc from a const VideoDesc (already the case
for setAudioDesc in DummyAudio).
MarcAntoine-Arnaud added a commit that referenced this pull request Jul 21, 2014
Dummy stream: new way to add
@MarcAntoine-Arnaud MarcAntoine-Arnaud merged commit c20fc7c into avTranscoder:master Jul 21, 2014
@cchampet cchampet deleted the dev_Dummy branch July 23, 2014 10:16
cchampet pushed a commit to cchampet/avTranscoder that referenced this pull request Dec 29, 2014
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