guix-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

237/376: Don't recompile the same regex over and over


From: Ludovic Courtès
Subject: 237/376: Don't recompile the same regex over and over
Date: Wed, 28 Jan 2015 22:05:18 +0000

civodul pushed a commit to tag 1.8
in repository guix.

commit 3b5fa8d50cd7e4db0691884effcbdc7809ad9ed9
Author: Eelco Dolstra <address@hidden>
Date:   Fri Oct 3 21:29:20 2014 +0200

    Don't recompile the same regex over and over
---
 src/libexpr/names.cc |    5 ++---
 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 c2b2733..cda5aa1 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 ebe113e..4b3dcdd 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;



reply via email to

[Prev in Thread] Current Thread [Next in Thread]