Re: [FFmpeg-devel] [PATCH]lavf/rmdec: Do not return EIO on EOF

2016-05-10 Thread Michael Niedermayer
On Tue, May 10, 2016 at 03:08:25PM +0200, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes an incorrect error message and an incorrect failure on > xerror as reported by applemax82 on both user mailing lists. > > Please comment, Carl Eugen > rmdec.c |4 +++- > 1 file changed, 3 inser

Re: [FFmpeg-devel] [PATCH]lavf/rmdec: Do not return EIO on EOF

2016-05-10 Thread Carl Eugen Hoyos
Paul B Mahol gmail.com> writes: > > How is the fact that the device produces damaged files > > (be it because of wireless or other issues) related to > > fixing broken audio for the given samples? > > Either too big predictors should be skipped or clipped. > How decoded output sounds when played

Re: [FFmpeg-devel] [PATCH]lavf/rmdec: Do not return EIO on EOF

2016-05-10 Thread Paul B Mahol
On 5/10/16, Carl Eugen Hoyos wrote: > Paul B Mahol gmail.com> writes: > >> OT but I remembered seeing overread messages from OP >> when using your patch on his files. > > How is the fact that the device produces damaged files > (be it because of wireless or other issues) related to > fixing broke

Re: [FFmpeg-devel] [PATCH]lavf/rmdec: Do not return EIO on EOF

2016-05-10 Thread Carl Eugen Hoyos
Paul B Mahol gmail.com> writes: > OT but I remembered seeing overread messages from OP > when using your patch on his files. How is the fact that the device produces damaged files (be it because of wireless or other issues) related to fixing broken audio for the given samples? Carl Eugen __

Re: [FFmpeg-devel] [PATCH]lavf/rmdec: Do not return EIO on EOF

2016-05-10 Thread Paul B Mahol
On 5/10/16, Carl Eugen Hoyos wrote: > Paul B Mahol gmail.com> writes: > >> Again not patch but useless diff file. > > Again no useful comment;-( > > Did you forget my question about the Babyphon? OT but I remembered seeing overread messages from OP when using your patch on his files. ___

Re: [FFmpeg-devel] [PATCH]lavf/rmdec: Do not return EIO on EOF

2016-05-10 Thread Carl Eugen Hoyos
Paul B Mahol gmail.com> writes: > Again not patch but useless diff file. Again no useful comment;-( Did you forget my question about the Babyphon? Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ff

Re: [FFmpeg-devel] [PATCH]lavf/rmdec: Do not return EIO on EOF

2016-05-10 Thread Paul B Mahol
On 5/10/16, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes an incorrect error message and an incorrect failure on > > xerror as reported by applemax82 on both user mailing lists. > > Please comment, Carl Eugen > Again not patch but useless diff file. __

[FFmpeg-devel] [PATCH]lavf/rmdec: Do not return EIO on EOF

2016-05-10 Thread Carl Eugen Hoyos
Hi! Attached patch fixes an incorrect error message and an incorrect failure on xerror as reported by applemax82 on both user mailing lists. Please comment, Carl Eugen diff --git a/libavformat/rmdec.c b/libavformat/rmdec.c index cb0481e..a633582 100644 --- a/libavformat/rmdec.c +++ b/libavformat