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):
On 2009/11/10 08:57:11, Christian Plesner Hansen wrote:
> Don't need parens.

Done.

http://codereview.chromium.org/385001/diff/1/2#newcode105
Line 105: parser = BuildOptions()
On 2009/11/10 08:57:11, Christian Plesner Hansen wrote:
> Per google style this should be packed into a function that is only
called if
> __name__ == '__main__'.

Done.

http://codereview.chromium.org/385001

--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---

Reply via email to