Re: [PATCH] demux lavf: mp_read: Propagate EOF

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 <[email protected]>
> ---
> 
> 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

maybe the mp_msg() should be changed so it displays EOF insteda of
the decimal value but thats rather very minor

thanks

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The misfortune of the wise is better than the prosperity of the fool.
-- Epicurus

_______________________________________________
MPlayer-dev-eng mailing list
[email protected]
https://lists.mplayerhq.hu/mailman/listinfo/mplayer-dev-eng

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1

iEYEARECAAYFAloOAhMACgkQYR7HhwQLD6tguwCdEVGbxBHWxWD7FKJ4nuJV1PlP
ZWQAmwSvBldqHCjMsHaVS9+2AQqu10HO
=nS72
-----END PGP SIGNATURE-----