mdebray/tvl-depot
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(3p/nix/libexpr): Ensure ExprOpUpdate merges into destination

Browse source 
... this fixes nixpkgs eval!
This commit is contained in:
Vincent Ambo 2020-05-23 20:30:23 +01:00
parent 55b1a47647
commit af762abadc
1 changed files with 5 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

22
third_party/nix/src/libexpr/eval.cc vendored
View file

@ -1223,7 +1223,7 @@ void ExprOpImpl::eval(EvalState& state, Env& env, Value& v) {
mkBool(v, !state.evalBool(env, e1, pos) || state.evalBool(env, e2, pos));
}
void ExprOpUpdate::eval(EvalState& state, Env& env, Value& v) {
void ExprOpUpdate::eval(EvalState& state, Env& env, Value& dest) {
Value v1;
Value v2;
state.evalAttrs(env, e1, v1);
@ -1231,23 +1231,11 @@ void ExprOpUpdate::eval(EvalState& state, Env& env, Value& v) {
state.nrOpUpdates++;
if (v1.attrs->empty()) {
v = v2;
return;
}
if (v2.attrs->empty()) {
v = v1;
return;
}
state.mkAttrs(dest, /* capacity = */ 0);
state.mkAttrs(v, /* capacity = */ 0);
/* Merge the sets, preferring values from the second set. Make
sure to keep the resulting vector in sorted order. */
v.attrs->merge(v1.attrs);
v.attrs->merge(v2.attrs);
state.nrOpUpdateValuesCopied += v.attrs->size();
/* Merge the sets, preferring values from the second set. */
dest.attrs->merge(*v1.attrs);
dest.attrs->merge(*v2.attrs);
}
void ExprOpConcatLists::eval(EvalState& state, Env& env, Value& v) {