[FFmpeg-devel] [PATCH 06/11] avcodec/x86: cleanup simple_idct10

[Ronald S. Bultje]

Hi,

On Mon, Jun 19, 2017 at 11:10 AM, James Darnley <jdarnley at obe.tv> wrote:

> Use named arguments for the functions so we can remove a define.  The
> stride/linesize argument is now ptrdiff_t type so we no longer need to
> sign extend the register.
> ---
>  libavcodec/x86/proresdsp.asm              |  2 +-
>  libavcodec/x86/simple_idct10.asm          |  8 ++--
>  libavcodec/x86/simple_idct10_template.asm | 80
> ++++++++++++++-----------------
>  3 files changed, 41 insertions(+), 49 deletions(-)


LGTM.

Ronald