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

Michael Niedermayer michaelni
Sun Aug 3 17:00:47 CEST 2008


On Sun, Aug 03, 2008 at 04:49:05PM +0200, Diego Biurrun wrote:
> On Sun, Aug 03, 2008 at 03:30:27PM +0200, Michael Niedermayer wrote:
> > On Sun, Aug 03, 2008 at 01:47:11PM +0200, Diego Biurrun wrote:
> > > 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?  
> > 
> > Who said it is correct for the others?
> > 
> > > And why is it correct for libavcodec in the
> > > show_banner function 10 lines above?
> > 
> > Who said it is correct there?
> 
> Nobody said it is correct, but when you have 10 such examples in the
> immediate vicinity without a comment indicating that they should be
> changed, then it's very easy to misunderstand...
> 
> What is the problem with mixing the version of the headers and of the
> linked library anyway?  Do I understand correctly that it is for the
> case where ffmpeg/ffplay/ffserver is dynamically linked and the version
> of the linked libs differs from the version of the headers used to
> compile the binary?

yes, its kinda usefull for bugreports where the user has several libs and
by mistake linked to the wrong one.


> 
> And thus ideally there should be a _version() function for every single
> library?

yes

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

There will always be a question for which you do not know the correct awnser.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080803/d74706eb/attachment.pgp>



More information about the ffmpeg-devel mailing list