- Fixed bug caused by using a high-resolution timer (gettimeofday) on
Windows, which only has a 10ms resolution timer. Consequently the
random number seed used was from a significantly reduced pool, which
leads to duplicate UUIDs.
- Fixed bug that failed to anchor to end of string when searching for
"cal" or "itask", but instead matched on elements of the path,
like /usr/local/bin. Thanks to Eric Farris.
- Fixed bug that was causing more non-unique UUIDs. Here are the
changes made:
- Task.cpp, tasks are no longer provided with a UUID in Task::Task.
This prevents the global context.task from being constructed before
srandom/srand is called.
- main.cpp, instead of srandom/srand (time (NULL)), it now uses
struct timeval tv_usec member, which has a micro-second granularity,
instead of time (NULL) which has a second granularity. When
"task add ..." is called in a unit test, several calls are made per
second, this the random number generator is seeded with the same
value.
- Modified the unit test to cover all 6 tasks created, instead of
5.
- Fixed bug that caused code to use a combination of srandom and rand,
or srand and random. Should be srandom/random, srand/rand, not a
mix.
- Fixed bug that failed to set uuid in handleAdd, such that the uuid
already assigned in context.task was used, which was generated before
srand/srandom was is called.
- Relocated longUsage and shortUsage to report.cpp, where they belong,
which has the positive side effect of allowing all the compiled unit
tests to link.
- Renamed files task.h -> main.h, task.cpp -> main.cpp. This is to
prepare for the next rename, which addresses the OpenBSD bug.
- Removed utf8.h, utf8.cpp from Makefile.am, which were inadvertantly
added.
