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

Michael Niedermayer michaelni
Tue Jun 22 19:51:44 CEST 2010


On Tue, Jun 22, 2010 at 11:32:05AM -0400, Ronald S. Bultje wrote:
> Hi,
> 
> 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 ...

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Awnsering whenever a program halts or runs forever is
On a turing machine, in general impossible (turings halting problem).
On any real computer, always possible as a real computer has a finite number
of states N, and will either halt in less than N cycles or never halt.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100622/5825c755/attachment.pgp>



More information about the ffmpeg-devel mailing list