about summary refs log tree commit diff
path: root/src/libexpr/eval.cc
diff options
context:
space:
mode:
authorEelco Dolstra <e.dolstra@tudelft.nl>2010-04-16T14·03+0000
committerEelco Dolstra <e.dolstra@tudelft.nl>2010-04-16T14·03+0000
commit8ca4a001cb9e8ca2556c26a1b559b0322a8fb46a (patch)
tree2f068d0f0e4e4ff6cb29e01e2859c8d099c34eaa /src/libexpr/eval.cc
parent497e4ad12650e27ecbaf0e056fe0c54bc12a138b (diff)
* Improve sharing a bit.
Diffstat (limited to 'src/libexpr/eval.cc')
-rw-r--r--src/libexpr/eval.cc11
1 files changed, 7 insertions, 4 deletions
diff --git a/src/libexpr/eval.cc b/src/libexpr/eval.cc
index 89534688ece0..ce466ded491f 100644
--- a/src/libexpr/eval.cc
+++ b/src/libexpr/eval.cc
@@ -652,15 +652,18 @@ void ExprWith::eval(EvalState & state, Env & env, Value & v)
         /* Because the first `with' may be a shallow copy of another
            attribute set (through a tCopy node), we need to clone its
            `attrs' before modifying them. */
-        env2.values[0].attrs = new Bindings(*env2.values[0].attrs);
+        Bindings * old(env2.values[0].attrs);
+        state.mkAttrs(env2.values[0]);
+        foreach (Bindings::iterator, i, *old)
+            mkCopy((*env2.values[0].attrs)[i->first], i->second);
 
         foreach (Bindings::iterator, i, *env3->values[0].attrs) {
             Bindings::iterator j = env2.values[0].attrs->find(i->first);
             if (j == env2.values[0].attrs->end())
-                (*env2.values[0].attrs)[i->first] = i->second; // !!! sharing
+                mkCopy((*env2.values[0].attrs)[i->first], i->second);
         }
     }
-        
+
     state.eval(env2, body, v);
 }
 
@@ -731,7 +734,7 @@ void ExprOpUpdate::eval(EvalState & state, Env & env, Value & v)
     state.forceAttrs(v2);
     
     foreach (Bindings::iterator, i, *v2.attrs)
-        (*v.attrs)[i->first] = i->second; // !!! sharing
+        mkCopy((*v.attrs)[i->first], i->second);
 }