On Sat, Jun 04, 2011 at 04:34:17PM +0100, Måns Rullgård wrote:
> Diego Biurrun <[email protected]> writes:
> > --- a/libavcodec/mpc.h
> > +++ b/libavcodec/mpc.h
> > @@ -29,13 +29,15 @@
> >  #ifndef AVCODEC_MPC_H
> >  #define AVCODEC_MPC_H
> >  
> > +#include <stdint.h>
> > +
> >  #include "libavutil/lfg.h"
> >  #include "avcodec.h"
> >  #include "get_bits.h"
> >  #include "dsputil.h"
> >  #include "mpegaudio.h"
> > +#include "mpegaudiodsp.h"
> >  
> > -#include "mpcdata.h"
> >  
> >  #define BANDS            32
> >  #define SAMPLES_PER_BAND 36
> 
> This mixes several changes that might be better to split.

You already took care of this, thanks.

> > --- a/libavcodec/mpegaudiodectab.h
> > +++ b/libavcodec/mpegaudiodectab.h
> > @@ -27,7 +27,9 @@
> >  #ifndef AVCODEC_MPEGAUDIODECTAB_H
> >  #define AVCODEC_MPEGAUDIODECTAB_H
> >  
> > +#include <stddef.h>
> >  #include <stdint.h>
> > +
> >  #include "mpegaudio.h"
> 
> Unrelated cosmetic change, stddef.h addition OK.

I disagree, this is such a minor change that it is not worth splitting
into a separate commit that then clutters the history.

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

Reply via email to