Merge pull request ninja-build#441 from tfarina/missing-virtual

Add missing 'virtual' annotation to ReadFile() override.

src/metrics.h

@@ -62,12 +62,10 @@ struct Metrics {
 /// Epoch varies between platforms; only useful for measuring elapsed time.
 int64_t GetTimeMillis();
 
-
 /// A simple stopwatch which returns the time
 /// in seconds since Restart() was called.
-class Stopwatch
-{
-public:
+class Stopwatch {
+ public:
   Stopwatch() : started_(0) {}
 
   /// Seconds since Restart() call.
@@ -80,7 +78,6 @@ class Stopwatch
   uint64_t Now() const;
 };
 
-
 /// The primary interface to metrics.  Use METRIC_RECORD("foobar") at the top
 /// of a function to get timing stats recorded for each call of the function.
 #define METRIC_RECORD(name)                                             \

src/ninja.cc

@@ -140,7 +140,7 @@ int GuessParallelism() {
 /// An implementation of ManifestParser::FileReader that actually reads
 /// the file.
 struct RealFileReader : public ManifestParser::FileReader {
-  bool ReadFile(const string& path, string* content, string* err) {
+  virtual bool ReadFile(const string& path, string* content, string* err) {
     return ::ReadFile(path, content, err) == 0;
   }
 };