LGTM On Fri, Feb 20, 2009 at 9:37 AM, <[email protected]> wrote: > Reviewers: Mads Ager, > > Description: > Minor fix to fix building without ENABLE_LOGGING_AND_PROFILING defined. > > Please review this at http://codereview.chromium.org/21534 > > SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/ > > Affected files: > M src/log.cc > > > Index: src/log.cc > =================================================================== > --- src/log.cc (revision 1320) > +++ src/log.cc (working copy) > @@ -679,6 +679,7 @@ > } > > > +#ifdef ENABLE_LOGGING_AND_PROFILING > int Logger::CodeObjectSize(Code* code) { > // Check that the assumptions about the layout of the code object holds. > ASSERT_EQ(reinterpret_cast<unsigned int>(code->instruction_start()) - > @@ -686,6 +687,7 @@ > Code::kHeaderSize); > return code->instruction_size() + Code::kHeaderSize; > } > +#endif > > > void Logger::CodeCreateEvent(const char* tag, Code* code, const char* > comment) { > > >
--~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
