Fix preparing log file name.

src/log.cc

 // Copyright 2011 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "src/log.h"
 
 #include <cstdarg>
 #include <sstream>
 
 #include "src/v8.h"
@@ skipping 1749 matching lines @@
 
 
 static void AddIsolateIdIfNeeded(std::ostream& os,  // NOLINT
                                  Isolate* isolate) {
   if (FLAG_logfile_per_isolate) os << "isolate-" << isolate << "-";
 }
 
 
 static void PrepareLogFileName(std::ostream& os,  // NOLINT
                                Isolate* isolate, const char* file_name) {
-  AddIsolateIdIfNeeded(os, isolate);
+  int slash_count = 0;
   for (const char* p = file_name; *p; p++) {
-    if (*p == '%') {
+    if (base::OS::isDirectorySeparator(*p))
+      slash_count++;

[Yang, 2015/03/03 08:36:57]

no line break after if.

[sejunho, 2015/03/03 10:14:50]

Done.

+  }
+
+  for (const char* p = file_name; *p; p++) {
+    if (slash_count == 0) {
+      AddIsolateIdIfNeeded(os, isolate);
+      slash_count--;
+    }
+
+    if (base::OS::isDirectorySeparator(*p)) {
+      slash_count--;
+      os << *p;
+    } else if (*p == '%') {

[Yang, 2015/03/03 08:36:57]

How about

if (*p == '%') {
  ... 
} else {
  if (base::OS::isDirectorySeparator(*p)) slash_count--;
  os << *p;
}

[sejunho, 2015/03/03 10:14:50]

OK.

       p++;
       switch (*p) {
         case '\0':
           // If there's a % at the end of the string we back up
           // one character so we can escape the loop properly.
           p--;
           break;
         case 'p':
           os << base::OS::GetCurrentProcessId();
           break;
@@ skipping 125 matching lines @@
   if (jit_logger_) {
     removeCodeEventListener(jit_logger_);
     delete jit_logger_;
     jit_logger_ = NULL;
   }
 
   return log_->Close();
 }
 
 } }  // namespace v8::internal