[webkit-reviews] review granted: [Bug 21281] Some parser cleanup : [Attachment 24003] patch

[bugzilla-daemon at webkit.org]

Cameron Zwarich (cpst) <cwzwarich at uwaterloo.ca> has granted 's request for
review:
Bug 21281: Some parser cleanup
https://bugs.webkit.org/show_bug.cgi?id=21281

Attachment 24003: patch
https://bugs.webkit.org/attachment.cgi?id=24003&action=edit

------- Additional Comments from Cameron Zwarich (cpst)
<cwzwarich at uwaterloo.ca>
In the ChangeLog, you say wtf:: instead of WTF::.

You use both sourceID and sourceId in your patch. I know the latter is because
the original code had it, but you should probably pick one and stick with it.
Since sourceID is our coding style, that seems like a good choice.

I will say r=me, but due to Darin's earlier review, I would like to see this
looked over by him after it lands.