tests: Remove a potential race from test-thread
authorPhilip Withnall <philip.withnall@collabora.co.uk>
Thu, 9 Jan 2014 18:02:26 +0000 (18:02 +0000)
committerOlivier Crête <olivier.crete@collabora.com>
Fri, 31 Jan 2014 06:48:58 +0000 (01:48 -0500)
commit441f0b2257682b74ffccaac82a0c5d54c4eb53ec
tree95b19218fc1daa28be05ba85f761f3b201e1eb1d
parentd0c5adbf56f398a6c00b75c56c80c541ec467839
tests: Remove a potential race from test-thread

A g_usleep() was introduced in commit
e22ecb19662970a42def6df74db6359e58084727 to fix a potential race where
the non-main threads would finish all their work before the error_loop
was started, leaving the error_loop to run for 30s then abort the test.

A better fix, instead of doing a racy sleep, is to have the non-main
threads spin until the error_loop is running. GLib takes care of all the
locking for us, and since we don’t care about efficiency for test cases,
spinning is fine (the wait is also going to be quite short).
tests/test-thread.c