Merge pull request #68 from smolck/silence-warnings

Fix a few minor issues (compiler warnings)
vhakulinen · Aug 6, 2019 · dea3dd2 · dea3dd2
2 parents 3375207 + 4e5a728
commit dea3dd2
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git 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
@@ -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
@@ -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())