Re: [FFmpeg-devel] [PATCH] avfilter/af_amix: do not leave unset PTS for frames after first stream is over

2020-09-14 Thread Nicolas George
Paul B Mahol (12020-09-13):
> First stream is used only to get number of samples to put into each output 
> frame.
> 
> Signed-off-by: Paul B Mahol 
> ---
>  libavfilter/af_amix.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)

I cannot judge the change of the code since I do not know this code, but
based on the commit message, something like this is necessary in this
filter.

-- 
  Nicolas George


signature.asc
Description: PGP signature
___
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] [PATCH] avfilter/af_amix: do not leave unset PTS for frames after first stream is over

2020-09-13 Thread Paul B Mahol
First stream is used only to get number of samples to put into each output 
frame.

Signed-off-by: Paul B Mahol 
---
 libavfilter/af_amix.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/libavfilter/af_amix.c b/libavfilter/af_amix.c
index cae9d4585a..c4d8916a57 100644
--- a/libavfilter/af_amix.c
+++ b/libavfilter/af_amix.c
@@ -309,6 +309,8 @@ static int output_frame(AVFilterLink *outlink)
 }
 }
 }
+
+s->next_pts = frame_list_next_pts(s->frame_list);
 } else {
 /* first input closed: use the available samples */
 nb_samples = INT_MAX;
@@ -324,7 +326,6 @@ static int output_frame(AVFilterLink *outlink)
 }
 }
 
-s->next_pts = frame_list_next_pts(s->frame_list);
 frame_list_remove_samples(s->frame_list, nb_samples);
 
 calculate_scales(s, nb_samples);
-- 
2.17.1

___
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".