[webkit-reviews] review denied: [Bug 28121] [Haiku] Modifications on JavaScriptCore. : [Attachment 34406] Modifications on JavaScriptCore to allow Haiku port.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sun Aug 9 07:39:28 PDT 2009


Eric Seidel <eric at webkit.org> has denied Maxime Simon
<simon.maxime at gmail.com>'s request for review:
Bug 28121: [Haiku] Modifications on JavaScriptCore.
https://bugs.webkit.org/show_bug.cgi?id=28121

Attachment 34406: Modifications on JavaScriptCore to allow Haiku port.
https://bugs.webkit.org/attachment.cgi?id=34406&action=review

------- Additional Comments from Eric Seidel <eric at webkit.org>
Bah!  We need to abstract the Collector to use some sort of Threading*
abstraction!

This is the 3rd patch I've reviewed this week to add #ifdefs there. :(

+#include "platform/NotImplemented.h"

should be "NotImplemented.h" no need for "platform/"

r- for the wrong include path.	If you'd be kind enough to fix the threading
abstraction in some way (e.g. to at least use a static inline function for the
Haiku case that would be nice.)


More information about the webkit-reviews mailing list