[FFmpeg-devel] [Ffmpeg-devel] [PATCH] HD DVD subtitle decoding

[Ian Caulfield]

Finally had a chance to catch up on this...

On 19/02/07, Michael Niedermayer <michaelni at gmx.at> wrote:
>
> Hi
>
> On Mon, Feb 19, 2007 at 10:48:35AM +0000, Ian Caulfield wrote:
> > On 18/02/07, Michael Niedermayer <michaelni at gmx.at> wrote:
> > >Hi
> > >iam more in favor of putting the YUV_TO_RGB*_CCIR into a header and
> > >including this
> > >what the function above does seems quite specific to dvdsubdec.c
> > >so IMHO it should be there rather
> >
> > I think dvbsubdec does a yuv->rgb conversion in more or less the same
> > way. I'll move the code over to a separate header (colorspace.h?)
>
> if you keep the loop in dvdsubdec.c then moving YUV_TO_RGB*_CCIR to
> colorspace.h sounds like a good idea


colorspace.patch moves these defines to colorspace.h (via an svn cp from
imgconvert.c), and removes the copied code from dvbsub.c and dvbsubdec.c.

hd-dvdsub-new.patch is the updated version of the HD-DVD subtitle patch,
using the new colorspace include.

Ian
-------------- next part --------------
A non-text attachment was scrubbed...
Name: colorspace.patch
Type: text/x-diff
Size: 84762 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070716/5fd50f66/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: hd-dvdsub-new.patch
Type: text/x-diff
Size: 10684 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070716/5fd50f66/attachment-0001.patch>