mjwybrow, bug in CVS with AMD64, 'Error: clock_t is not 4 bytes'
Hi mjwybrow,
A small bug exists in the code of the file: libavoid/timer.cpp After running the code on an AMD64 system:
Error: clock_t is not 4 bytes Aborted
This snippet has a bug: if (sizeof(clock_t) != 4) { fprintf(stderr, "Error: clock_t is not 4 bytes\n"); abort(); }
Its always a good idea to never compare a size_of with a constant (unless in things like configure scripts or with very lowlevel program code). I havnt read the rest of the file yet (so I dont know why you insist on having it 4 bytes), but it is obvious that such structures have different sizes on different architectures. If necessary, hop me an email and we can work this problem out via IRC (if you have not an 64-bit system available).
Greets,
Edwin de Jong
On Tue, 30 Aug 2005, Edwin de Jong wrote:
Whoops! Thanks for catching that. I've removed the offending code in CVS. The code was there because I was having problems with timer values wrapping around when generating performance results for a paper describing the library code. The timers aren't used by Inkscape.
Thanks, Michael
participants (2)
-
Edwin de Jong
-
Michael Wybrow