[MPlayer-dev-eng] [PATCH] demux lavf: mp_read: Propagate EOF
Alexander Strasser
eclipse7 at gmx.net
Fri Nov 17 01:06:10 EET 2017
Hi,
thank you for having a look at this Michael and Nicolas!
On 2017-11-16 22:24 +0100, Michael Niedermayer wrote:
> On Sun, Nov 12, 2017 at 10:16:52PM +0100, Alexander Strasser wrote:
> > Return AVERROR_EOF from mp_read callback, if stream->eof is non-zero.
> >
> > Fixes ticket #2335
> >
> > Signed-off-by: Alexander Strasser <eclipse7 at gmx.net>
> > ---
> >
> > Please look at the ticket for more details.
> >
> > Although there was a patch committed in FFmpeg, that avoids
> > the problem for now, I don't want to wait until the next bump
> > re-introduces the problem.
> >
> > libmpdemux/demux_lavf.c | 2 ++
> > 1 file changed, 2 insertions(+)
>
> LGTM
Committed.
> maybe the mp_msg() should be changed so it displays EOF insteda of
> the decimal value but thats rather very minor
I agree on that, but as the fix is rather important I didn't want to
wait any longer. At least one can see if EOF is returned by spotting
large negative decimal values.
[...]
Alexander
More information about the MPlayer-dev-eng
mailing list