[FFmpeg-devel] [Ffmpeg-devel] [PATCH] HD DVD subtitle decoding
Benoit Fouet
benoit.fouet
Tue Jul 17 14:33:28 CEST 2007
Michael Niedermayer wrote:
> Hi
>
> On Mon, Jul 16, 2007 at 02:48:40PM +0100, Ian Caulfield wrote:
>
>> 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.
>>
>
> ok
>
>
Applied
--
Ben
Purple Labs S.A.
www.purplelabs.com
More information about the ffmpeg-devel
mailing list