Remove silencing of warnings and fix warnings

Joseph T. Lyons created

Change summary

crates/zed2/src/main.rs          |  5 +----
crates/zed2/src/open_listener.rs | 11 +++++------
2 files changed, 6 insertions(+), 10 deletions(-)

Detailed changes

crates/zed2/src/main.rs 🔗

@@ -1,6 +1,3 @@
-#![allow(unused_variables, dead_code, unused_mut)]
-// todo!() this is to make transition easier.
-
 // Allow binary to be called Zed for a nice application menu when running executable directly
 #![allow(non_snake_case)]
 
@@ -411,7 +408,7 @@ async fn installation_id() -> Result<(String, bool)> {
     Ok((installation_id, false))
 }
 
-async fn restore_or_create_workspace(app_state: &Arc<AppState>, mut cx: AsyncAppContext) {
+async fn restore_or_create_workspace(app_state: &Arc<AppState>, cx: AsyncAppContext) {
     async_maybe!({
         if let Some(location) = workspace::last_opened_workspace_paths().await {
             cx.update(|cx| workspace::open_paths(location.paths().as_ref(), app_state, None, cx))?

crates/zed2/src/open_listener.rs 🔗

@@ -251,12 +251,11 @@ pub async fn handle_cli_connection(
                                 let wait = async move {
                                     if paths.is_empty() {
                                         let (done_tx, done_rx) = oneshot::channel();
-                                        let _subscription =
-                                            workspace.update(&mut cx, |workspace, cx| {
-                                                cx.on_release(move |_, _, _| {
-                                                    let _ = done_tx.send(());
-                                                })
-                                            });
+                                        let _subscription = workspace.update(&mut cx, |_, cx| {
+                                            cx.on_release(move |_, _, _| {
+                                                let _ = done_tx.send(());
+                                            })
+                                        });
                                         let _ = done_rx.await;
                                     } else {
                                         let _ = futures::future::try_join_all(item_release_futures)