From 263e91c9f9dc41b952dbaad50ab52f15dba002c3 Mon Sep 17 00:00:00 2001 From: Jeremy Wall Date: Sat, 23 Mar 2019 21:16:30 -0500 Subject: [PATCH] MAINT: Cleanup some todos and wording. --- src/build/mod.rs | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/build/mod.rs b/src/build/mod.rs index cf4ae06..d4a6485 100644 --- a/src/build/mod.rs +++ b/src/build/mod.rs @@ -141,7 +141,7 @@ macro_rules! eval_binary_expr { }; } -// TODO(jwall): Use the builder patter here. Just like AstWalker. +// TODO(jwall): Use the builder pattern here. Just like AstWalker. impl<'a> FileBuilder<'a> { /// Constructs a new Builder. pub fn new>( @@ -1019,7 +1019,6 @@ impl<'a> FileBuilder<'a> { fn eval_binary(&self, def: &BinaryOpDef, scope: &Scope) -> Result, Box> { let kind = &def.kind; if let &BinaryExprType::IN = kind { - // TODO Should we support this operation on strings too? return self.do_element_check(&def.left, &def.right, scope); } if let &BinaryExprType::IS = kind {