BuildLog: Use Log::insert(Log::value_type()) to avoid invalid strings.

The MSVC std::string implementation copies strings, so using make_pair
resulted in a pointer to invalid memory. This ensures the insert uses a
StringPiece without an intermediate std::string copy, so the correct pointer
ends up in the hash_map.

1 files changed, 2 insertions, 2 deletions

diff --git a/src/build_log.cc b/src/build_log.cc
index f342e83..4c875d9 100644
--- a/src/build_log.cc
+++ b/src/build_log.cc
@@ -85,7 +85,7 @@ void BuildLog::RecordCommand(Edge* edge, int start_time, int end_time,
     } else {
       log_entry = new LogEntry;
       log_entry->output = path;
-      log_.insert(make_pair(log_entry->output.c_str(), log_entry));
+      log_.insert(Log::value_type(log_entry->output, log_entry));
     }
     log_entry->command = command;
     log_entry->start_time = start_time;
@@ -169,7 +169,7 @@ bool BuildLog::Load(const string& path, string* err) {
     } else {
       entry = new LogEntry;
       entry->output = output;
-      log_.insert(make_pair(entry->output, entry));
+      log_.insert(Log::value_type(entry->output, entry));
       ++unique_entry_count;
     }
     ++total_entry_count;