On Mon, Oct 03, 2011 at 11:37:35AM -0700, Ronald S. Bultje wrote: > From: "Ronald S. Bultje" <[email protected]> > > --- a/libavcodec/dsputil.c > +++ b/libavcodec/dsputil.c > @@ -145,6 +145,41 @@ void ff_init_scantable(uint8_t *permutation, ScanTable > *st, const uint8_t *src_s > > +void ff_init_scantable_permutation(uint8_t *idct_permutation, > + int idct_permutation_type) > +{ > + int i; > + > + switch(idct_permutation_type){ > + case FF_NO_IDCT_PERM: > + for(i=0; i<64; i++) > + idct_permutation[i]= i; > + break; > @@ -3123,32 +3158,6 @@ av_cold void dsputil_init(DSPContext* c, > AVCodecContext *avctx) > > - switch(c->idct_permutation_type){ > - case FF_NO_IDCT_PERM: > - for(i=0; i<64; i++) > - c->idct_permutation[i]= i; > - break;
Please maintain K&R style and keep the case statements at the indentation level as the switch. > --- /dev/null > +++ b/libavcodec/proresdsp.h > @@ -0,0 +1,38 @@ > + > +#ifndef LIBAVCODEC_PRORESDSP_H > +#define LIBAVCODEC_PRORESDSP_H AVCODEC_PRORESDSP_H > +#define PRORES_BITS_PER_SAMPLE 10 ///< output precision of that decoder "that"? Just write "prores"... Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
