On Mon, Mar 12, 2018 at 14:38:21 -0800, Lou Logan wrote:

> But you don't necessarily need to make a new patch to address the
> minor whitespace issue. You can wait for other comments and include
> it with any other requested changes.

Another whitespace nit:

>  if (frame->pict_type==AV_PICTURE_TYPE_I) {

Please use single spaces around operators, i.e.
  if (frame->pict_type == AV_PICTURE_TYPE_I) {

Moritz
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to