about summary refs log tree commit diff
diff options
context:
space:
mode:
authorEelco Dolstra <eelco.dolstra@logicblox.com>2014-10-03T19·29+0200
committerEelco Dolstra <eelco.dolstra@logicblox.com>2014-10-03T19·29+0200
commit3b5fa8d50cd7e4db0691884effcbdc7809ad9ed9 (patch)
tree1aef7d220f01871ae2b5e0d49a585f966d321ccf
parent104e55bb7f593b1ac3c54ffda48a9d72af7fbe6b (diff)
Don't recompile the same regex over and over
-rw-r--r--src/libexpr/names.cc5
-rw-r--r--src/libexpr/names.hh6
2 files changed, 8 insertions, 3 deletions
diff --git a/src/libexpr/names.cc b/src/libexpr/names.cc
index c2b2733347f6..cda5aa1952ea 100644
--- a/src/libexpr/names.cc
+++ b/src/libexpr/names.cc
@@ -1,6 +1,5 @@
 #include "names.hh"
 #include "util.hh"
-#include "regex.hh"
 
 
 namespace nix {
@@ -34,8 +33,8 @@ DrvName::DrvName(const string & s) : hits(0)
 bool DrvName::matches(DrvName & n)
 {
     if (name != "*") {
-        Regex regex(name);
-        if (!regex.matches(n.name)) return false;
+        if (!regex) regex = std::shared_ptr<Regex>(new Regex(name));
+        if (!regex->matches(n.name)) return false;
     }
     if (version != "" && version != n.version) return false;
     return true;
diff --git a/src/libexpr/names.hh b/src/libexpr/names.hh
index ebe113e82ac1..4b3dcddf70d7 100644
--- a/src/libexpr/names.hh
+++ b/src/libexpr/names.hh
@@ -1,6 +1,9 @@
 #pragma once
 
+#include <memory>
+
 #include "types.hh"
+#include "regex.hh"
 
 namespace nix {
 
@@ -14,6 +17,9 @@ struct DrvName
     DrvName();
     DrvName(const string & s);
     bool matches(DrvName & n);
+
+private:
+    std::shared_ptr<Regex> regex;
 };
 
 typedef list<DrvName> DrvNames;