Browse code

avformat/matroskaenc: add an option to disable writting CRC32 elements

Also add missing Changelog entry.

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

James Almer authored on 2016/10/07 06:17:04
Showing 3 changed files
... ...
@@ -35,6 +35,7 @@ version <next>:
35 35
 - sdl1 output device and sdl1 support removed
36 36
 - extended mov edit list support
37 37
 - libfaac encoder removed
38
+- Matroska muxer now writes CRC32 elements by default in all Level 1 elements
38 39
 
39 40
 
40 41
 version 3.1:
... ...
@@ -134,6 +134,7 @@ typedef struct MatroskaMuxContext {
134 134
     int is_dash;
135 135
     int dash_track_number;
136 136
     int is_live;
137
+    int write_crc;
137 138
 
138 139
     uint32_t chapter_id_offset;
139 140
     int wrote_chapters;
... ...
@@ -334,7 +335,7 @@ static void end_ebml_master_crc32(AVIOContext *pb, AVIOContext **dyn_cp, Matrosk
334 334
 
335 335
     if (pb->seekable) {
336 336
         size = avio_close_dyn_buf(*dyn_cp, &buf);
337
-        if (mkv->mode != MODE_WEBM) {
337
+        if (mkv->write_crc && mkv->mode != MODE_WEBM) {
338 338
             AV_WL32(crc, av_crc(av_crc_get_table(AV_CRC_32_IEEE_LE), UINT32_MAX, buf, size) ^ UINT32_MAX);
339 339
             put_ebml_binary(pb, EBML_ID_CRC32, crc, sizeof(crc));
340 340
         }
... ...
@@ -1485,7 +1486,7 @@ static int mkv_write_tags(AVFormatContext *s)
1485 1485
 
1486 1486
     if (mkv->tags.pos) {
1487 1487
         if (s->pb->seekable && !mkv->is_live)
1488
-            put_ebml_void(s->pb, avio_tell(mkv->tags_bc) + ((mkv->mode != MODE_WEBM) ? 2 /* ebml id + data size */ + 4 /* CRC32 */ : 0));
1488
+            put_ebml_void(s->pb, avio_tell(mkv->tags_bc) + ((mkv->write_crc && mkv->mode != MODE_WEBM) ? 2 /* ebml id + data size */ + 4 /* CRC32 */ : 0));
1489 1489
         else
1490 1490
             end_ebml_master_crc32(s->pb, &mkv->tags_bc, mkv, mkv->tags);
1491 1491
     }
... ...
@@ -1729,7 +1730,7 @@ static int mkv_write_header(AVFormatContext *s)
1729 1729
         }
1730 1730
     }
1731 1731
     if (s->pb->seekable)
1732
-        put_ebml_void(s->pb, avio_tell(pb) + ((mkv->mode != MODE_WEBM) ? 2 /* ebml id + data size */ + 4 /* CRC32 */ : 0));
1732
+        put_ebml_void(s->pb, avio_tell(pb) + ((mkv->write_crc && mkv->mode != MODE_WEBM) ? 2 /* ebml id + data size */ + 4 /* CRC32 */ : 0));
1733 1733
     else
1734 1734
         end_ebml_master_crc32(s->pb, &mkv->info_bc, mkv, mkv->info);
1735 1735
     pb = s->pb;
... ...
@@ -2390,6 +2391,7 @@ static const AVOption options[] = {
2390 2390
     { "dash_track_number", "Track number for the DASH stream", OFFSET(dash_track_number), AV_OPT_TYPE_INT, { .i64 = 1 }, 0, 127, FLAGS },
2391 2391
     { "live", "Write files assuming it is a live stream.", OFFSET(is_live), AV_OPT_TYPE_BOOL, { .i64 = 0 }, 0, 1, FLAGS },
2392 2392
     { "allow_raw_vfw", "allow RAW VFW mode", OFFSET(allow_raw_vfw), AV_OPT_TYPE_BOOL, { .i64 = 0 }, 0, 1, FLAGS },
2393
+    { "write_crc32", "write a CRC32 element inside every Level 1 element", OFFSET(write_crc), AV_OPT_TYPE_BOOL, { .i64 = 1 }, 0, 1, FLAGS },
2393 2394
     { NULL },
2394 2395
 };
2395 2396
 
... ...
@@ -33,7 +33,7 @@
33 33
 // Also please add any ticket numbers that you believe might be affected here
34 34
 #define LIBAVFORMAT_VERSION_MAJOR  57
35 35
 #define LIBAVFORMAT_VERSION_MINOR  51
36
-#define LIBAVFORMAT_VERSION_MICRO 102
36
+#define LIBAVFORMAT_VERSION_MICRO 103
37 37
 
38 38
 #define LIBAVFORMAT_VERSION_INT AV_VERSION_INT(LIBAVFORMAT_VERSION_MAJOR, \
39 39
                                                LIBAVFORMAT_VERSION_MINOR, \