History

avformat/matroskaenc: Don't waste bytes to Write Tag length fields

This is possible by using a dynamic buffer to write them; said dynamic buffer is (re)used and reset as appropriate.

Andreas Rheinhardt committed 4 years ago

Commit b468ddc1 parent 58df81b26 files · +131 −122Browse files

Showing the changes to libavformat (1 of 26 files). Show all

Unified · Split

libavformat/matroskaenc.c+61−52
@@ -216,6 +216,13 @@ typedef struct MatroskaMuxContext { · 215 unmodified lines
216216
217217 BlockContext cur_block;
218218
219+ /* Used as temporary buffer to use the minimal amount of bytes
220+ * to write the length field of EBML Masters.
221+ * Every user has to reset the buffer after using it and
222+ * different uses may not overlap. It is currently used in
223+ * mkv_write_tag(). */
224+ AVIOContext *tmp_bc;
225+
219226 AVPacket *cur_audio_pkt;
220227
221228 unsigned nb_attachments;
@@ -247,6 +254,9 @@ typedef struct MatroskaMuxContext { · 25 unmodified lines
247254 /** 4 * (1-byte EBML ID, 1-byte EBML size, 8-byte uint max) */
248255 #define MAX_CUETRACKPOS_SIZE 40
249256
257+/** 2 + 1 Simpletag header, 2 + 1 + 8 Name "DURATION", 23B for TagString */
258+#define DURATION_SIMPLETAG_SIZE (2 + 1 + (2 + 1 + 8) + 23)
259+
250260 /** Seek preroll value for opus */
251261 #define OPUS_SEEK_PREROLL 80000000
252262
@@ -824,6 +834,7 @@ static void mkv_deinit(AVFormatContext *s) · 571 unmodified lines
824834 ffio_free_dyn_buf(&mkv->info.bc);
825835 ffio_free_dyn_buf(&mkv->track.bc);
826836 ffio_free_dyn_buf(&mkv->tags.bc);
837+ ffio_free_dyn_buf(&mkv->tmp_bc);
827838
828839 av_freep(&mkv->cur_block.h2645_nalu_list.nalus);
829840 av_freep(&mkv->cues.entries);
@@ -1921,24 +1932,14 @@ static int mkv_write_simpletag(AVIOContext *pb, const AVDictionaryEntry *t) · 1091 unmodified lines
19211932 return ret;
19221933 }
19231934
1924−static int mkv_write_tag_targets(MatroskaMuxContext *mkv, AVIOContext **pb,
1925− ebml_master *tag, uint32_t elementid, uint64_t uid)
1935+static void mkv_write_tag_targets(MatroskaMuxContext *mkv, AVIOContext *pb,
1936+ uint32_t elementid, uint64_t uid)
19261937 {
1927− ebml_master targets;
1928− int ret;
1929−
1930− if (!*pb) {
1931− ret = start_ebml_master_crc32(pb, mkv);
1932− if (ret < 0)
1933− return ret;
1934− }
1935−
1936− *tag = start_ebml_master(*pb, MATROSKA_ID_TAG, 0);
1937− targets = start_ebml_master(*pb, MATROSKA_ID_TAGTARGETS, 4 + 1 + 8);
1938+ ebml_master targets = start_ebml_master(pb, MATROSKA_ID_TAGTARGETS,
1939+ 4 + 1 + 8);
19381940 if (elementid)
1939− put_ebml_uid(*pb, elementid, uid);
1940− end_ebml_master(*pb, targets);
1941− return 0;
1941+ put_ebml_uid(pb, elementid, uid);
1942+ end_ebml_master(pb, targets);
19421943 }
19431944
19441945 static int mkv_check_tag_name(const char *name, uint32_t elementid)
@@ -1956,29 +1957,41 @@ static int mkv_check_tag_name(const char *name, uint32_t elementid) · 11 unmodified lines
19561957 }
19571958
19581959 static int mkv_write_tag(MatroskaMuxContext *mkv, const AVDictionary *m,
1959− AVIOContext **pb, ebml_master *tag,
1960+ AVIOContext **pb, unsigned reserved_size,
19601961 uint32_t elementid, uint64_t uid)
19611962 {
19621963 const AVDictionaryEntry *t = NULL;
1963− ebml_master tag2;
1964− int ret;
1964+ AVIOContext *const tmp_bc = mkv->tmp_bc;
1965+ uint8_t *buf;
1966+ int ret, size;
19651967
1966− ret = mkv_write_tag_targets(mkv, pb, tag ? tag : &tag2, elementid, uid);
1967− if (ret < 0)
1968− return ret;
1968+ mkv_write_tag_targets(mkv, tmp_bc, elementid, uid);
19691969
19701970 while ((t = av_dict_get(m, "", t, AV_DICT_IGNORE_SUFFIX))) {
19711971 if (mkv_check_tag_name(t->key, elementid)) {
1972− ret = mkv_write_simpletag(*pb, t);
1972+ ret = mkv_write_simpletag(tmp_bc, t);
19731973 if (ret < 0)
1974− return ret;
1974+ goto end;
19751975 }
19761976 }
1977+ if (reserved_size)
1978+ put_ebml_void(tmp_bc, reserved_size);
19771979
1978− if (!tag)
1979− end_ebml_master(*pb, tag2);
1980+ size = avio_get_dyn_buf(tmp_bc, &buf);
1981+ if (tmp_bc->error) {
1982+ ret = tmp_bc->error;
1983+ goto end;
1984+ }
1985+ if (!*pb) {
1986+ ret = start_ebml_master_crc32(pb, mkv);
1987+ if (ret < 0)
1988+ goto end;
1989+ }
1990+ put_ebml_binary(*pb, MATROSKA_ID_TAG, buf, size);
19801991
1981− return 0;
1992+end:
1993+ ffio_reset_dyn_buf(tmp_bc);
1994+ return ret;
19821995 }
19831996
19841997 static int mkv_check_tag(const AVDictionary *m, uint32_t elementid)
@@ -1995,15 +2008,14 @@ static int mkv_check_tag(const AVDictionary *m, uint32_t elementid) · 10 unmodified lines
19952008 static int mkv_write_tags(AVFormatContext *s)
19962009 {
19972010 MatroskaMuxContext *mkv = s->priv_data;
1998− ebml_master tag, *tagp = IS_SEEKABLE(s->pb, mkv) ? &tag : NULL;
1999− int i, ret;
2011+ int i, ret, seekable = IS_SEEKABLE(s->pb, mkv);
20002012
20012013 mkv->wrote_tags = 1;
20022014
20032015 ff_metadata_conv_ctx(s, ff_mkv_metadata_conv, NULL);
20042016
20052017 if (mkv_check_tag(s->metadata, 0)) {
2006− ret = mkv_write_tag(mkv, s->metadata, &mkv->tags.bc, NULL, 0, 0);
2018+ ret = mkv_write_tag(mkv, s->metadata, &mkv->tags.bc, 0, 0, 0);
20072019 if (ret < 0)
20082020 return ret;
20092021 }
@@ -2015,29 +2027,16 @@ static int mkv_write_tags(AVFormatContext *s) · 5 unmodified lines
20152027 if (st->codecpar->codec_type == AVMEDIA_TYPE_ATTACHMENT)
20162028 continue;
20172029
2018− if (!tagp && !mkv_check_tag(st->metadata, MATROSKA_ID_TAGTARGETS_TRACKUID))
2030+ if (!seekable && !mkv_check_tag(st->metadata, MATROSKA_ID_TAGTARGETS_TRACKUID))
20192031 continue;
20202032
2021− ret = mkv_write_tag(mkv, st->metadata, &mkv->tags.bc, tagp,
2033+ ret = mkv_write_tag(mkv, st->metadata, &mkv->tags.bc,
2034+ seekable ? DURATION_SIMPLETAG_SIZE : 0,
20222035 MATROSKA_ID_TAGTARGETS_TRACKUID, track->uid);
20232036 if (ret < 0)
20242037 return ret;
2025−
2026− if (tagp) {
2027− AVIOContext *pb = mkv->tags.bc;
2028− ebml_master simpletag;
2029−
2030− simpletag = start_ebml_master(pb, MATROSKA_ID_SIMPLETAG,
2031− 2 + 1 + 8 + 23);
2032− put_ebml_string(pb, MATROSKA_ID_TAGNAME, "DURATION");
2033− track->duration_offset = avio_tell(pb);
2034−
2035− // Reserve space to write duration as a 20-byte string.
2036− // 2 (ebml id) + 1 (data size) + 20 (data)
2037− put_ebml_void(pb, 23);
2038− end_ebml_master(pb, simpletag);
2039− end_ebml_master(pb, tag);
2040− }
2038+ if (seekable)
2039+ track->duration_offset = avio_tell(mkv->tags.bc) - DURATION_SIMPLETAG_SIZE;
20412040 }
20422041
20432042 if (mkv->nb_attachments && !IS_WEBM(mkv)) {
@@ -2051,7 +2050,7 @@ static int mkv_write_tags(AVFormatContext *s) · 7 unmodified lines
20512050 if (!mkv_check_tag(st->metadata, MATROSKA_ID_TAGTARGETS_ATTACHUID))
20522051 continue;
20532052
2054− ret = mkv_write_tag(mkv, st->metadata, &mkv->tags.bc, NULL,
2053+ ret = mkv_write_tag(mkv, st->metadata, &mkv->tags.bc, 0,
20552054 MATROSKA_ID_TAGTARGETS_ATTACHUID, track->uid);
20562055 if (ret < 0)
20572056 return ret;
@@ -2136,7 +2135,7 @@ static int mkv_write_chapters(AVFormatContext *s) · 78 unmodified lines
21362135 ff_metadata_conv(&c->metadata, ff_mkv_metadata_conv, NULL);
21372136
21382137 if (mkv_check_tag(c->metadata, MATROSKA_ID_TAGTARGETS_CHAPTERUID)) {
2139− ret = mkv_write_tag(mkv, c->metadata, tags, NULL,
2138+ ret = mkv_write_tag(mkv, c->metadata, tags, 0,
21402139 MATROSKA_ID_TAGTARGETS_CHAPTERUID, uid);
21412140 if (ret < 0)
21422141 goto fail;
@@ -2336,6 +2335,10 @@ static int mkv_write_header(AVFormatContext *s) · 193 unmodified lines
23362335 AVIOContext *pb = s->pb;
23372336 int ret, version = 2;
23382337
2338+ ret = avio_open_dyn_buf(&mkv->tmp_bc);
2339+ if (ret < 0)
2340+ return ret;
2341+
23392342 if (!IS_WEBM(mkv) ||
23402343 av_dict_get(s->metadata, "stereo_mode", NULL, 0) ||
23412344 av_dict_get(s->metadata, "alpha_mode", NULL, 0))
@@ -3010,6 +3013,7 @@ after_cues: · 668 unmodified lines
30103013
30113014 // update stream durations
30123015 if (mkv->tags.bc) {
3016+ AVIOContext *tags_bc = mkv->tags.bc;
30133017 int i;
30143018 for (i = 0; i < s->nb_streams; ++i) {
30153019 const AVStream *st = s->streams[i];
@@ -3018,17 +3022,22 @@ after_cues: · 2 unmodified lines
30183022 if (track->duration_offset > 0) {
30193023 double duration_sec = track->duration * av_q2d(st->time_base);
30203024 char duration_string[20] = "";
3025+ ebml_master simpletag;
30213026
30223027 av_log(s, AV_LOG_DEBUG, "stream %d end duration = %" PRIu64 "\n", i,
30233028 track->duration);
30243029
3025− avio_seek(mkv->tags.bc, track->duration_offset, SEEK_SET);
3030+ avio_seek(tags_bc, track->duration_offset, SEEK_SET);
3031+ simpletag = start_ebml_master(tags_bc, MATROSKA_ID_SIMPLETAG,
3032+ 2 + 1 + 8 + 23);
3033+ put_ebml_string(tags_bc, MATROSKA_ID_TAGNAME, "DURATION");
30263034
30273035 snprintf(duration_string, 20, "%02d:%02d:%012.9f",
30283036 (int) duration_sec / 3600, ((int) duration_sec / 60) % 60,
30293037 fmod(duration_sec, 60));
30303038
3031− put_ebml_binary(mkv->tags.bc, MATROSKA_ID_TAGSTRING, duration_string, 20);
3039+ put_ebml_binary(tags_bc, MATROSKA_ID_TAGSTRING, duration_string, 20);
3040+ end_ebml_master(tags_bc, simpletag);
30323041 }
30333042 }
30343043

Keyboard shortcuts

General

Show keyboard shortcuts
?
Focus the current search
/

Issues and pull requests

Open filters
f

Issues, pull requests and Inbox

Select next item
j↓
Select previous item
k↑
Open selected item
Enter
Dismiss filters or clear selection
Esc

Repository code

Go to file
t