On Fri, 13 Jun 2025, softworkz wrote:
From: softworkz <softwo...@hotmail.com> Also moves point of log writing to after rename of temp files and changes log level to INFO. The completion of a segment is important enough to justify logging at INFO level. Signed-off-by: softworkz <softwo...@hotmail.com> --- libavformat/segment.c | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/libavformat/segment.c b/libavformat/segment.c index 04e973a198..608cad0ba4 100644 --- a/libavformat/segment.c +++ b/libavformat/segment.c @@ -417,8 +417,6 @@ static int segment_end(AVFormatContext *s, int write_trailer, int is_last) } } - av_log(s, AV_LOG_VERBOSE, "segment:'%s' count:%d ended\n", - seg->avf->url, seg->segment_count); seg->segment_count++; if (seg->increment_tc) { @@ -471,6 +469,12 @@ static int segment_end(AVFormatContext *s, int write_trailer, int is_last) av_free(final_filename); } + av_log(s, AV_LOG_INFO, "SegmentComplete=%s:%d Index=%d Start=%f End=%f Duration=%f offset_pts=%s start_pts=%s Frames=%d filename=%s\n", + av_get_media_type_string(s->streams[seg->reference_stream_index]->codecpar->codec_type), + seg->reference_stream_index, seg->segment_idx, seg->cur_entry.start_time, seg->cur_entry.end_time, + seg->cur_entry.end_time - seg->cur_entry.start_time, av_ts2str(seg->cur_entry.offset_pts), av_ts2str(seg->cur_entry.start_pts), + seg->segment_frame_count, seg->cur_entry.filename);
No CamelCase please, follow snake_case as the other similar messages. And yes, VERBOSE is the proper loglevel for this.
+ return ret; fail: @@ -994,6 +998,8 @@ calc_times: if (pkt->dts != AV_NOPTS_VALUE) pkt->dts += offset; + seg->cur_entry.offset_pts = av_rescale_q(offset, st->time_base, AV_TIME_BASE_Q); +
This line does not belong to this patch, or the patch description is incomplete.
Thanks, Marton
av_log(s, AV_LOG_DEBUG, " -> pts:%s pts_time:%s dts:%s dts_time:%s\n", av_ts2str(pkt->pts), av_ts2timestr(pkt->pts, &st->time_base), av_ts2str(pkt->dts), av_ts2timestr(pkt->dts, &st->time_base)); -- ffmpeg-codebot _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".