[FFmpeg-devel] [PATCH] vp56.h: change a div to a shift

Ronald S. Bultje rsbultje
Tue Jun 22 21:24:44 CEST 2010


Hi,

On Tue, Jun 22, 2010 at 1:51 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Tue, Jun 22, 2010 at 11:32:05AM -0400, Ronald S. Bultje wrote:
>> as per $attached, all variables are always >= 0 (and c->high is
>> >=128), but c->high is defined as int so the compiler wouldn't know
>> this by itself. This patch helps it along a bit, changing a / 256 to a
>> >> 8.
>>
>> Ronald
>
>> ?vp56.h | ? ?2 +-
>> ?1 file changed, 1 insertion(+), 1 deletion(-)
>> a7b00ef0d69998a9d61cfa5b6596762c61940af3 ?change_div_to_shift.patch
>
> iam not maintainer of the file but i would have thought such changes
> can just be commited wthout patch ...

I tend to just send emails anyway, better have one eye too many than
one too little.

Applied.

Ronald



More information about the ffmpeg-devel mailing list