On Tue, Jun 28, 2016 at 07:34:31AM +0200, Anton Khirnov wrote:
> Quoting Diego Biurrun (2016-06-27 16:22:40)
> > --- a/libavcodec/Makefile
> > +++ b/libavcodec/Makefile
> > @@ -640,28 +640,13 @@ OBJS-$(CONFIG_ISO_MEDIA)               += 
> > mpeg4audio.o mpegaudiodata.o
> > -OBJS-$(CONFIG_MATROSKA_MUXER)          += flac.o flacdata.o
> >  OBJS-$(CONFIG_MOV_DEMUXER)             += ac3tab.o
> > -OBJS-$(CONFIG_MP2_MUXER)               += mpegaudiodata.o 
> > mpegaudiodecheader.o
> 
> This looks ok in principle, but the mp2 muxer is defined in the same
> file as mp3, even though it does not use any of its functions. Those
> functions (which do depend on this mpegaudio stuff) are compiled
> unconditionally, so IIUC it depends on the compiler DCEing away all this
> unused code.

Not DCE as there are no references to nonexisting functions or
something, it's just unused code that sits there and gcc complains
about.

Diego
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to