Lgtm!
http://codereview.chromium.org/215052/diff/4001/4003 File tools/jsmin.py (right): http://codereview.chromium.org/215052/diff/4001/4003#newcode30 Line 30: # Suppress copyright warning: pylint: disable-msg=C6304 I would suggest removing this and just letting pylint complain. http://codereview.chromium.org/215052/diff/4001/4003#newcode125 Line 125: # Enters it into the mapping table for this scope. Why is this not part of the docstring? http://codereview.chromium.org/215052 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
