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

2017-06-20 Thread James Darnley
On 2017-06-19 20:30, Ronald S. Bultje wrote: > Hi, > > On Mon, Jun 19, 2017 at 11:10 AM, James Darnley 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. >> --- >

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

2017-06-19 Thread Ronald S. Bultje
Hi, On Mon, Jun 19, 2017 at 11:10 AM, James Darnley 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 +- > li