LGTM
http://codereview.chromium.org/56102/diff/6/1006 File src/debug.h (right): http://codereview.chromium.org/56102/diff/6/1006#newcode508 Line 508: // No debugger currently active unload the debugger if possible. Hard to read. How about: Unload the debugger if possible. Only called when no debugger is currently active. http://codereview.chromium.org/56102/diff/6/1005 File test/cctest/test-debug.cc (right): http://codereview.chromium.org/56102/diff/6/1005#newcode3774 Line 3774: // Debugger message handler which counts then number of times it is called. then -> the http://codereview.chromium.org/56102 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
