Revision: 3723
Author: [email protected]
Date: Wed Jan 27 09:10:28 2010
Log: Get it to compile when ENABLE_LOGGING_AND_PROFILING isn't defined.
I think it currently is defined on all platforms except Android.
Review URL: http://codereview.chromium.org/554101
http://code.google.com/p/v8/source/detail?r=3723

Modified:
 /branches/bleeding_edge/src/log.cc

=======================================
--- /branches/bleeding_edge/src/log.cc  Mon Jan 25 14:53:18 2010
+++ /branches/bleeding_edge/src/log.cc  Wed Jan 27 09:10:28 2010
@@ -904,10 +904,10 @@
 }


+#ifdef ENABLE_LOGGING_AND_PROFILING
 void Logger::MoveEventInternal(LogEventsAndTags event,
                                Address from,
                                Address to) {
-#ifdef ENABLE_LOGGING_AND_PROFILING
   static Address prev_to_ = NULL;
   if (!Log::IsEnabled() || !FLAG_log_code) return;
   LogMessageBuilder msg;
@@ -922,12 +922,12 @@
   }
   msg.Append('\n');
   msg.WriteToLogFile();
-#endif
-}
+}
+#endif


-void Logger::DeleteEventInternal(LogEventsAndTags event, Address from) {
 #ifdef ENABLE_LOGGING_AND_PROFILING
+void Logger::DeleteEventInternal(LogEventsAndTags event, Address from) {
   if (!Log::IsEnabled() || !FLAG_log_code) return;
   LogMessageBuilder msg;
   msg.Append("%s,", log_events_[event]);
@@ -938,8 +938,8 @@
   }
   msg.Append('\n');
   msg.WriteToLogFile();
-#endif
-}
+}
+#endif


 void Logger::ResourceEvent(const char* name, const char* tag) {

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

Reply via email to