On 2/26/15, James Almer <jamr...@gmail.com> wrote:
> On 25/02/15 11:55 AM, Paul B Mahol wrote:
>> +static inline void line_copy8(uint8_t *line, const uint8_t *srcp, int
>> width, int mergin)
>
> Unless I'm missing something, "mergin" seems to always be 1.
> The code below could be simplified.

The function could be used with other arguments for mergin in later patch(es).
>
>> +{
>> +    memcpy(line, srcp, width);
>> +
>> +    for (int i = mergin; i > 0; i--) {
>> +        line[-i] = line[i];
>> +        line[width - 1 + i] = line[width - 1 - i];
>> +    }
>> +}
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to