diff options
author | Thiago Farina <tfarina@chromium.org> | 2011-12-20 21:53:31 (GMT) |
---|---|---|
committer | Thiago Farina <tfarina@chromium.org> | 2011-12-20 21:53:31 (GMT) |
commit | c33b19887e6b690ed18c42070bf2f0498b47f915 (patch) | |
tree | 9fd3342ce9c41ef3c5761528f01facc66e9ebaec /src/build_log.h | |
parent | 2f2ea98a267c75eec0179cbd34ce227a2f2f2db6 (diff) | |
download | Ninja-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.h | 3 |
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_; |