| Re: [AD] C99 isms in test.c |
[ Thread Index |
Date Index
| More lists.liballeg.org/allegro-developers Archives
]
Evert,
Did you apply this patch? AFAICS, the sinf and friends are still in there. It
won't compile for me now. Apologies for not reporting this earlier.
Pete
On Thursday 13 October 2005 21:08, Evert Glebbeek wrote:
> On Sunday 09 October 2005 23:30, Evert Glebbeek wrote:
> > See near the bottom of this thread for details:
> > http://www.allegro.cc/forums/view_thread.php?_id=534779
> >
> > The last patch to test.c added some calls to sinf(), atanf(), cosf() and
> > others. These are C99 functions and therefor no-no.
> > The fix is simple, just remove the f prefix. I can (hopefully) submit a
> > patch tomorrow.
>
> Ok, so my bike broke down on monday and I've been working through the past
> couple of evenings, so I didn't get round to it before now.
> Anyway, patch attached. It fixes the C99isms, corrects a small mistake and
> tries to fool clever compilers that might optimise out the unused result
> of the calculation.
>
> Evert