[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