diff options
| author | Adam Joseph <adam@westernsemico.com> | 2022-11-01T00·13-0700 |
|---|---|---|
| committer | clbot <clbot@tvl.fyi> | 2022-11-04T00·30+0000 |
| commit | 06494742062e77036827dfc7c91dea507b44447f (patch) | |
| tree | df533a3a31c5fd8eb8170668907f86704d60e080 /tvix/eval/src/value/attrs.rs | |
| parent | 73fb474752e844ceb39643dad9804382a5766cba (diff) | |
fix(tvix/eval): remove impl PartialEq for Value r/5236
It isn't possible to implement PartialEq properly for Value, because any sensible implementation needs to force() thunks, which cannot be done without a `&mut VM`. The existing derive(PartialEq) has false negatives, which caused the bug which cl/7142 fixed. Fortunately that bug was easy to find, but a silent false negative deep within the bowels of nixpkgs could be a real nightmare to hunt down. Let's just remove the PartialEq impl for Value, and the other derive(PartialEq)'s that depend on it. Signed-off-by: Adam Joseph <adam@westernsemico.com> Change-Id: Iacd3726fefc7fc1edadcd7e9b586e04cf8466775 Reviewed-on: https://cl.tvl.fyi/c/depot/+/7144 Reviewed-by: kanepyork <rikingcoding@gmail.com> Reviewed-by: tazjin <tazjin@tvl.su> Tested-by: BuildkiteCI
Diffstat (limited to 'tvix/eval/src/value/attrs.rs')
| -rw-r--r-- | tvix/eval/src/value/attrs.rs | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tvix/eval/src/value/attrs.rs b/tvix/eval/src/value/attrs.rs index ca4d17178a5e..59e477db4acf 100644 --- a/tvix/eval/src/value/attrs.rs +++ b/tvix/eval/src/value/attrs.rs @@ -19,7 +19,7 @@ use super::Value; #[cfg(test)] mod tests; -#[derive(Clone, Debug, PartialEq)] +#[derive(Clone, Debug)] enum AttrsRep { Empty, Map(BTreeMap<NixString, Value>), @@ -79,7 +79,7 @@ impl AttrsRep { } #[repr(transparent)] -#[derive(Clone, Debug, PartialEq)] +#[derive(Clone, Debug)] pub struct NixAttrs(AttrsRep); impl TotalDisplay for NixAttrs { |