| Re: [AD] [ alleg-Bugs-1650580 ] Problem in fixed point library. |
[ Thread Index |
Date Index
| More lists.liballeg.org/allegro-developers Archives
]
On 2007-02-04, Elias Pschernig <elias@xxxxxxxxxx> wrote:
> I'll apply the below patch if nobody objects - the bug reporter claims
> to have tested it, and LONG_LONG also is used for fixmul so should cause
> no problem here.
>
> Index: include/allegro/inline/fmaths.inl
> ===================================================================
> --- include/allegro/inline/fmaths.inl (revision 7706)
> +++ include/allegro/inline/fmaths.inl (working copy)
> @@ -151,7 +151,7 @@
> return (x < 0) ? -0x7FFFFFFF : 0x7FFFFFFF;
> }
> else
> - return ftofix(fixtof(x) / fixtof(y));
> + return ((LONG_LONG)x << 16) / (LONG_LONG)y;
> })
Didn't we discuss the fixed point functions on allegro.cc last year or
so? Don't apply this just yet.
Peter