[FFmpeg-devel] [PATCH 2/5] lavf/concatdec: check match_streams() return value.

Nicolas George george at nsup.org
Fri Apr 25 10:40:59 CEST 2014


Le tridi 3 floréal, an CCXXII, Michael Niedermayer a écrit :
> LGTM, but you are the author so you should know all this better
> than me. In that same sense IMO the whole patchset is ok

Thanks. For files that I authored or maintain, I send patches so that people
have a little time to see what is coming (including conflicts with their own
work-in-progress), to spot obvious mistakes, to suggest better solutions,
etc. Only if they have time and are interested. If I see no reply, I will
wait a few days / for the other patches to be approved and push or ask you
to merge.

And of course, when the patch adds a user-visible feature (like 5/5 in this
series), feedback about the feature itself is as important as feedback about
the implementation.

For this series, I will wait some more time in case someone has useful and
accurate explanations about the MP4 / AnnexB mess before requesting a merge.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140425/ed79b44a/attachment.asc>


More information about the ffmpeg-devel mailing list