Merge #1374

1374: Implement `cargo lint` and fix some clippy errors r=alanhdu a=alanhdu

This creates a `cargo lint` command that runs clippy with certain lints disabled. I've also gone ahead and fixed some of the lint errors, although there are many more still to go.

cc #848 

Co-authored-by: Alan Du <[email protected]>

1 files changed, 3 insertions, 6 deletions

diff --git a/crates/thread_worker/src/lib.rs b/crates/thread_worker/src/lib.rs
index d67e44e38..d8d0d9bf2 100644
--- a/crates/thread_worker/src/lib.rs
+++ b/crates/thread_worker/src/lib.rs
@@ -19,13 +19,10 @@ impl Drop for ScopedThread {
         log::info!(".. {} terminated with {}", name, if res.is_ok() { "ok" } else { "err" });
 
         // escalate panic, but avoid aborting the process
-        match res {
-            Err(e) => {
-                if !thread::panicking() {
-                    panic!(e)
-                }
+        if let Err(e) = res {
+            if !thread::panicking() {
+                panic!(e)
             }
-            _ => (),
         }
     }
 }