[FFmpeg-devel] [PATCH] examples/demuxing: add audio decoding/demuxing
Stefano Sabatini
stefasab at gmail.com
Tue Sep 11 01:20:41 CEST 2012
On date Monday 2012-09-10 11:43:09 +0200, Stefano Sabatini encoded:
> On date Monday 2012-09-10 10:55:12 +0200, Takis Issaris encoded:
> > Hi Stefano,
> >
> >
> > 2012/9/8 Stefano Sabatini <stefasab at gmail.com>:
> > > ---
> > > doc/examples/demuxing.c | 302 ++++++++++++++++++++++++++++++++++-------------
> > > 1 files changed, 222 insertions(+), 80 deletions(-)
> >
> >
> > IMHO it would be better to add this as a separate example, so the
> > original very simple single stream example is still available to new
> > users of the FFmpeg libraries.
>
> I'd prefer to keep a single example (demuxing.c) rather than splitting it
> into demuxing_video.c and demuxing_audio.c.
>
> Rationale (in order of importance):
> - the user may need to do both at the same time
> - most code is shared (easier on the developer/maintainer: less code
> duplication and less synching errors)
> - it is consistent with muxing.c
> - the user should be able to factor out the audio/video code with no
> much effort
I'll push it tomorrow, unless I see someone interested in reviewing it
before.
--
FFmpeg = Furious Fast Mythic Purposeless Excellent Gorilla
More information about the ffmpeg-devel
mailing list