Added a wait with timeout to the platform semaphore class

Added a wait with timeout to the platform semaphore class.

The code has been compiled and tested on Windows, Linux and Mac OS.

The FreeBSD version is a copy of the Linux version which should work on FreeBSD as well. According to the FreeBSD documentation clock_gettime is part of the standard C library so the assumption is that no additional link libraries is required for FreeBSD.

Committed: http://code.google.com/p/v8/source/detail?r=1526

[Erik Corry, 2009-03-18 08:34:23]

LGTM

http://codereview.chromium.org/48123/diff/1/5
File src/platform-linux.cc (right):

http://codereview.chromium.org/48123/diff/1/5#newcode612
Line 612: // Split timeout into seconds and nanoseconds parts.
seconds and nanoseconds -> second and nanosecond

http://codereview.chromium.org/48123/diff/1/5#newcode622
Line 622: // Calculate realtime for end of timeout.
real time twowords.

http://codereview.chromium.org/48123/diff/1/7
File src/platform.h (right):

http://codereview.chromium.org/48123/diff/1/7#newcode420
Line 420: // true is returned. The timeout value is specified in miiliseconds.
I would prefer microseconds.

http://codereview.chromium.org/48123/diff/1/2
File test/cctest/test-lock.cc (right):

http://codereview.chromium.org/48123/diff/1/2#newcode48
Line 48: ok = sem->Wait(1000);
Please reduce the length of this wait so the tests continue to run fast.

[Søren Thygesen Gjesse, 2009-03-18 09:03:32]

http://codereview.chromium.org/48123/diff/1/5
File src/platform-linux.cc (right):

http://codereview.chromium.org/48123/diff/1/5#newcode612
Line 612: // Split timeout into seconds and nanoseconds parts.
On 2009/03/18 08:34:23, Erik Corry wrote:
> seconds and nanoseconds -> second and nanosecond

Done.

http://codereview.chromium.org/48123/diff/1/5#newcode622
Line 622: // Calculate realtime for end of timeout.
On 2009/03/18 08:34:23, Erik Corry wrote:
> real time twowords.

Done.

http://codereview.chromium.org/48123/diff/1/7
File src/platform.h (right):

http://codereview.chromium.org/48123/diff/1/7#newcode420
Line 420: // true is returned. The timeout value is specified in miiliseconds.
On 2009/03/18 08:34:23, Erik Corry wrote:
> I would prefer microseconds.

Changed API to use microseconds.

http://codereview.chromium.org/48123/diff/1/2
File test/cctest/test-lock.cc (right):

http://codereview.chromium.org/48123/diff/1/2#newcode48
Line 48: ok = sem->Wait(1000);
On 2009/03/18 08:34:23, Erik Corry wrote:
> Please reduce the length of this wait so the tests continue to run fast.

With API at microseconds I kept 1000, and added tests for 0 and 100 as well.