[FFmpeg-devel] [PATCH] vf_fps: when reading EOF, using current_pts to duplicate the last frame if needed.

[Thierry Foucu]

HI Clement

On Fri, Sep 15, 2017 at 7:55 AM, Clément Bœsch <u at pkh.me> wrote:

> On Fri, Sep 15, 2017 at 04:53:45PM +0200, Clément Bœsch wrote:
> > On Fri, Sep 15, 2017 at 11:15:14AM -0300, James Almer wrote:
> > > On 9/15/2017 10:15 AM, Clément Bœsch wrote:
> > > > On Fri, Sep 15, 2017 at 02:44:59AM +0200, Michael Niedermayer wrote:
> > > >> On Tue, Sep 12, 2017 at 06:45:57PM -0700, Thierry Foucu wrote:
> > > >>> Fix ticket #2674
> > > >>> Tested with examples from ticket 2674.
> > > >>> ---
> > > >>> Sorry Michael, I forgot to configure using --enable-gpl.
> > > >>> Please find new patch.
> > > >>>
> > > >>>
> > > >>>  libavfilter/vf_fps.c             | 44
> +++++++++++++++++++++++++++++++++++-----
> > > >>>  tests/ref/fate/filter-fps        |  6 ++++++
> > > >>>  tests/ref/fate/filter-fps-r      |  4 ++++
> > > >>>  tests/ref/fate/filter-mpdecimate |  1 -
> > > >>>  tests/ref/fate/m4v-cfr           |  1 -
> > > >>>  5 files changed, 49 insertions(+), 7 deletions(-)
> > > >>
> > > >> applied
> > > >>
> > > >
> > > > asan doesn't like this commit (it stalls)
> > >
> > > This is the second time it stalls with a leak. Sounds like gcc-asan 7.x
> > > is buggy.
> > >
> >
> > it's a FATE bug, not an asan bug AFAICT
> >
>
> Too fast to reply again, my bad. It's indeed a GCC bug.
>

Sorry about it. I'm glad you found the problem.


>
> --
> Clément B.
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
>