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

chore(tvix/eval): address current clippy lints

Browse source 
Change-Id: I76326c20a525044e89d3cd1392a29faa3414ca04
Reviewed-on: https://cl.tvl.fyi/c/depot/+/6529
Reviewed-by: sterni <sternenseemann@systemli.org>
Tested-by: BuildkiteCI
This commit is contained in:
Vincent Ambo 2022-09-12 00:08:45 +03:00 committed by tazjin
parent 1844c788f5
commit 2fe193ce41
1 changed files with 1 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
tvix/eval/src/compiler/mod.rs
View file

@ -716,7 +716,6 @@ impl Compiler<'_, '_> {
// Otherwise, this variable is missing. // Otherwise, this variable is missing.
self.emit_error(self.span_for(&node), ErrorKind::UnknownStaticVariable); self.emit_error(self.span_for(&node), ErrorKind::UnknownStaticVariable);
return;
} }
LocalPosition::Known(idx) => { LocalPosition::Known(idx) => {
@ -1245,7 +1244,7 @@ impl Compiler<'_, '_> {
fn expr_static_attr_str(&self, node: &ast::Attr) -> Option<String> { fn expr_static_attr_str(&self, node: &ast::Attr) -> Option<String> {
match node { match node {
ast::Attr::Ident(ident) => Some(ident.ident_token().unwrap().text().into()), ast::Attr::Ident(ident) => Some(ident.ident_token().unwrap().text().into()),
ast::Attr::Str(s) => self.expr_static_str(&s), ast::Attr::Str(s) => self.expr_static_str(s),
// The dynamic node type is just a wrapper. C++ Nix does not // The dynamic node type is just a wrapper. C++ Nix does not
// care about the dynamic wrapper when determining whether the // care about the dynamic wrapper when determining whether the