[webkit-reviews] review requested: [Bug 19750] Merge inc/dec halves of PrefixResolveNode and PostfixResolveNode : [Attachment 21930] No method pointers, fewer parentheses, fixed fixme.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jun 25 08:13:17 PDT 2008


Gavin Barraclough <barraclough at apple.com> has asked Cameron Zwarich (cpst)
<cwzwarich at uwaterloo.ca> for review:
Bug 19750: Merge inc/dec halves of PrefixResolveNode and PostfixResolveNode
https://bugs.webkit.org/show_bug.cgi?id=19750

Attachment 21930: No method pointers, fewer parentheses, fixed fixme.
https://bugs.webkit.org/attachment.cgi?id=21930&action=edit

------- Additional Comments from Gavin Barraclough <barraclough at apple.com>

Cameron,

I've replaced the method pointers with simple static helper methods in
nodes.cpp – if you could have a look & see what you think that'd be great.

cheers,
G.


More information about the webkit-reviews mailing list