[FFmpeg-devel] [PATCH 1/2] lavfi/avf_concat: add some doxy.

Nicolas George george at nsup.org
Sun Aug 9 13:56:52 EEST 2020


The semantic of delta_pts is not obvious.

Signed-off-by: Nicolas George <george at nsup.org>
---
 libavfilter/avf_concat.c | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/libavfilter/avf_concat.c b/libavfilter/avf_concat.c
index 28bd5407ad..246628498a 100644
--- a/libavfilter/avf_concat.c
+++ b/libavfilter/avf_concat.c
@@ -224,6 +224,12 @@ static void close_input(AVFilterContext *ctx, unsigned in_no)
            ctx->input_pads[in_no].name, cat->nb_in_active);
 }
 
+/**
+ * Compute the delta_ts of the next segment.
+ *
+ * If everything starts at 0 as expected, it is the timestamp where the
+ * next segment begins in the output.
+ */
 static void find_next_delta_ts(AVFilterContext *ctx, int64_t *seg_delta)
 {
     ConcatContext *cat = ctx->priv;
@@ -238,6 +244,14 @@ static void find_next_delta_ts(AVFilterContext *ctx, int64_t *seg_delta)
     *seg_delta = pts;
 }
 
+/**
+ * Send silence to pad audio streams to the duration of video streams
+ *
+ * @param ctx        codec context
+ * @param in_no      input number
+ * @param out_no     output number
+ * @param seg_delta  segment delta, i.e. output timestamp to reach.
+ */
 static int send_silence(AVFilterContext *ctx, unsigned in_no, unsigned out_no,
                         int64_t seg_delta)
 {
-- 
2.27.0



More information about the ffmpeg-devel mailing list