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

Jimmy Christensen jimmy
Tue May 26 05:51:25 CEST 2009


On 2009-05-25 23:58, Michael Niedermayer wrote:
> On Mon, May 25, 2009 at 08:54:12AM +0200, Jimmy Christensen wrote:
>> On 2009-05-15 03:51, Michael Niedermayer wrote:
>>> On Mon, May 11, 2009 at 11:31:25AM +0200, Jimmy Christensen wrote:
> [...]
>>>
>>>
>>> [...]
>>>> +    for (x = 0; x<   s->height; x++) {
>>>> +        uint8_t *dst = ptr;
>>>> +        for (y = 0; y<   s->width; y++) {
>>>
>>>> +            rgbBuffer = AV_RB32(buf);
>>>> +            memcpy(&rgb10Field,&rgbBuffer, 4);
>>>> +            rgb16Field.R = rgb10Field.R * 64; // 10-bit>   16-bit
>>>> +            rgb16Field.G = rgb10Field.G * 64; // 10-bit>   16-bit
>>>> +            rgb16Field.B = rgb10Field.B * 64; // 10-bit>   16-bit
>>>> +            memcpy(dst,&rgb16Field, dstBpp);
>>>
>>> not portable
>>>
>>
>> I rewrote a little, but still uses memcpy from rgb16Field to dst, since
>> that should actually be portable.
>
> you will have to quote page and pararaph of the C standard that gurantees
> bitfields to work the way you seem to belive they do
>

So the bit masking is fine (RED10, GREEN10 and BLUE10), but I need to 
change the memcpy?



More information about the ffmpeg-devel mailing list