On Fri, 21 Oct 2011 14:52:17 -0700, "Ronald S. Bultje" <[email protected]> 
wrote:
> Hi,
> 
> On Fri, Oct 21, 2011 at 2:32 PM, Michael Karcher
> <[email protected]> wrote:
> > ---
> >  libavformat/id3v2.c  |  121 
> > ++++++++++++++++++++++++++++++++++++++++++++++++++
> >  libavformat/id3v2.h  |   10 ++++-
> >  libavformat/mp3enc.c |  116 
> > ++----------------------------------------------
> >  3 files changed, 134 insertions(+), 113 deletions(-)
> 
> <lame> would it be ok to put it in a new file (e.g. id3v2enc.c, or
> so)? This way we can easily disable it without #if MUXER stuff in
> source files if people want decoders/demuxers only. </lame>
> 

+1, id3v2.c is cluttered enough as it is.
Please move it to a new file instead.

-- 
Anton Khirnov
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to