[FFmpeg-devel] [PATCH] avformat/movenc: rename MOVMuxContext.nb_streams to nb_tracks
James Almer
jamrial at gmail.com
Fri Dec 8 01:41:22 EET 2023
It's more appropiate given what it keeps a count of.
Signed-off-by: James Almer <jamrial at gmail.com>
---
libavformat/movenc.c | 82 ++++++++++++++++++++++----------------------
libavformat/movenc.h | 2 +-
2 files changed, 42 insertions(+), 42 deletions(-)
diff --git a/libavformat/movenc.c b/libavformat/movenc.c
index e39f1ac987..a902e4d4e7 100644
--- a/libavformat/movenc.c
+++ b/libavformat/movenc.c
@@ -3836,7 +3836,7 @@ static int mov_write_iods_tag(AVIOContext *pb, MOVMuxContext *mov)
int64_t pos = avio_tell(pb);
int audio_profile = mov->iods_audio_profile;
int video_profile = mov->iods_video_profile;
- for (i = 0; i < mov->nb_streams; i++) {
+ for (i = 0; i < mov->nb_tracks; i++) {
if (mov->tracks[i].entry > 0 || mov->flags & FF_MOV_FLAG_EMPTY_MOOV) {
has_audio |= mov->tracks[i].par->codec_type == AVMEDIA_TYPE_AUDIO;
has_video |= mov->tracks[i].par->codec_type == AVMEDIA_TYPE_VIDEO;
@@ -3878,7 +3878,7 @@ static int mov_write_mvex_tag(AVIOContext *pb, MOVMuxContext *mov)
int i;
avio_wb32(pb, 0x0); /* size */
ffio_wfourcc(pb, "mvex");
- for (i = 0; i < mov->nb_streams; i++)
+ for (i = 0; i < mov->nb_tracks; i++)
mov_write_trex_tag(pb, &mov->tracks[i]);
return update_size(pb, pos);
}
@@ -3890,7 +3890,7 @@ static int mov_write_mvhd_tag(AVIOContext *pb, MOVMuxContext *mov)
int version;
int timescale;
- for (i = 0; i < mov->nb_streams; i++) {
+ for (i = 0; i < mov->nb_tracks; i++) {
if (mov->tracks[i].entry > 0 && mov->tracks[i].timescale) {
int64_t max_track_len_temp = av_rescale_rnd(
calc_pts_duration(mov, &mov->tracks[i]),
@@ -4568,14 +4568,14 @@ static int mov_setup_track_ids(MOVMuxContext *mov, AVFormatContext *s)
next_generated_track_id = s->streams[i]->id;
}
- for (i = 0; i < mov->nb_streams; i++) {
+ for (i = 0; i < mov->nb_tracks; i++) {
if (mov->tracks[i].entry <= 0 && !(mov->flags & FF_MOV_FLAG_FRAGMENT))
continue;
mov->tracks[i].track_id = i >= s->nb_streams ? ++next_generated_track_id : s->streams[i]->id;
}
} else {
- for (i = 0; i < mov->nb_streams; i++) {
+ for (i = 0; i < mov->nb_tracks; i++) {
if (mov->tracks[i].entry <= 0 && !(mov->flags & FF_MOV_FLAG_FRAGMENT))
continue;
@@ -4598,7 +4598,7 @@ static int mov_write_moov_tag(AVIOContext *pb, MOVMuxContext *mov,
mov_setup_track_ids(mov, s);
- for (i = 0; i < mov->nb_streams; i++) {
+ for (i = 0; i < mov->nb_tracks; i++) {
if (mov->tracks[i].entry <= 0 && !(mov->flags & FF_MOV_FLAG_FRAGMENT))
continue;
@@ -4613,7 +4613,7 @@ static int mov_write_moov_tag(AVIOContext *pb, MOVMuxContext *mov,
mov->tracks[i].tref_tag = MKTAG('c','h','a','p');
mov->tracks[i].tref_id = mov->tracks[mov->chapter_track].track_id;
}
- for (i = 0; i < mov->nb_streams; i++) {
+ for (i = 0; i < mov->nb_tracks; i++) {
MOVTrack *track = &mov->tracks[i];
if (track->tag == MKTAG('r','t','p',' ')) {
track->tref_tag = MKTAG('h','i','n','t');
@@ -4624,14 +4624,14 @@ static int mov_write_moov_tag(AVIOContext *pb, MOVMuxContext *mov,
AV_PKT_DATA_FALLBACK_TRACK );
if (sd && sd->size == sizeof(int)) {
int *fallback = (int *)sd->data;
- if (*fallback >= 0 && *fallback < mov->nb_streams) {
+ if (*fallback >= 0 && *fallback < mov->nb_tracks) {
track->tref_tag = MKTAG('f','a','l','l');
track->tref_id = mov->tracks[*fallback].track_id;
}
}
}
}
- for (i = 0; i < mov->nb_streams; i++) {
+ for (i = 0; i < mov->nb_tracks; i++) {
if (mov->tracks[i].tag == MKTAG('t','m','c','d')) {
int src_trk = mov->tracks[i].src_track;
mov->tracks[src_trk].tref_tag = mov->tracks[i].tag;
@@ -4646,7 +4646,7 @@ static int mov_write_moov_tag(AVIOContext *pb, MOVMuxContext *mov,
mov_write_mvhd_tag(pb, mov);
if (mov->mode != MODE_MOV && mov->mode != MODE_AVIF && !mov->iods_skip)
mov_write_iods_tag(pb, mov);
- for (i = 0; i < mov->nb_streams; i++) {
+ for (i = 0; i < mov->nb_tracks; i++) {
if (mov->tracks[i].entry > 0 || mov->flags & FF_MOV_FLAG_FRAGMENT ||
mov->mode == MODE_AVIF) {
int ret = mov_write_trak_tag(s, pb, mov, &(mov->tracks[i]), i < s->nb_streams ? s->streams[i] : NULL);
@@ -4710,7 +4710,7 @@ static int mov_write_isml_manifest(AVIOContext *pb, MOVMuxContext *mov, AVFormat
mov_setup_track_ids(mov, s);
- for (i = 0; i < mov->nb_streams; i++) {
+ for (i = 0; i < mov->nb_tracks; i++) {
MOVTrack *track = &mov->tracks[i];
struct mpeg4_bit_rate_values bit_rates =
calculate_mpeg4_bit_rates(track);
@@ -5017,7 +5017,7 @@ static int mov_add_tfra_entries(AVIOContext *pb, MOVMuxContext *mov, int tracks,
int size)
{
int i;
- for (i = 0; i < mov->nb_streams; i++) {
+ for (i = 0; i < mov->nb_tracks; i++) {
MOVTrack *track = &mov->tracks[i];
MOVFragmentInfo *info;
if ((tracks >= 0 && i != tracks) || !track->entry)
@@ -5055,7 +5055,7 @@ static int mov_add_tfra_entries(AVIOContext *pb, MOVMuxContext *mov, int tracks,
static void mov_prune_frag_info(MOVMuxContext *mov, int tracks, int max)
{
int i;
- for (i = 0; i < mov->nb_streams; i++) {
+ for (i = 0; i < mov->nb_tracks; i++) {
MOVTrack *track = &mov->tracks[i];
if ((tracks >= 0 && i != tracks) || !track->entry)
continue;
@@ -5128,7 +5128,7 @@ static int mov_write_moof_tag_internal(AVIOContext *pb, MOVMuxContext *mov,
mov->first_trun = 1;
mov_write_mfhd_tag(pb, mov);
- for (i = 0; i < mov->nb_streams; i++) {
+ for (i = 0; i < mov->nb_tracks; i++) {
MOVTrack *track = &mov->tracks[i];
if (tracks >= 0 && i != tracks)
continue;
@@ -5222,7 +5222,7 @@ static int mov_write_sidx_tags(AVIOContext *pb, MOVMuxContext *mov,
} else {
avio_buf = pb;
}
- for (i = 0; i < mov->nb_streams; i++) {
+ for (i = 0; i < mov->nb_tracks; i++) {
MOVTrack *track = &mov->tracks[i];
if (tracks >= 0 && i != tracks)
continue;
@@ -5363,7 +5363,7 @@ static int mov_write_mfra_tag(AVIOContext *pb, MOVMuxContext *mov)
if (mov->flags & FF_MOV_FLAG_ISML)
goto done_mfra;
- for (i = 0; i < mov->nb_streams; i++) {
+ for (i = 0; i < mov->nb_tracks; i++) {
MOVTrack *track = &mov->tracks[i];
if (track->nb_frag_info)
mov_write_tfra_tag(mfra_pb, track);
@@ -5868,7 +5868,7 @@ static int mov_flush_fragment(AVFormatContext *s, int force)
}
}
- for (i = 0; i < mov->nb_streams; i++) {
+ for (i = 0; i < mov->nb_tracks; i++) {
MOVTrack *track = &mov->tracks[i];
if (track->entry <= 1)
continue;
@@ -5900,15 +5900,15 @@ static int mov_flush_fragment(AVFormatContext *s, int force)
uint8_t *buf;
int buf_size, moov_size;
- for (i = 0; i < mov->nb_streams; i++)
+ for (i = 0; i < mov->nb_tracks; i++)
if (!mov->tracks[i].entry && !is_cover_image(mov->tracks[i].st))
break;
/* Don't write the initial moov unless all tracks have data */
- if (i < mov->nb_streams && !force)
+ if (i < mov->nb_tracks && !force)
return 0;
moov_size = get_moov_size(s);
- for (i = 0; i < mov->nb_streams; i++)
+ for (i = 0; i < mov->nb_tracks; i++)
mov->tracks[i].data_offset = pos + moov_size + 8;
avio_write_marker(s->pb, AV_NOPTS_VALUE, AVIO_DATA_MARKER_HEADER);
@@ -5936,7 +5936,7 @@ static int mov_flush_fragment(AVFormatContext *s, int force)
mov->moov_written = 1;
mov->mdat_size = 0;
- for (i = 0; i < mov->nb_streams; i++) {
+ for (i = 0; i < mov->nb_tracks; i++) {
mov->tracks[i].entry = 0;
mov->tracks[i].end_reliable = 0;
}
@@ -5945,7 +5945,7 @@ static int mov_flush_fragment(AVFormatContext *s, int force)
}
if (mov->frag_interleave) {
- for (i = 0; i < mov->nb_streams; i++) {
+ for (i = 0; i < mov->nb_tracks; i++) {
MOVTrack *track = &mov->tracks[i];
int ret;
if ((ret = mov_flush_fragment_interleaving(s, track)) < 0)
@@ -5957,7 +5957,7 @@ static int mov_flush_fragment(AVFormatContext *s, int force)
mdat_size = avio_tell(mov->mdat_buf);
}
- for (i = 0; i < mov->nb_streams; i++) {
+ for (i = 0; i < mov->nb_tracks; i++) {
MOVTrack *track = &mov->tracks[i];
if (mov->flags & FF_MOV_FLAG_SEPARATE_MOOF || mov->frag_interleave)
track->data_offset = 0;
@@ -5986,7 +5986,7 @@ static int mov_flush_fragment(AVFormatContext *s, int force)
av_rescale(mov->tracks[first_track].cluster[0].dts, AV_TIME_BASE, mov->tracks[first_track].timescale),
(has_video ? starts_with_key : mov->tracks[first_track].cluster[0].flags & MOV_SYNC_SAMPLE) ? AVIO_DATA_MARKER_SYNC_POINT : AVIO_DATA_MARKER_BOUNDARY_POINT);
- for (i = 0; i < mov->nb_streams; i++) {
+ for (i = 0; i < mov->nb_tracks; i++) {
MOVTrack *track = &mov->tracks[i];
int buf_size, write_moof = 1, moof_tracks = -1;
uint8_t *buf;
@@ -6198,7 +6198,7 @@ int ff_mov_write_packet(AVFormatContext *s, AVPacket *pkt)
if (par->codec_id == AV_CODEC_ID_H264 && trk->vos_len > 0 && *(uint8_t *)trk->vos_data != 1 && !TAG_IS_AVCI(trk->tag)) {
/* from x264 or from bytestream H.264 */
/* NAL reformatting needed */
- if (trk->hint_track >= 0 && trk->hint_track < mov->nb_streams) {
+ if (trk->hint_track >= 0 && trk->hint_track < mov->nb_tracks) {
ret = ff_avc_parse_nal_units_buf(pkt->data, &reformatted_data,
&size);
if (ret < 0)
@@ -6218,7 +6218,7 @@ int ff_mov_write_packet(AVFormatContext *s, AVPacket *pkt)
} else if (par->codec_id == AV_CODEC_ID_HEVC && trk->vos_len > 6 &&
(AV_RB24(trk->vos_data) == 1 || AV_RB32(trk->vos_data) == 1)) {
/* extradata is Annex B, assume the bitstream is too and convert it */
- if (trk->hint_track >= 0 && trk->hint_track < mov->nb_streams) {
+ if (trk->hint_track >= 0 && trk->hint_track < mov->nb_tracks) {
ret = ff_hevc_annexb2mp4_buf(pkt->data, &reformatted_data,
&size, 0, NULL);
if (ret < 0)
@@ -6236,7 +6236,7 @@ int ff_mov_write_packet(AVFormatContext *s, AVPacket *pkt)
}
}
} else if (par->codec_id == AV_CODEC_ID_AV1) {
- if (trk->hint_track >= 0 && trk->hint_track < mov->nb_streams) {
+ if (trk->hint_track >= 0 && trk->hint_track < mov->nb_tracks) {
ret = ff_av1_filter_obus_buf(pkt->data, &reformatted_data,
&size, &offset);
if (ret < 0)
@@ -6419,7 +6419,7 @@ int ff_mov_write_packet(AVFormatContext *s, AVPacket *pkt)
trk->sample_count += samples_in_chunk;
mov->mdat_size += size;
- if (trk->hint_track >= 0 && trk->hint_track < mov->nb_streams)
+ if (trk->hint_track >= 0 && trk->hint_track < mov->nb_tracks)
ff_mov_add_hinted_packet(s, pkt, trk->hint_track, trk->entry,
reformatted_data ? reformatted_data + offset
: NULL, size);
@@ -6594,7 +6594,7 @@ static int mov_write_packet(AVFormatContext *s, AVPacket *pkt)
* 3) See mov_write_trailer for how the final end sample is
* handled.
*/
- for (i = 0; i < mov->nb_streams; i++) {
+ for (i = 0; i < mov->nb_tracks; i++) {
MOVTrack *trk = &mov->tracks[i];
int ret;
@@ -6869,7 +6869,7 @@ static void mov_free(AVFormatContext *s)
avcodec_parameters_free(&mov->tracks[mov->chapter_track].par);
}
- for (i = 0; i < mov->nb_streams; i++) {
+ for (i = 0; i < mov->nb_tracks; i++) {
MOVTrack *const track = &mov->tracks[i];
if (track->tag == MKTAG('r','t','p',' '))
@@ -7101,14 +7101,14 @@ static int mov_init(AVFormatContext *s)
s->streams[0]->disposition |= AV_DISPOSITION_DEFAULT;
}
- mov->nb_streams = s->nb_streams;
+ mov->nb_tracks = s->nb_streams;
if (mov->mode & (MODE_MP4|MODE_MOV|MODE_IPOD) && s->nb_chapters)
- mov->chapter_track = mov->nb_streams++;
+ mov->chapter_track = mov->nb_tracks++;
if (mov->flags & FF_MOV_FLAG_RTP_HINT) {
for (i = 0; i < s->nb_streams; i++)
if (rtp_hinting_needed(s->streams[i]))
- mov->nb_streams++;
+ mov->nb_tracks++;
}
if (mov->write_btrt < 0) {
@@ -7145,12 +7145,12 @@ static int mov_init(AVFormatContext *s)
}
}
- mov->nb_streams += mov->nb_meta_tmcd;
+ mov->nb_tracks += mov->nb_meta_tmcd;
}
// Reserve an extra stream for chapters for the case where chapters
// are written in the trailer
- mov->tracks = av_calloc(mov->nb_streams + 1, sizeof(*mov->tracks));
+ mov->tracks = av_calloc(mov->nb_tracks + 1, sizeof(*mov->tracks));
if (!mov->tracks)
return AVERROR(ENOMEM);
@@ -7571,7 +7571,7 @@ static int compute_moov_size(AVFormatContext *s)
if (moov_size < 0)
return moov_size;
- for (i = 0; i < mov->nb_streams; i++)
+ for (i = 0; i < mov->nb_tracks; i++)
mov->tracks[i].data_offset += moov_size;
moov_size2 = get_moov_size(s);
@@ -7581,7 +7581,7 @@ static int compute_moov_size(AVFormatContext *s)
/* if the size changed, we just switched from stco to co64 and need to
* update the offsets */
if (moov_size2 != moov_size)
- for (i = 0; i < mov->nb_streams; i++)
+ for (i = 0; i < mov->nb_tracks; i++)
mov->tracks[i].data_offset += moov_size2 - moov_size;
return moov_size2;
@@ -7596,7 +7596,7 @@ static int compute_sidx_size(AVFormatContext *s)
if (sidx_size < 0)
return sidx_size;
- for (i = 0; i < mov->nb_streams; i++)
+ for (i = 0; i < mov->nb_tracks; i++)
mov->tracks[i].data_offset += sidx_size;
return sidx_size;
@@ -7645,7 +7645,7 @@ static int mov_write_trailer(AVFormatContext *s)
* Before actually writing the trailer, make sure that there are no
* dangling subtitles, that need a terminating sample.
*/
- for (i = 0; i < mov->nb_streams; i++) {
+ for (i = 0; i < mov->nb_tracks; i++) {
MOVTrack *trk = &mov->tracks[i];
if (trk->par->codec_id == AV_CODEC_ID_MOV_TEXT &&
!trk->last_sample_is_subtitle_end) {
@@ -7664,7 +7664,7 @@ static int mov_write_trailer(AVFormatContext *s)
// when we are not doing fragments.
if (!mov->chapter_track && !(mov->flags & FF_MOV_FLAG_FRAGMENT)) {
if (mov->mode & (MODE_MP4|MODE_MOV|MODE_IPOD) && s->nb_chapters) {
- mov->chapter_track = mov->nb_streams++;
+ mov->chapter_track = mov->nb_tracks++;
if ((res = mov_create_chapter_track(s, mov->chapter_track)) < 0)
return res;
}
@@ -7716,7 +7716,7 @@ static int mov_write_trailer(AVFormatContext *s)
res = 0;
} else {
mov_auto_flush_fragment(s, 1);
- for (i = 0; i < mov->nb_streams; i++)
+ for (i = 0; i < mov->nb_tracks; i++)
mov->tracks[i].data_offset = 0;
if (mov->flags & FF_MOV_FLAG_GLOBAL_SIDX) {
int64_t end;
diff --git a/libavformat/movenc.h b/libavformat/movenc.h
index e85d83abdb..60363198c9 100644
--- a/libavformat/movenc.h
+++ b/libavformat/movenc.h
@@ -188,7 +188,7 @@ typedef struct MOVMuxContext {
const AVClass *av_class;
int mode;
int64_t time;
- int nb_streams;
+ int nb_tracks;
int nb_meta_tmcd; ///< number of new created tmcd track based on metadata (aka not data copy)
int chapter_track; ///< qt chapter track number
int64_t mdat_pos;
--
2.43.0
More information about the ffmpeg-devel
mailing list