diff options
author | James Almer <jamrial@gmail.com> | 2021-02-08 19:04:05 -0300 |
---|---|---|
committer | James Almer <jamrial@gmail.com> | 2021-02-13 13:05:26 -0300 |
commit | d5d6751a5505463824bf37e86a3a79010ba31d0b (patch) | |
tree | 928bdb278381b073204ab318d3862550c2e58196 | |
parent | 93e2fa933f1625f488a2d88f9d50254ef13401d2 (diff) |
avformat/mux: return a pointer to the packet in ff_interleaved_peek()
And make it const, so the caller doesn't attempt to change it.
ff_get_muxer_ts_offset() should be used to get the muxer timestamp offset.
Signed-off-by: James Almer <jamrial@gmail.com>
-rw-r--r-- | libavformat/internal.h | 8 | ||||
-rw-r--r-- | libavformat/movenc.c | 18 | ||||
-rw-r--r-- | libavformat/mux.c | 20 |
3 files changed, 16 insertions, 30 deletions
diff --git a/libavformat/internal.h b/libavformat/internal.h index 33ece6b172..0ffdc87b6a 100644 --- a/libavformat/internal.h +++ b/libavformat/internal.h @@ -865,14 +865,10 @@ int ff_bprint_to_codecpar_extradata(AVCodecParameters *par, struct AVBPrint *buf /** * Find the next packet in the interleaving queue for the given stream. - * The pkt parameter is filled in with the queued packet, including - * references to the data (which the caller is not allowed to keep or - * modify). * - * @return 0 if a packet was found, a negative value if no packet was found + * @return a pointer to a packet if one was found, NULL otherwise. */ -int ff_interleaved_peek(AVFormatContext *s, int stream, - AVPacket *pkt, int add_offset); +const AVPacket *ff_interleaved_peek(AVFormatContext *s, int stream); int ff_get_muxer_ts_offset(AVFormatContext *s, int stream_index, int64_t *offset); diff --git a/libavformat/movenc.c b/libavformat/movenc.c index 62192e5c24..f97383fcfe 100644 --- a/libavformat/movenc.c +++ b/libavformat/movenc.c @@ -5301,15 +5301,19 @@ static int mov_flush_fragment(AVFormatContext *s, int force) for (i = 0; i < s->nb_streams; i++) { MOVTrack *track = &mov->tracks[i]; if (!track->end_reliable) { - AVPacket pkt; - if (!ff_interleaved_peek(s, i, &pkt, 1)) { + const AVPacket *pkt = ff_interleaved_peek(s, i); + if (pkt) { + int64_t offset, dts, pts; + ff_get_muxer_ts_offset(s, i, &offset); + pts = pkt->pts + offset; + dts = pkt->dts + offset; if (track->dts_shift != AV_NOPTS_VALUE) - pkt.dts += track->dts_shift; - track->track_duration = pkt.dts - track->start_dts; - if (pkt.pts != AV_NOPTS_VALUE) - track->end_pts = pkt.pts; + dts += track->dts_shift; + track->track_duration = dts - track->start_dts; + if (pts != AV_NOPTS_VALUE) + track->end_pts = pts; else - track->end_pts = pkt.dts; + track->end_pts = dts; } } } diff --git a/libavformat/mux.c b/libavformat/mux.c index ae46844c66..062ba8d789 100644 --- a/libavformat/mux.c +++ b/libavformat/mux.c @@ -1062,30 +1062,16 @@ int ff_get_muxer_ts_offset(AVFormatContext *s, int stream_index, int64_t *offset return 0; } -int ff_interleaved_peek(AVFormatContext *s, int stream, - AVPacket *pkt, int add_offset) +const AVPacket *ff_interleaved_peek(AVFormatContext *s, int stream) { AVPacketList *pktl = s->internal->packet_buffer; while (pktl) { if (pktl->pkt.stream_index == stream) { - *pkt = pktl->pkt; - if (add_offset) { - AVStream *st = s->streams[pkt->stream_index]; - int64_t offset = st->internal->mux_ts_offset; - - if (s->output_ts_offset) - offset += av_rescale_q(s->output_ts_offset, AV_TIME_BASE_Q, st->time_base); - - if (pkt->dts != AV_NOPTS_VALUE) - pkt->dts += offset; - if (pkt->pts != AV_NOPTS_VALUE) - pkt->pts += offset; - } - return 0; + return &pktl->pkt; } pktl = pktl->next; } - return AVERROR(ENOENT); + return NULL; } /** |