On Tue, Jan 10, 2017 at 05:33:49PM +0100, Diego Biurrun wrote: > patch proably OK > > On Tue, Jan 10, 2017 at 11:12:07AM +0100, Anton Khirnov wrote: > > --- a/libavcodec/mpegaudiodsp_template.c > > +++ b/libavcodec/mpegaudiodsp_template.c > > @@ -73,7 +73,177 @@ static inline int round_sample(int64_t *sum) > > [MDCT_BUF_SIZE/2, MDCT_BUF_SIZE/2 + 17] are actually used. The rest > > is just to preserve alignment for SIMD implementations. > > */ > > -DECLARE_ALIGNED(16, INTFLOAT, RENAME(ff_mdct_win))[8][MDCT_BUF_SIZE]; > > +// the tables are generated by the following code > > Keeping the comment belonging to the table above the generator code is > a tad confusing. IMO just say > > // the foo and bar tables below are generated by the following code
*were* generated Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
