[FFmpeg-devel] [PATCH 03/37] avformat/matroskadec: Compactify structure

Andreas Rheinhardt andreas.rheinhardt at gmail.com
Fri May 17 01:29:47 EEST 2019


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>
---
 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;
 
-- 
2.21.0



More information about the ffmpeg-devel mailing list