[FFmpeg-cvslog] avformat/matroskadec: Compactify structure

Andreas Rheinhardt git at videolan.org
Fri Jun 7 22:51:31 EEST 2019

ffmpeg | branch: master | Andreas Rheinhardt <andreas.rheinhardt at gmail.com> | Fri May 17 00:29:47 2019 +0200| [410a0824f07ac4a526f633409cf893a897d2269c] | committer: Michael Niedermayer

avformat/matroskadec: Compactify structure

Matroska EBML IDs can be only four bytes long maximally, so it is
natural to use uint32_t for them. By doing this and rearranging the
elements of the MatroskaLevel1Element structure, one can reduce the size
of said structure.

Notice that this field is not read via the generic reading process for
EBML_UINT, so one is not forced to use an uint64_t for it.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=410a0824f07ac4a526f633409cf893a897d2269c

 libavformat/matroskadec.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c
index 6da9b15d79..3b8ddc5ecb 100644
--- a/libavformat/matroskadec.c
+++ b/libavformat/matroskadec.c
@@ -310,8 +310,8 @@ typedef struct MatroskaCluster {
 } MatroskaCluster;
 typedef struct MatroskaLevel1Element {
-    uint64_t id;
     uint64_t pos;
+    uint32_t id;
     int parsed;
 } MatroskaLevel1Element;

More information about the ffmpeg-cvslog mailing list