[webkit-reviews] review granted: [Bug 16464] Modify WebCore to use win32 thread primitives : [Attachment 17965] Updated based on review.

[bugzilla-daemon at webkit.org]

Darin Adler <darin at apple.com> has granted Brent Fulgham <bfulgham at gmail.com>'s
request for review:
Bug 16464: Modify WebCore to use win32 thread primitives
http://bugs.webkit.org/show_bug.cgi?id=16464

Attachment 17965: Updated based on review.
http://bugs.webkit.org/attachment.cgi?id=17965&action=edit

------- Additional Comments from Darin Adler <darin at apple.com>
 71 static const long MaxSemaphoreCount = static_cast<long>((~0u) >> 1);

I think it needs to be ~0UL rather than ~0U. And also there is no need for
parentheses there -- the precedence of the ~ operator is tighter than the >>
operator, and I think it's easier to read as is.

Patch has some tabs in it. We can't check in patches with tabs.

 115	 unsigned was_gone=0;

Unused local variable here.

Otherwise looks good. r=me, although I'm a little nervous about the level of
testing that will be required to vet this on Windows with Safari.