Protect BuildLog data member variables by making them private.

They are not accessed outside of BuildLog and there is even a SetConfig function
to set the |config_| variable.

So better to make them private to BuildLog now while nobody is using it outside.

Signed-off-by: Thiago Farina <tfarina@chromium.org>

1 files changed, 6 insertions, 2 deletions

diff --git a/src/build_log.cc b/src/build_log.cc
index 852a9d0..e3e96e5 100644
--- a/src/build_log.cc
+++ b/src/build_log.cc
@@ -35,11 +35,15 @@ namespace {
 const char kFileSignature[] = "# ninja log v%d\n";
 const int kCurrentVersion = 3;
 
-}
+}  // namespace
 
 BuildLog::BuildLog()
   : log_file_(NULL), config_(NULL), needs_recompaction_(false) {}
 
+BuildLog::~BuildLog() {
+  Close();
+}
+
 bool BuildLog::OpenForWrite(const string& path, string* err) {
   if (config_ && config_->dry_run)
     return true;  // Do nothing, report success.
@@ -144,7 +148,7 @@ bool BuildLog::Load(const string& path, string* err) {
       end_time = atoi(start);
       start = end + 1;
     }
-    
+
     if (log_version >= 3) {
       // In v3 we log the restat mtime.
       char* end = strchr(start, ' ');