LGTM
http://codereview.chromium.org/125256/diff/9/1011 File tools/logreader.js (right): http://codereview.chromium.org/125256/diff/9/1011#newcode91 Line 91: return function addressParser (str) { Maybe add parentheses arround the function returned 'return (function addressParser (str) { ... })' - can make it easier to see where it ends. Why does it have a name anyway? http://codereview.chromium.org/125256 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
