[FFmpeg-devel] [PATCH 0/3] avformat/movenc: refactor to avoid code duplication
barsnick at gmx.net
Wed Sep 19 16:58:29 EEST 2018
While mov_write_tmpo_tag() had code to write a 16-bit integer atom,
mov_write_int8_metadata() had code to write 8-bit and 32-bit integer atoms
with an 8-bit value. Both functions were extremely similar though.
Consolidated the two in three steps, preserving mov_write_int8_metadata()'s
constraints (that it only writes a uint8_t value). The patches could
obviously be squashed.
OPEN: The herewith introduced compatability macro mov_write_int8_metadata()
could be dropped in favor of using the replacement function directly.
OPEN: No sure whether the metadata key "tmpo" is the (only) proper one
for use of this atom.
indicates it might be "bpm", as does
That would be a subsequent patch adding
mov_write_int_metadata (s, pb, "tmpo", "bpm" , 2, 2);
as an additional translation (at the risk of duplicating the atom).
Otherwise: where in the libs or the application are metadata keys
Moritz Barsnick (3):
avformat/movenc: allow padding to two bytes in
avformat/movenc: add mov_write_int_metadata()
avformat/movenc: use mov_write_int_metadata() in place of
libavformat/movenc.c | 48 +++++++++++++++++++-----------------------------
1 file changed, 19 insertions(+), 29 deletions(-)
More information about the ffmpeg-devel