[FFmpeg-devel] [PATCH 3/6] fftools/ffplay: drop an unused function argument

Stefano Sabatini stefasab at gmail.com
Sun Mar 12 17:05:19 EET 2023


On date Friday 2023-03-10 12:56:32 +0100, Anton Khirnov wrote:
> ---
>  fftools/ffplay.c | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/fftools/ffplay.c b/fftools/ffplay.c
> index d6479aef5f..f09ff59ccc 100644
> --- a/fftools/ffplay.c
> +++ b/fftools/ffplay.c
> @@ -1553,7 +1553,8 @@ static double vp_duration(VideoState *is, Frame *vp, Frame *nextvp) {
>      }
>  }
>  
> -static void update_video_pts(VideoState *is, double pts, int64_t pos, int serial) {
> +static void update_video_pts(VideoState *is, double pts, int serial)
> +{
>      /* update current video pts */
>      set_clock(&is->vidclk, pts, serial);
>      sync_clock_to_slave(&is->extclk, &is->vidclk);
> @@ -1618,7 +1619,7 @@ retry:
>  
>              SDL_LockMutex(is->pictq.mutex);
>              if (!isnan(vp->pts))
> -                update_video_pts(is, vp->pts, vp->pos, vp->serial);
> +                update_video_pts(is, vp->pts, vp->serial);
>              SDL_UnlockMutex(is->pictq.mutex);
>  
>              if (frame_queue_nb_remaining(&is->pictq) > 1) {

LGTM.


More information about the ffmpeg-devel mailing list