[Ffmpeg-devel] [PATCH] cook compatible decoder

[Benjamin Larsson]

Michael Niedermayer wrote:

>Hi
>
>On Fri, Nov 04, 2005 at 11:29:25AM +0100, Benjamin Larsson wrote:
>  
>
>> [...]
>>
>>I think a swap is needed, I just don't know the order.
>>
>>        obuf[i] = le2me_32(0x37c511f2^buf[i]);
>>    
>>
>
>another issue here is that at least buf might not be 32bit aligned (if
>sub_packet_size %4 != 0 if i understand it correctly)
>i guess a simple if(sub_packet_size %4) return -1 would be the correct
>solution? or does such a size make sense?
>
>[...]
>  
>
IIRC firstrun.rm has a sub_packet_size of 93, so it needs to be handled.
Currently all sub_packets are stored in an unaligned manner if the
sub_packet_size
is not evenly divided by 4. Is 32bit alignment mandated?

MvH
Benjamin Larsson