FFmpeg

117687 Commits

37 Branches

400 Tags

492 MiB

Tag: Branch: Tree: 0cdcbab9e9

Commit Graph

Author	SHA1	Message	Date
Andreas Rheinhardt	d53acf452f	avformat/matroskaenc: Don't write \0 unnecessarily Writing the duration SimpleTag is special: It's size is reserved in advance via an EBML Void element (if seekable) and this reserved space is overwritten when writing the trailer; it does not use put_ebml_string(). The string to write is created via snprintf on a buffer of size 20; this buffer is then written via put_ebml_binary() with a size of 20. EBML strings need not be zero-terminated; if not, they are implicitly terminated by the element's length field. snprintf() always zero-terminates the buffer, i.e. the last byte can be discarded when using an EBML string. This patch does this. The FATE changes are as expected: One byte saved for every track; the only exception is the matroska-qt-mode test: An additional byte is saved because an additional byte could be saved from the enclosing Tags length field. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>	2 years ago
James Almer	30f1f89572	fate/matroska: add HDR10+ muxing tests Signed-off-by: James Almer <jamrial@gmail.com>	2 years ago
Mohammad Izadi	89e3f5abb7	fate: add a test for HDR10+ metadata in HEVC Signed-off-by: James Almer <jamrial@gmail.com>	4 years ago

Author

SHA1

Message

Date

Andreas Rheinhardt

d53acf452f

avformat/matroskaenc: Don't write \0 unnecessarily

Writing the duration SimpleTag is special: It's size is
reserved in advance via an EBML Void element (if seekable)
and this reserved space is overwritten when writing the trailer;
it does not use put_ebml_string().

The string to write is created via snprintf on a buffer
of size 20; this buffer is then written via put_ebml_binary()
with a size of 20.

EBML strings need not be zero-terminated; if not, they
are implicitly terminated by the element's length field.
snprintf() always zero-terminates the buffer, i.e.
the last byte can be discarded when using an EBML string.
This patch does this.

The FATE changes are as expected: One byte saved for every
track; the only exception is the matroska-qt-mode test:
An additional byte is saved because an additional byte
could be saved from the enclosing Tags length field.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>

2 years ago

James Almer

30f1f89572

fate/matroska: add HDR10+ muxing tests

Signed-off-by: James Almer <jamrial@gmail.com>

2 years ago

Mohammad Izadi

89e3f5abb7

fate: add a test for HDR10+ metadata in HEVC

Signed-off-by: James Almer <jamrial@gmail.com>

4 years ago

2 Commits (0cdcbab9e9184dc63b9c00e418ff10f88df0f060)