[webkit-reviews] review granted: [Bug 103027] [Chromium] fastMalloc has an extra branch on Windows : [Attachment 176043] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Nov 26 17:29:01 PST 2012


Eric Seidel <eric at webkit.org> has granted Adam Barth <abarth at webkit.org>'s
request for review:
Bug 103027: [Chromium] fastMalloc has an extra branch on Windows
https://bugs.webkit.org/show_bug.cgi?id=103027

Attachment 176043: Patch
https://bugs.webkit.org/attachment.cgi?id=176043&action=review

------- Additional Comments from Eric Seidel <eric at webkit.org>
LGTM.  Other ports which USE_SYSTEM_MALLOC might want this implied as well, but
we can make that decision if/when others switch it on.	Also, I wonder if these
shouldn't be USE_ macros.  ENABLE has historically been for features.


More information about the webkit-reviews mailing list