about summary refs log tree commit diff
diff options
context:
space:
mode:
-rw-r--r--tvix/eval/src/compiler/optimiser.rs26
-rw-r--r--tvix/eval/src/warnings.rs4
2 files changed, 0 insertions, 30 deletions
diff --git a/tvix/eval/src/compiler/optimiser.rs b/tvix/eval/src/compiler/optimiser.rs
index ace5335d68..4c28b79a85 100644
--- a/tvix/eval/src/compiler/optimiser.rs
+++ b/tvix/eval/src/compiler/optimiser.rs
@@ -9,7 +9,6 @@ use ast::Expr;
 pub(super) fn optimise_expr(c: &mut Compiler, slot: LocalIdx, expr: ast::Expr) -> ast::Expr {
     match expr {
         Expr::BinOp(_) => optimise_bin_op(c, slot, expr),
-        Expr::Paren(_) => optimise_paren(c, expr),
         _ => expr.to_owned(),
     }
 }
@@ -124,28 +123,3 @@ fn optimise_bin_op(c: &mut Compiler, slot: LocalIdx, expr: ast::Expr) -> ast::Ex
 
     expr
 }
-
-/// Detect useless parenthesis around primitive expressions.
-fn optimise_paren(c: &mut Compiler, expr: ast::Expr) -> ast::Expr {
-    if let Expr::Paren(inner) = &expr {
-        let inner = inner.expr().unwrap();
-
-        if let Expr::Paren(_) = &inner {
-            c.emit_warning(&expr, WarningKind::UselessParens);
-            return optimise_paren(c, inner);
-        }
-
-        if let Expr::Literal(_)
-        | Expr::Str(_)
-        | Expr::Select(_)
-        | Expr::List(_)
-        | Expr::AttrSet(_)
-        | Expr::Ident(_) = &inner
-        {
-            c.emit_warning(&expr, WarningKind::UselessParens);
-            return inner;
-        }
-    }
-
-    expr
-}
diff --git a/tvix/eval/src/warnings.rs b/tvix/eval/src/warnings.rs
index 5395fc20b9..aa567f3027 100644
--- a/tvix/eval/src/warnings.rs
+++ b/tvix/eval/src/warnings.rs
@@ -17,7 +17,6 @@ pub enum WarningKind {
     DeadCode,
     EmptyInherit,
     EmptyLet,
-    UselessParens,
 
     /// Tvix internal warning for features triggered by users that are
     /// not actually implemented yet, but do not cause runtime failures.
@@ -101,8 +100,6 @@ impl EvalWarning {
 
             WarningKind::EmptyLet => "this `let`-expression contains no bindings".to_string(),
 
-            WarningKind::UselessParens => "these parenthesis can be removed".to_string(),
-
             WarningKind::NotImplemented(what) => {
                 format!("feature not yet implemented in tvix: {}", what)
             }
@@ -123,7 +120,6 @@ impl EvalWarning {
             WarningKind::DeadCode => "W008",
             WarningKind::EmptyInherit => "W009",
             WarningKind::EmptyLet => "W010",
-            WarningKind::UselessParens => "W011",
 
             WarningKind::NotImplemented(_) => "W999",
         }