[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