[FFmpeg-devel] [PATCH] examples/demuxing: add audio decoding/demuxing
Stefano Sabatini
stefasab at gmail.com
Tue Sep 11 17:51:51 CEST 2012
On date Tuesday 2012-09-11 01:20:41 +0200, Stefano Sabatini encoded:
> 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.
Pushed with some fixes.
--
FFmpeg = Fast and Fascinating Most Perennial Esoteric Gargoyle
More information about the ffmpeg-devel
mailing list