http://codereview.chromium.org/42643/diff/6/1008 File src/debug.cc (right):
http://codereview.chromium.org/42643/diff/6/1008#newcode1910 Line 1910: void Debugger::InvokeMessageHandler(Vector< uint16_t> message) { On 2009/03/27 08:01:09, Mads Ager wrote: > Remove space in Vector< uint16_t>? Done. http://codereview.chromium.org/42643/diff/6/1008#newcode1919 Line 1919: // If there is not message thread just invoke the message handler from the On 2009/03/27 08:01:09, Mads Ager wrote: > not -> no Done. http://codereview.chromium.org/42643/diff/6/1008#newcode1974 Line 1974: Vector<uint16_t>(const_cast<uint16_t *>(command.start()), On 2009/03/27 08:01:09, Mads Ager wrote: > remove space in "uint16_t *"? Done. http://codereview.chromium.org/42643/diff/6/1008#newcode2007 Line 2007: // Send an empty command to the debugger if in a break to make JavaScript run On 2009/03/27 08:01:09, Mads Ager wrote: > This line looks too long. Done. http://codereview.chromium.org/42643/diff/6/1008#newcode2062 Line 2062: DebugMessageThread::DebugMessageThread() { On 2009/03/27 08:01:09, Mads Ager wrote: > Just inline this in the header? Done. http://codereview.chromium.org/42643/diff/6/1008#newcode2067 Line 2067: DebugMessageThread::~DebugMessageThread() { On 2009/03/27 08:01:09, Mads Ager wrote: > Ditto. Done. http://codereview.chromium.org/42643/diff/6/1007 File src/debug.h (right): http://codereview.chromium.org/42643/diff/6/1007#newcode541 Line 541: static Semaphore* command_received_; // Non-zero when command queue is non-empty. On 2009/03/27 08:01:09, Mads Ager wrote: > The lines look too long. Put the comments on the line before the declaration? Rephrased comment. http://codereview.chromium.org/42643 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
