Yep On Wed, Jun 17, 2009 at 16:14, Kasper Lund <[email protected]> wrote:
> LGTM. Does it lint? > > On Wed, Jun 17, 2009 at 4:13 PM, <[email protected]> wrote: > > Reviewers: Kasper Lund, > > > > Description: > > Fix lint error. > > > > [email protected] > > > > > > Please review this at http://codereview.chromium.org/126276 > > > > SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/ > > > > Affected files: > > M test/cctest/test-debug.cc > > > > > > Index: test/cctest/test-debug.cc > > =================================================================== > > --- test/cctest/test-debug.cc (revision 2209) > > +++ test/cctest/test-debug.cc (working copy) > > @@ -4281,7 +4281,7 @@ > > bar->Call(env->Global(), 0, NULL); > > CHECK_EQ(4, break_point_hit_count); > > } > > - > > + > > // Remove the debug event listener without clearing breakpoints. Do > this > > // outside a handle scope. > > v8::Debug::SetDebugEventListener(NULL); > > > > > > > --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
