[FFmpeg-devel] [FFmpeg-cvslog] avcodec: add ARBC decoder

James Almer jamrial at gmail.com
Sun Jan 27 16:15:19 EET 2019


On 1/27/2019 10:06 AM, Paul B Mahol wrote:
> On 1/27/19, Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:
>> 2019-01-27 13:55 GMT+01:00, Paul B Mahol <git at videolan.org>:
>>
>>> +                    frame->data[0][frame->linesize[0] * (h - j) + 3 * k +
>>> 0] = color[0];
>>> +                    frame->data[0][frame->linesize[0] * (h - j) + 3 * k +
>>> 1] = color[1];
>>> +                    frame->data[0][frame->linesize[0] * (h - j) + 3 * k +
>>> 2] = color[2];
>>
>> Isn't this AV_Wx24()?
>>
>> Carl Eugen
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at ffmpeg.org
>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
> 
> I do not do micro optimizations.

Don't be ridiculous. intreadwrite macros not only make sure the most
optimized way to read and write bytes is used, but also make the code
much more readable. And you know it, because you've have used them
plenty of times.


More information about the ffmpeg-devel mailing list