On Sun, Feb 24, 2019 at 09:23:48AM +0100, Steve Lhomme wrote:
> On 23/02/2019 19:37, Michael Niedermayer wrote:
> >On Sat, Feb 23, 2019 at 11:14:33AM +0100, Steve Lhomme wrote:
> >>From: Steve Lhomme <rob...@ycbcr.xyz>
> >>
> >>---
> >>  libavformat/matroskadec.c | 12 +++++++-----
> >>  1 file changed, 7 insertions(+), 5 deletions(-)
> >I think the commit message is a bit terse. This is not just a cosmetic
> >change (which one might think from reading just the commit message and
> >not looking at the change or related mails)
> >
> >Can you provide a more verbose commit message ?
> >ill apply it with that
> >
> >thanks
> >
> >[...]
> 
> How about this ?
> 
> "
> avformat:matroskadec: use a define to mark the EBML length is unknown
> 
> Unifying the way the EBML unknown length is signaled, rather than using two
> incompatible values. UINT64_MAX cannot be read as a valid EBML length with
> the
> current code.
> 
> Co-authored-by: Steve Lhomme <rob...@ycbcr.xyz>
> Co-authored-by: Dale Curtis <dalecur...@chromium.org>
> 
> "

will apply with this as commit message

thanks

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus

Attachment: signature.asc
Description: PGP signature

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

Reply via email to