summaryrefslogtreecommitdiffstats
path: root/src/build_log.h
diff options
context:
space:
mode:
authorThiago Farina <tfarina@chromium.org>2011-12-20 21:53:31 (GMT)
committerThiago Farina <tfarina@chromium.org>2011-12-20 21:53:31 (GMT)
commitc33b19887e6b690ed18c42070bf2f0498b47f915 (patch)
tree9fd3342ce9c41ef3c5761528f01facc66e9ebaec /src/build_log.h
parent2f2ea98a267c75eec0179cbd34ce227a2f2f2db6 (diff)
downloadNinja-c33b19887e6b690ed18c42070bf2f0498b47f915.zip
Ninja-c33b19887e6b690ed18c42070bf2f0498b47f915.tar.gz
Ninja-c33b19887e6b690ed18c42070bf2f0498b47f915.tar.bz2
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>
Diffstat (limited to 'src/build_log.h')
-rw-r--r--src/build_log.h3
1 files changed, 2 insertions, 1 deletions
diff --git a/src/build_log.h b/src/build_log.h
index 99ae5a0..ee63ed2 100644
--- a/src/build_log.h
+++ b/src/build_log.h
@@ -34,7 +34,7 @@ struct Edge;
/// from it
struct BuildLog {
BuildLog();
- ~BuildLog() { Close(); }
+ ~BuildLog();
void SetConfig(BuildConfig* config) { config_ = config; }
bool OpenForWrite(const string& path, string* err);
@@ -69,6 +69,7 @@ struct BuildLog {
/// Rewrite the known log entries, throwing away old data.
bool Recompact(const string& path, string* err);
+ private:
typedef ExternalStringHashMap<LogEntry*>::Type Log;
Log log_;
FILE* log_file_;