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>

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

Reply via email to