Re: [AD] Timer patch

[Ryan D. <goa...@gm...>]

On Jan 29, 2008 4:52 PM, Peter Wang <nov...@gm...> wrote:

> On 2008-01-29, Ryan Dickie <goa...@gm...> wrote:
> > On Jan 29, 2008 3:34 PM, Ryan Dickie <goa...@gm...> wrote:
>  >
> > > I have made those style changes (and found some dead code as well).
> See
> > > attachment
> > >
> > > -- Ryan Dickie
> > >
> >
> > Trent tested it out on windows for me. We found a silly typo in the
> > wtime.cpatch. I've fixed it. See attached
>
> For future reference, you can use the interdiff program from patchutils
> to generate relative diffs.
>
> http://cyberelk.net/tim/software/patchutils/
>
> Peter
>
>
> -------------------------------------------------------------------------
> This SF.net email is sponsored by: Microsoft
> Defy all challenges. Microsoft(R) Visual Studio 2008.
> http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
> --
> https://lists.sourceforge.net/lists/listinfo/alleg-developers
>


I've attached another patch for the periodic timers. I've switched them to
doubles as well. I did not touch the windows one. The unix timer
utimernu.cis now built entirely on portable allegro functions. This
code should work
for windows as well provided we get windows using a more accurate time and
sleep function.

I might have missed this as well in my first patch. No compiler warnings
were given but timestamp is probably set using al_current_time(). I switched
it to a double.

--- include/allegro5/events.h    (revision 9954)
+++ include/allegro5/events.h    (working copy)
@@ -142,7 +142,7 @@
 #define _AL_EVENT_HEADER(srctype)                    \
    ALLEGRO_EVENT_TYPE type;                          \
    srctype *source;                                  \
-   unsigned long timestamp;                          \
+   double timestamp;                          \
    signed int _refcount;            /* internal */   \
    union ALLEGRO_EVENT *_next;      /* internal */   \
    union ALLEGRO_EVENT *_next_free  /* internal */

-- Ryan Dickie