1
0
mirror of https://github.com/FFmpeg/FFmpeg.git synced 2024-12-28 20:53:54 +02:00

avformat/movenc: Refactor mov_write_dvcc_dvvc_tag to use ff_isom_put_dvcc_dvvc

Improves code legibility by not using bit shifts.
Also avoids duplicating the dvcC/dvvC ISOM box writing code.

Signed-off-by: quietvoid <tcChlisop0@gmail.com>
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
This commit is contained in:
quietvoid 2022-01-01 17:51:52 +01:00 committed by Andreas Rheinhardt
parent 9906f9ae3b
commit f0fb005f59
2 changed files with 8 additions and 17 deletions

View File

@ -342,7 +342,8 @@ OBJS-$(CONFIG_MOV_DEMUXER) += mov.o mov_chan.o mov_esds.o \
qtpalette.o replaygain.o dovi_isom.o qtpalette.o replaygain.o dovi_isom.o
OBJS-$(CONFIG_MOV_MUXER) += movenc.o av1.o avc.o hevc.o vpcc.o \ OBJS-$(CONFIG_MOV_MUXER) += movenc.o av1.o avc.o hevc.o vpcc.o \
movenchint.o mov_chan.o rtp.o \ movenchint.o mov_chan.o rtp.o \
movenccenc.o movenc_ttml.o rawutils.o movenccenc.o movenc_ttml.o rawutils.o \
dovi_isom.o
OBJS-$(CONFIG_MP2_MUXER) += rawenc.o OBJS-$(CONFIG_MP2_MUXER) += rawenc.o
OBJS-$(CONFIG_MP3_DEMUXER) += mp3dec.o replaygain.o OBJS-$(CONFIG_MP3_DEMUXER) += mp3dec.o replaygain.o
OBJS-$(CONFIG_MP3_MUXER) += mp3enc.o rawenc.o id3v2enc.o OBJS-$(CONFIG_MP3_MUXER) += mp3enc.o rawenc.o id3v2enc.o

View File

@ -27,6 +27,7 @@
#include "movenc.h" #include "movenc.h"
#include "avformat.h" #include "avformat.h"
#include "avio_internal.h" #include "avio_internal.h"
#include "dovi_isom.h"
#include "riff.h" #include "riff.h"
#include "avio.h" #include "avio.h"
#include "isom.h" #include "isom.h"
@ -1911,6 +1912,8 @@ static int mov_write_sv3d_tag(AVFormatContext *s, AVIOContext *pb, AVSphericalMa
static int mov_write_dvcc_dvvc_tag(AVFormatContext *s, AVIOContext *pb, AVDOVIDecoderConfigurationRecord *dovi) static int mov_write_dvcc_dvvc_tag(AVFormatContext *s, AVIOContext *pb, AVDOVIDecoderConfigurationRecord *dovi)
{ {
uint8_t buf[ISOM_DVCC_DVVC_SIZE];
avio_wb32(pb, 32); /* size = 8 + 24 */ avio_wb32(pb, 32); /* size = 8 + 24 */
if (dovi->dv_profile > 10) if (dovi->dv_profile > 10)
ffio_wfourcc(pb, "dvwC"); ffio_wfourcc(pb, "dvwC");
@ -1918,23 +1921,10 @@ static int mov_write_dvcc_dvvc_tag(AVFormatContext *s, AVIOContext *pb, AVDOVIDe
ffio_wfourcc(pb, "dvvC"); ffio_wfourcc(pb, "dvvC");
else else
ffio_wfourcc(pb, "dvcC"); ffio_wfourcc(pb, "dvcC");
avio_w8(pb, dovi->dv_version_major);
avio_w8(pb, dovi->dv_version_minor);
avio_wb16(pb, (dovi->dv_profile << 9) | (dovi->dv_level << 3) |
(dovi->rpu_present_flag << 2) | (dovi->el_present_flag << 1) |
dovi->bl_present_flag);
avio_wb32(pb, (dovi->dv_bl_signal_compatibility_id << 28) | 0);
ffio_fill(pb, 0, 4 * 4); /* reserved */ ff_isom_put_dvcc_dvvc(s, buf, dovi);
av_log(s, AV_LOG_DEBUG, "DOVI in %s box, version: %d.%d, profile: %d, level: %d, " avio_write(pb, buf, sizeof(buf));
"rpu flag: %d, el flag: %d, bl flag: %d, compatibility id: %d\n",
dovi->dv_profile > 10 ? "dvwC" : (dovi->dv_profile > 7 ? "dvvC" : "dvcC"),
dovi->dv_version_major, dovi->dv_version_minor,
dovi->dv_profile, dovi->dv_level,
dovi->rpu_present_flag,
dovi->el_present_flag,
dovi->bl_present_flag,
dovi->dv_bl_signal_compatibility_id);
return 32; /* 8 + 24 */ return 32; /* 8 + 24 */
} }