diff --git a/src/ui/popupmenu/lazy_loader.rs b/src/ui/popupmenu/lazy_loader.rs
index c5bf0942..ee0db19b 100644
--- a/src/ui/popupmenu/lazy_loader.rs
+++ b/src/ui/popupmenu/lazy_loader.rs
@@ -19,7 +19,8 @@ struct State {
     /// Once we're loaded some (or all) data, this closure gets called if
     /// one exists. The first value in the tuple can is indication on the
     /// number of items needed before calling the closure.
-    once_loaded: Option<(Option<i32>, Box<Fn(&Vec<CompletionItemWidgetWrap>)>)>,
+    once_loaded:
+        Option<(Option<i32>, Box<dyn Fn(&Vec<CompletionItemWidgetWrap>)>)>,
 
     list: gtk::ListBox,
     css_provider: gtk::CssProvider,
diff --git a/src/ui/popupmenu/popupmenu.rs b/src/ui/popupmenu/popupmenu.rs
index 9feb4261..8000a41c 100644
--- a/src/ui/popupmenu/popupmenu.rs
+++ b/src/ui/popupmenu/popupmenu.rs
@@ -435,7 +435,7 @@ impl Popupmenu {
                 list.select_row(Some(&item.row));
 
                 {
-                    let mut id = Arc::new(ThreadGuard::new(None));
+                    let id = Arc::new(ThreadGuard::new(None));
                     let id_ref = id.clone();
                     let list = list.clone();
                     // Ensure that the row is in the view, but make sure first
diff --git a/src/ui/ui.rs b/src/ui/ui.rs
index 74c44169..d56a8527 100644
--- a/src/ui/ui.rs
+++ b/src/ui/ui.rs
@@ -400,7 +400,7 @@ fn handle_request(
         Request::CursorTooltipStyles => {
             let styles = state.cursor_tooltip.get_styles();
 
-            let mut res: Vec<Value> =
+            let res: Vec<Value> =
                 styles.into_iter().map(|s| s.into()).collect();
 
             Ok(res.into())