+1 -1

compiler-cli/src/fs/tests.rs

View file Reviewed

··· 113 113 super::write(&path.join(file), contents).unwrap(); 114 114 } 115 115 116 116 - let mut chosen_files = super::erlang_files(&path).collect_vec(); 116 116 + let mut chosen_files = super::erlang_files(path).collect_vec(); 117 117 chosen_files.sort_unstable(); 118 118 119 119 let expected_files = included_files.iter().map(|s| path.join(s)).collect_vec();

+1 -1

compiler-core/src/erlang/tests.rs

View file Reviewed

··· 118 118 built_module.attach_doc_and_module_comments(); 119 119 120 120 let line_numbers = LineNumbers::new(src); 121 121 - module(&built_module.ast, &line_numbers, &root) 121 121 + module(&built_module.ast, &line_numbers, root) 122 122 .unwrap() 123 123 .replace( 124 124 std::include_str!("../../templates/echo.erl"),

+4 -4

compiler-core/src/exhaustiveness.rs

View file Reviewed

··· 233 233 // and try again. 234 234 Pattern::Assign { name, pattern } => { 235 235 self.body.assign(name.clone(), check.var.clone()); 236 236 - check.pattern = pattern.clone(); 236 236 + check.pattern = *pattern; 237 237 continue; 238 238 } 239 239 // All other patterns are not unconditional, so we just keep them. ··· 341 341 | Pattern::BitArray { .. } 342 342 | Pattern::EmptyList => vec![], 343 343 344 344 - Pattern::StringPrefix { rest, .. } => vec![rest.clone()], 344 344 + Pattern::StringPrefix { rest, .. } => vec![*rest], 345 345 346 346 Pattern::Tuple { elements } => elements.clone(), 347 347 Pattern::Constructor { arguments, .. } => arguments.clone(), 348 348 - Pattern::List { first, rest } => vec![first.clone(), rest.clone()], 348 348 + Pattern::List { first, rest } => vec![*first, *rest], 349 349 } 350 350 } 351 351 } ··· 820 820 821 821 // If the type is a tuple there's only one runtime check we could 822 822 // perform that actually makes sense. 823 823 - BranchMode::Tuple { elems } => vec![self.is_tuple_check(&elems)], 823 823 + BranchMode::Tuple { elems } => vec![self.is_tuple_check(elems)], 824 824 825 825 // If the type being matched on is a list we know the resulting 826 826 // decision tree node is only ever going to have two different paths:

+2 -2

compiler-core/src/exhaustiveness/missing_patterns.rs

View file Reviewed

··· 110 110 for (check, body) in choices { 111 111 let term = check_to_term(var.clone(), check, environment); 112 112 terms.push(term); 113 113 - add_missing_patterns(&body, subjects, terms, missing, environment); 113 113 + add_missing_patterns(body, subjects, terms, missing, environment); 114 114 _ = terms.pop(); 115 115 } 116 116 ··· 121 121 for (check, body) in choices { 122 122 let term = check_to_term(var.clone(), check, environment); 123 123 terms.push(term); 124 124 - add_missing_patterns(&body, subjects, terms, missing, environment); 124 124 + add_missing_patterns(body, subjects, terms, missing, environment); 125 125 _ = terms.pop(); 126 126 } 127 127 }

+1 -1

compiler-core/src/type_/expression.rs

View file Reviewed

··· 3994 3994 subject: Arc<Type>, 3995 3995 pattern: &TypedPattern, 3996 3996 ) -> Result<(), Error> { 3997 3997 - let mut case = exhaustiveness::CaseToCompile::new(&vec![subject]); 3997 3997 + let mut case = exhaustiveness::CaseToCompile::new(&[subject]); 3998 3998 case.add_pattern(pattern); 3999 3999 let output = case.compile(self.environment); 4000 4000

+2 -2

test-output/src/tests/echo.rs

View file Reviewed

··· 44 44 target: Option<Target>, 45 45 runtime: Option<Runtime>, 46 46 ) -> String { 47 47 - let _ = fs::delete_directory(&paths.build_directory()).expect("delete build directory content"); 47 47 + fs::delete_directory(&paths.build_directory()).expect("delete build directory content"); 48 48 49 49 let Command { 50 50 program, ··· 53 53 cwd: _, 54 54 stdio: _, 55 55 } = run::setup( 56 56 - &paths, 56 56 + paths, 57 57 vec![], 58 58 target, 59 59 runtime,