Lgtm.
http://codereview.chromium.org/385001/diff/1/2 File tools/process-heap-prof.py (right): http://codereview.chromium.org/385001/diff/1/2#newcode77 Line 77: if (options.count): Don't need parens. http://codereview.chromium.org/385001/diff/1/2#newcode105 Line 105: parser = BuildOptions() Per google style this should be packed into a function that is only called if __name__ == '__main__'. http://codereview.chromium.org/385001 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
