[webkit-reviews] review canceled: [Bug 118482] fourthTier: Change JSStack to use macro to determine stack growth direction : [Attachment 206266] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Jul 9 17:21:12 PDT 2013
Michael Saboff <msaboff at apple.com> has canceled Michael Saboff
<msaboff at apple.com>'s request for review:
Bug 118482: fourthTier: Change JSStack to use macro to determine stack growth
direction
https://bugs.webkit.org/show_bug.cgi?id=118482
Attachment 206266: Patch
https://bugs.webkit.org/attachment.cgi?id=206266&action=review
------- Additional Comments from Michael Saboff <msaboff at apple.com>
Limited the #ifdef's to JSStack files. The ifdef's are needed due to doing
arithmetic or comparisons with pointers or size_t type values. In both cases,
we can't use the multiply by -1 or 1 trick. In some cases, we could create an
#ifdef'ed help function to do <= comparisons or pointer arithmetic, but I felt
that would obfuscate the reading of the code.
If this patch isn't acceptable, then I think it is mega patch time.
More information about the webkit-reviews
mailing list