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

[Benoit Fouet]

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