maurice/snix
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

refactor(3p/nix/libexpr): Remove the nix::Symbol default constructor

Browse source 
Having a default constructor for this causes a variety of annoying
situations across the codebase in which this is initialised to an
unexpected value, leading to constant guarding against those
conditions.

It turns out there's actually no intrinsic reason that this default
constructor needs to exist. The biggest one was addressed in CL/1138
and this commit cleans up the remaining bits.

Change-Id: I4a847f50bc90e72f028598196592a7d8730a4e01
Reviewed-on: https://cl.tvl.fyi/c/depot/+/1139
Reviewed-by: isomer <isomer@tvl.fyi>
Tested-by: BuildkiteCI
This commit is contained in:
Vincent Ambo 2020-07-13 21:20:52 +01:00 • committed by tazjin
parent afd1367337
commit fa161e9a38
10 changed files with 50 additions and 37 deletions
Download patch file Download diff file Expand all files Collapse all files

11
third_party/nix/src/libexpr/eval.hh vendored
View file

@ -62,10 +62,13 @@ class EvalState {
SymbolTable symbols;
const Symbol sWith, sOutPath, sDrvPath, sType, sMeta, sName, sValue, sSystem,
sOverrides, sOutputs, sOutputName, sIgnoreNulls, sFile, sLine, sColumn,
sFunctor, sToString, sRight, sWrong, sStructuredAttrs, sBuilder, sArgs,
sOutputHash, sOutputHashAlgo, sOutputHashMode;
Symbol sDerivationNix;
sOutputs, sOutputName, sIgnoreNulls, sFile, sLine, sColumn, sFunctor,
sToString, sRight, sWrong, sStructuredAttrs, sBuilder, sArgs, sOutputHash,
sOutputHashAlgo, sOutputHashMode;
// Symbol representing the path to the built-in 'derivation.nix'
// file, set during primops initialisation.
std::optional<Symbol> sDerivationNix;
/* If set, force copying files to the Nix store even if they
already exist there. */