patch probably OK
On Tue, Jan 10, 2017 at 11:12:08AM +0100, Anton Khirnov wrote:
> --- a/libavcodec/mpegaudiodsp_template.c
> +++ b/libavcodec/mpegaudiodsp_template.c
> @@ -245,7 +245,274 @@ DECLARE_ALIGNED(16, const INTFLOAT,
> RENAME(ff_mdct_win))[8][MDCT_BUF_SIZE] = {
> #endif
> };
>
> -DECLARE_ALIGNED(16, MPA_INT, RENAME(ff_mpa_synth_window))[512+256];
> +// the tables are generated by the following code
The foo and bar tables below were generated by the following code
> +#if 0
> +{
> + int i, j;
I suggest preserving the function name.
> + window[512+128+16*i+j] = window[64*i+48-j];
> +}
> +#endif
> +DECLARE_ALIGNED(16, const MPA_INT, RENAME(ff_mpa_synth_window))[512 + 256] =
> {
An empty line after the #endif would help readability.
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
