[FFmpeg-devel] [PATCH] Fix non-rounding up to next 16-bit aligned bug in IFF decoder

Måns Rullgård mans
Thu Apr 29 16:47:57 CEST 2010


Michael Niedermayer <michaelni at gmx.at> writes:

> On Thu, Apr 29, 2010 at 03:36:07PM +0200, Sebastian Vater wrote:
> [...]
>> static void decodeplane8(uint8_t *dst,
>>                          const uint8_t *buf,
>>                          const unsigned buf_size,
>>                          const unsigned bps,
>>                          const unsigned plane)
>> {
>>     START_TIMER;
>>     const uint8_t *end = dst + (buf_size * 8);
>>     const uint64_t *lut = plane8_lut[plane];
>>     for(; dst < end; dst += 8) {
>>         const uint64_t v  = AV_RN64A(dst) | lut[*buf++];
>>         AV_WN64A(dst, v);
>>     }
>>     STOP_TIMER("decodeplane8");
>> }
>
> this looks very nice

Is the length always a multiple of 8 now?  Earlier you said 4...
What's the deal?

-- 
M?ns Rullg?rd
mans at mansr.com



More information about the ffmpeg-devel mailing list