Return immediately from GetLogLines if logging to memory isn't enabled.

src/log.cc

Index: src/log.cc
diff --git a/src/log.cc b/src/log.cc
index b4e20540a14bc4ba631efc7fcfc2114f59d05a75..5297a302d80d7cdfde3fd8c224c6625de6f577c6 100644
--- a/src/log.cc
+++ b/src/log.cc
@@ -423,6 +423,7 @@ void Log::Close() {
 
 
 int Log::GetLogLines(int from_pos, char* dest_buf, int max_size) {
+  if (Write != WriteToMemory) return 0;

[Søren Thygesen Gjesse, 2009/05/08 09:59:33]

Maybe we should consider having an enum to indicate to where log output is going
instead of checking the write function.

[Mikhail Naganov, 2009/05/08 10:06:15]

And end up have two variables for specifying mode (Write and the new enum)? No.
I think the current semantics is fine.

   ASSERT(output_.buffer != NULL);
   ASSERT(output_buffer_write_pos_ >= output_.buffer);
   ASSERT(from_pos >= 0);