[FFmpeg-devel] [PATCH] Add DPX decoder rev-25

Jimmy Christensen jimmy
Thu Jun 11 22:26:09 CEST 2009


On 2009-06-11 18:56, Michael Niedermayer wrote:
> On Thu, Jun 11, 2009 at 05:59:56AM +0200, Jimmy Christensen wrote:
>> On 2009-06-11 00:51, Michael Niedermayer wrote:
>>> On Wed, Jun 10, 2009 at 06:17:00PM +0200, Jimmy Christensen wrote:
>>>> On 2009-06-10 16:48, Diego Biurrun wrote:
>>>>> On Wed, Jun 10, 2009 at 02:36:58PM +0200, Jimmy Christensen wrote:
>>>>>>
>>>>>> I was wondering how close this DPX patch is to get accepted. Is the
>>>>>> general quality good enough or does it need to be approved a lot more?
>>>>>
>>>>> Close.
>>>>>
>>>>
>>>> Thanks. Have started looking into the 3D lut filter for doing log/lin
>>>> conversion. A bit challenging and slow as hell but have already gotten
>>>> some
>>>> promising results.
>>>>
>>>>>> --- libavcodec/dpx.c	(revision 0)
>>>>>> +++ libavcodec/dpx.c	(revision 0)
>>>>>> @@ -0,0 +1,227 @@
>>>>>> +static unsigned int read32(const uint8_t **ptr, int is_big)
>>>>>> +{
>>>>>> +    unsigned int temp;
>>>>>> +    if(is_big)
>>>>>
>>>>> if (
>>>>>
>>>>
>>>> Fixed.
>>>>
>>> [...]
>>>
>>>> +    /* Check if the files "magic number" is "SDPX" which means it uses
>>>> +     * big-endian or XPDS which is for little-endian files */
>>>> +    if (magic_num == AV_RL32("SDPX"))
>>>> +        endian = 0;
>>>> +    else if (magic_num == AV_RB32("SDPX"))
>>>> +        endian = 1;
>>>> +    else {
>>>
>>> please add {} here they need no extra lines and they make future patches
>>> smaller
>>
>> Added, although in the beginning I was told to remove more or less those
>> exact {} by Diego :)
>>
>>  From Diego :
>>>>> +	if(magic_num == 0x53445058) {
>>>>> +		endian = 0x0;
>>>>> +	}
>>> useless {}
>>
>> But does that go for all one line if's or just that pair?
>
> it goes for all one line if() without a following else
>

Ah, I see.



More information about the ffmpeg-devel mailing list