From 2300a372676d8cd1a12ac02ec63f958c6e168b91 Mon Sep 17 00:00:00 2001
From: Tom French <tom@tomfren.ch>
Date: Mon, 24 Feb 2025 17:59:21 +0000
Subject: [PATCH] chore: clippy fixes

---
 .../src/ssa/checks/check_for_underconstrained_values.rs    | 2 +-
 tooling/lsp/src/modules.rs                                 | 7 ++-----
 2 files changed, 3 insertions(+), 6 deletions(-)

diff --git a/compiler/noirc_evaluator/src/ssa/checks/check_for_underconstrained_values.rs b/compiler/noirc_evaluator/src/ssa/checks/check_for_underconstrained_values.rs
index 46cf0f12afe..88b4fb18fb3 100644
--- a/compiler/noirc_evaluator/src/ssa/checks/check_for_underconstrained_values.rs
+++ b/compiler/noirc_evaluator/src/ssa/checks/check_for_underconstrained_values.rs
@@ -327,7 +327,7 @@ impl DependencyContext {
         function.dfg[block].instructions().iter().for_each(|instruction| {
             if let Instruction::Call { func, arguments } = &function.dfg[*instruction] {
                 if let Value::Function(callee) = &function.dfg[*func] {
-                    if all_functions[&callee].runtime().is_brillig() {
+                    if all_functions[callee].runtime().is_brillig() {
                         // Skip already visited locations (happens often in unrolled functions)
                         let call_stack = function.dfg.get_instruction_call_stack(*instruction);
                         let location = call_stack.last();
diff --git a/tooling/lsp/src/modules.rs b/tooling/lsp/src/modules.rs
index 56529949b3e..4e7ef64b2c0 100644
--- a/tooling/lsp/src/modules.rs
+++ b/tooling/lsp/src/modules.rs
@@ -248,15 +248,12 @@ pub(crate) fn module_def_id_relative_path(
             interner,
         )
     } else {
-        let Some(module_full_path) = relative_module_full_path(
+        relative_module_full_path(
             module_def_id,
             current_module_id,
             current_module_parent_id,
             interner,
-        ) else {
-            return None;
-        };
-        module_full_path
+        )?
     };
 
     let path = if defining_module.is_some() || !matches!(module_def_id, ModuleDefId::ModuleId(..)) {