[FFmpeg-devel] [PATCH 2/4] Replace remaining av_new_stream() with avformat_new_stream().

Clément Bœsch ubitux at gmail.com
Sat Nov 5 15:26:42 CET 2011


On Sat, Nov 05, 2011 at 02:08:56PM +0100, Stefano Sabatini wrote:
> On date Saturday 2011-11-05 13:30:50 +0100, Clément Bœsch encoded:
> > ---
> >  doc/examples/muxing.c     |    3 ++-
> >  libavdevice/dshow.c       |    3 ++-
> >  libavdevice/lavfi.c       |    3 ++-
> >  libavdevice/openal-dec.c  |    2 +-
> >  libavformat/act.c         |    2 +-
> >  libavformat/bintext.c     |    2 +-
> >  libavformat/g723_1.c      |    2 +-
> >  libavformat/libmodplug.c  |    4 ++--
> >  libavformat/loasdec.c     |    2 +-
> >  libavformat/microdvddec.c |    2 +-
> >  libavformat/pmpdec.c      |    5 +++--
> >  libavformat/wav.c         |    3 ++-
> >  12 files changed, 19 insertions(+), 14 deletions(-)
> 
> Looks fine, although I don't like the fact that the new API requires
> to *remember* to set the stream id. Thanks.

Pushed with the requested changes.

About the stream id stuff, it is justified here:
http://lists.libav.org/pipermail/libav-devel/2011-October/012675.html

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20111105/62994f6b/attachment.asc>


More information about the ffmpeg-devel mailing list