Re: [FFmpeg-devel] [PATCH]lavc/dvbsub: Do not fail on clut depth 0

2015-08-04 Thread wm4
On Tue, 4 Aug 2015 13:18:54 + (UTC) Carl Eugen Hoyos wrote: > wm4 googlemail.com> writes: > > > Seems like a case for AV_EF_EXPLODE. > > I wonder if this isn't a bug in our decoder given > that the output looks very ugly and not an issue > with the given stream but somebody will have to

Re: [FFmpeg-devel] [PATCH]lavc/dvbsub: Do not fail on clut depth 0

2015-08-04 Thread Carl Eugen Hoyos
wm4 googlemail.com> writes: > Seems like a case for AV_EF_EXPLODE. I wonder if this isn't a bug in our decoder given that the output looks very ugly and not an issue with the given stream but somebody will have to check. I opened ticket #4754 for the visual issue. > Also, why can't you send

Re: [FFmpeg-devel] [PATCH]lavc/dvbsub: Do not fail on clut depth 0

2015-08-04 Thread wm4
On Tue, 4 Aug 2015 13:31:10 +0200 Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes the regression reported in ticket #4752, the subtitles > look very bad before and after but I guess this should be a different ticket. > > Please comment, Carl Eugen Seems like a case for AV_EF_EXPLODE.

Re: [FFmpeg-devel] [PATCH]lavc/dvbsub: Do not fail on clut depth 0

2015-08-04 Thread Hendrik Leppkes
On Tue, Aug 4, 2015 at 1:31 PM, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes the regression reported in ticket #4752, the subtitles > look very bad before and after but I guess this should be a different ticket. > Makes sense, I guess, it specifically didn't return an error code before,

[FFmpeg-devel] [PATCH]lavc/dvbsub: Do not fail on clut depth 0

2015-08-04 Thread Carl Eugen Hoyos
Hi! Attached patch fixes the regression reported in ticket #4752, the subtitles look very bad before and after but I guess this should be a different ticket. Please comment, Carl Eugen diff --git a/libavcodec/dvbsubdec.c b/libavcodec/dvbsubdec.c index ce03e12..23f24f1 100644 --- a/libavcodec/dvb