diff options
author | Eelco Dolstra <eelco.dolstra@logicblox.com> | 2014-10-03T19·29+0200 |
---|---|---|
committer | Eelco Dolstra <eelco.dolstra@logicblox.com> | 2014-10-03T19·29+0200 |
commit | 3b5fa8d50cd7e4db0691884effcbdc7809ad9ed9 (patch) | |
tree | 1aef7d220f01871ae2b5e0d49a585f966d321ccf | |
parent | 104e55bb7f593b1ac3c54ffda48a9d72af7fbe6b (diff) |
Don't recompile the same regex over and over
-rw-r--r-- | src/libexpr/names.cc | 5 | ||||
-rw-r--r-- | src/libexpr/names.hh | 6 |
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; |