[FFmpeg-devel] Small modifcation to libavformat/dvbsubdec.c
michaelni at gmx.at
Mon Sep 16 18:09:57 CEST 2013
On Mon, Sep 16, 2013 at 08:46:39AM +0100, JULIAN GARDNER wrote:
> > From: JULIAN GARDNER <joolzg at btinternet.com>
> >To: FFmpegdevelopment discussions and patches <ffmpeg-devel at ffmpeg.org>
> >Sent: Wednesday, 4 September 2013, 23:51
> >Subject: Small modifcation to libavformat/dvbsubdec.c
> >diff --git a/libavcodec/dvbsubdec.c b/libavcodec/dvbsubdec.c
> >index 955925a..faa510a 100644
> >--- a/libavcodec/dvbsubdec.c
> >+++ b/libavcodec/dvbsubdec.c
> >@@ -1015,9 +1015,9 @@ static void dvbsub_parse_clut_segment(AVCodecContext *avctx,
> > if (depth & 0x80)
> > clut->clut4[entry_id] = RGBA(r,g,b,255 - alpha);
> >- if (depth & 0x40)
> >+ else if (depth & 0x40)
> > clut->clut16[entry_id] = RGBA(r,g,b,255 - alpha);
> >- if (depth &
> >+ else if (depth & 0x20)
> > clut->clut256[entry_id] = RGBA(r,g,b,255 - alpha);
> > }
> > }
> >Just so it adheres to spec. Page 25 of ETS300743 V1.2.1, page 25.
> >2-bit/entry_CLUT_flag: If set to '1', this indicates that this CLUT value is to be loaded into the identified entry of the
> 2-bit/entry CLUT.
> >4-bit/entry_CLUT_flag: If set to '1', this indicates that this CLUT value is to be loaded into the identified entry of the
> 4-bit/entry CLUT.
> >8-bit/entry_CLUT_flag: If set to '1', this indicates that this CLUT value is to be loaded into the identified entry of the
> 8-bit/entry CLUT.
> >NOTE 1: Only one N-bit/entry_CLUT_flag shall be set to 1 per CLUT_entry_id and its associated Y-, Cr-, Cb- and
> T -values.
> Any comment on this, will it be pushed?
what does this patch fix ?
please provide a testcase or if it fixes spec compliance a quote of
the spec saying that if multiple bits are set this new behavior is
to be used.
You quote just a part of the spec that says that the affected case
shall not occur. why would a change in behavior for a case that lies
outside the spec be a matter of spec adherence ?
I surely am missing something here ...
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
No human being will ever know the Truth, for even if they happen to say it
by chance, they would not even known they had done so. -- Xenophanes
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel