[FFmpeg-devel] [PATCH 03/23] hpeldsp: add half-pel functions (currently copies of dsputil).

Ronald S. Bultje rsbultje at gmail.com
Wed Mar 13 04:46:19 CET 2013


Hi,

On Tue, Mar 12, 2013 at 7:32 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Tue, Mar 12, 2013 at 07:28:13AM -0700, Ronald S. Bultje wrote:
>> From: "Ronald S. Bultje" <rsbultje at gmail.com>
> [...]
>> +static inline void FUNCC(OPNAME ## _pixels2_xy2)(uint8_t *p_block, const uint8_t *p_pixels, ptrdiff_t line_size, int h)\
>> +{\
>> +        int i, a0, b0, a1, b1;\
>> +        pixel *block = (pixel*)p_block;\
>> +        const pixel *pixels = (const pixel*)p_pixels;\
>
> comparing source and destination of the copy:
>
> -static inline void FUNCC(OPNAME ## _pixels2_xy2)(uint8_t *_block, const uint8_t *_pixels, ptrdiff_t line_size, int h)\
> +static inline void FUNCC(OPNAME ## _pixels2_xy2)(uint8_t *p_block, const uint8_t *p_pixels, ptrdiff_t line_size, int h)\
>  {\
>          int i, a0, b0, a1, b1;\
> -        pixel *block = (pixel*)_block;\
> -        const pixel *pixels = (const pixel*)_pixels;\
> +        pixel *block = (pixel*)p_block;\
> +        const pixel *pixels = (const pixel*)p_pixels;\
>          line_size >>= sizeof(pixel)-1;\
>          a0= pixels[0];\
>          b0= pixels[1] + 2;\
>
> I dont really mind the change but i suspect this is unintended

Uhm, yeah, not sure how that happened...

Ronald


More information about the ffmpeg-devel mailing list