Merge pull request #936 from nico/cleanup

Minor cleanups. No behavior change.

3 files changed, 4 insertions, 10 deletions

diff --git a/src/eval_env.cc b/src/eval_env.cc
index 8f5c8ee..e03a82e 100644
--- a/src/eval_env.cc
+++ b/src/eval_env.cc
@@ -74,7 +74,7 @@ bool Rule::IsReservedBinding(const string& var) {
       var == "rspfile_content";
 }
 
-const map<string, const Rule*> BindingEnv::GetRules() const {
+const map<string, const Rule*>& BindingEnv::GetRules() const {
   return rules_;
 }
 
diff --git a/src/eval_env.h b/src/eval_env.h
index 7b9bdf5..28c4d16 100644
--- a/src/eval_env.h
+++ b/src/eval_env.h
@@ -28,7 +28,6 @@ struct Rule;
 struct Env {
   virtual ~Env() {}
   virtual string LookupVariable(const string& var) = 0;
-  virtual const Rule* LookupRule(const string& rule_name) = 0;
 };
 
 /// A tokenized string that contains variable references.
@@ -77,7 +76,7 @@ struct Rule {
 /// as well as a pointer to a parent scope.
 struct BindingEnv : public Env {
   BindingEnv() : parent_(NULL) {}
-  explicit BindingEnv(Env* parent) : parent_(parent) {}
+  explicit BindingEnv(BindingEnv* parent) : parent_(parent) {}
 
   virtual ~BindingEnv() {}
   virtual string LookupVariable(const string& var);
@@ -85,7 +84,7 @@ struct BindingEnv : public Env {
   void AddRule(const Rule* rule);
   const Rule* LookupRule(const string& rule_name);
   const Rule* LookupRuleCurrentScope(const string& rule_name);
-  const map<string, const Rule*> GetRules() const;
+  const map<string, const Rule*>& GetRules() const;
 
   void AddBinding(const string& key, const string& val);
 
@@ -100,7 +99,7 @@ struct BindingEnv : public Env {
 private:
   map<string, string> bindings_;
   map<string, const Rule*> rules_;
-  Env* parent_;
+  BindingEnv* parent_;
 };
 
 #endif  // NINJA_EVAL_ENV_H_
diff --git a/src/graph.cc b/src/graph.cc
index 76c4e9a..e3253fd 100644
--- a/src/graph.cc
+++ b/src/graph.cc
@@ -207,7 +207,6 @@ struct EdgeEnv : public Env {
   EdgeEnv(Edge* edge, EscapeKind escape)
       : edge_(edge), escape_in_out_(escape) {}
   virtual string LookupVariable(const string& var);
-  virtual const Rule* LookupRule(const string& rule_name);
 
   /// Given a span of Nodes, construct a list of paths suitable for a command
   /// line.
@@ -219,10 +218,6 @@ struct EdgeEnv : public Env {
   EscapeKind escape_in_out_;
 };
 
-const Rule* EdgeEnv::LookupRule(const string& rule_name) {
-  return NULL;
-}
-
 string EdgeEnv::LookupVariable(const string& var) {
   if (var == "in" || var == "in_newline") {
     int explicit_deps_count = edge_->inputs_.size() - edge_->implicit_deps_ -