[FFmpeg-devel] [PATCH] VP8: correctly use optimal epel functions for splitmv mode

Ronald S. Bultje rsbultje
Mon Jun 28 15:53:47 CEST 2010


Hi,

On Mon, Jun 28, 2010 at 3:04 AM, David Conrad <lessen42 at gmail.com> wrote:
> On Jun 27, 2010, at 1:57 PM, Ronald S. Bultje wrote:
>> +static void vp8_mc_part(VP8Context *s, uint8_t *dst[3], AVFrame *ref_frame,
>> + ? ? ? ? ? ? ? ? ? ? ? ?int x_off, int y_off, int bx_off, int by_off,
>> + ? ? ? ? ? ? ? ? ? ? ? ?int block_w, int block_h,
>> + ? ? ? ? ? ? ? ? ? ? ? ?int width, int height, VP56mv *mv)
>
> inline
> (someone, maybe me, should experiment with manually forcing inlining stuff in vp8)

Changed.

>> @@ -112,7 +119,7 @@
>> ? ? ? ?{ -6, -7 } ? ? ? ? ? ? ? ? ? ? ? ? ? ?// '110', '111'
>> ?};
>>
>> -static const uint8_t vp8_mbsplits[4][16] = {
>> +static const uint8_t vp8_mbsplits[5][16] = {
>> ? ? ?{ ?0, ?0, ?0, ?0, ?0, ?0, ?0, ?0,
>> ? ? ? ? 1, ?1, ?1, ?1, ?1, ?1, ?1, ?1 ?},
>> ? ? ?{ ?0, ?0, ?1, ?1, ?0, ?0, ?1, ?1,
>> @@ -120,7 +127,9 @@
>> ? ? ?{ ?0, ?0, ?1, ?1, ?0, ?0, ?1, ?1,
>> ? ? ? ? 2, ?2, ?3, ?3, ?2, ?2, ?3, ?3 ?},
>> ? ? ?{ ?0, ?1, ?2, ?3, ?4, ?5, ?6, ?7,
>> - ? ? ? 8, ?9, 10, 11, 12, 13, 14, 15 ?}
>> + ? ? ? 8, ?9, 10, 11, 12, 13, 14, 15 ?},
>> + ? ?{ ?0, ?0, ?0, ?0, ?0, ?0, ?0, ?0,
>> + ? ? ? 0, ?0, ?0, ?0, ?0, ?0, ?0, ?0 ?}
>> ?};
>>
>> ?static const uint8_t vp8_mbfirstidx[4][16] = {
>
> Is this related?

Now, part of patch#2, probably merged wrongly. Removed.

> OK otherwise

Applied with those changes.

Ronald



More information about the ffmpeg-devel mailing list