[FFmpeg-devel] [PATCH] VP8 decoder

Måns Rullgård mans
Tue Jun 22 16:08:17 CEST 2010


"Ronald S. Bultje" <rsbultje at gmail.com> writes:

> Hi,
>
> On Tue, Jun 15, 2010 at 10:22 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
>> On Tue, Jun 15, 2010 at 09:12:30AM -0400, David Conrad wrote:
>>> +static void pred4x4_vertical_vp8_c(uint8_t *src, const uint8_t *topright, int stride){
>>> + ? ?const int lt= src[-1-1*stride];
>>> + ? ?LOAD_TOP_EDGE
>>> + ? ?LOAD_TOP_RIGHT_EDGE
>>> +
>>> + ? ?// 32-bit stores maybe
>>
>> yes
>
> Fixed, I think (I wanted to use AV_WN32() instead of AV_WL/B32, so I

If it's aligned you should use AV_WN32A().

> had to introduce a macro to write the four bytes in the correct order,
> doesn't something like that already exist somewhere?).

Pack 4 8-bit values into a 32-bit word?  I don't think such a macro
exists, but it should be added.

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



More information about the ffmpeg-devel mailing list