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

James Darnley jdarnley at obe.tv
Tue Jun 20 14:42:53 EEST 2017


On 2017-06-19 20:30, Ronald S. Bultje wrote:
> 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

Pushed, thanks.



More information about the ffmpeg-devel mailing list