2017-01-10 19:45 GMT+01:00 Gonzalo Garramuño <ggarr...@gmail.com>:
>
>
> El 10/01/2017 a las 10:21, Carl Eugen Hoyos escribió:
>>
>>           }
>> +    } else if (avctx->pix_fmt == AV_PIX_FMT_MONOWHITE) {
>> +        ptr = picture->data[0];
>> +        for (y = 0; y < s->height; y++) {
>> +            memcpy(ptr, ptr_data, s->width + 7 >> 3);
>> +            ptr_data += s->width + 7 >> 3;
>> +            ptr += picture->linesize[0];
>> +        }
>
> Just a nitpick.  I would put s->width + 7 >> 3 in a variable outside the
> loop and use it in memcpy and in the ptr_data.

Fixed locally.

Thank you, Carl Eugen
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to