[FFmpeg-devel] [PATCH] avformat:matroskadec: use a define to mark the EBML length is unknown

Steve Lhomme robux4 at ycbcr.xyz
Sun Feb 24 10:23:48 EET 2019


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 <robux4 at 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 <robux4 at ycbcr.xyz>
Co-authored-by: Dale Curtis <dalecurtis at chromium.org>

"


More information about the ffmpeg-devel mailing list