[FFmpeg-devel] [PATCH] cmdutils.c warning fix

Diego Biurrun diego
Sun Aug 3 13:47:11 CEST 2008


On Sat, Aug 02, 2008 at 07:23:19PM +0200, Michael Niedermayer wrote:
> On Sat, Aug 02, 2008 at 01:27:07PM +0200, Diego Biurrun wrote:
> > Here is a small one to get rid of
> > 
> > cmdutils.c:226: warning: 'avcodec_build' is deprecated (declared at ./libavcodec/avcodec.h:2534)
> 
> the patch is incorrect, and ive alraedy rejected it when (IIRC) stefano 
> proposed it.
> avcodec_build() is the version of the linked library, LIBAVCODEC_BUILD is
> the version of the headers.

Why is this wrong for libavcodec and correct for all other libs in the
show_version function?  And why is it correct for libavcodec in the
show_banner function 10 lines above?

Anyway, here is a patch using avcodec_version().

Diego
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cmdutils.diff
Type: text/x-diff
Size: 464 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080803/c034351b/attachment.diff>



More information about the ffmpeg-devel mailing list