]> git.lizzy.rs Git - rust.git/commitdiff
remove uselessly wrapped ?s. (clippy::meedless_question_mark
authorMatthias Krüger <matthias.krueger@famsik.de>
Wed, 17 Mar 2021 01:19:40 +0000 (02:19 +0100)
committerMatthias Krüger <matthias.krueger@famsik.de>
Wed, 17 Mar 2021 01:19:40 +0000 (02:19 +0100)
let x = Some(3);

let y = Some(x?);
can just be:
let y = x

crates/hir_expand/src/lib.rs
crates/proc_macro_api/src/process.rs
crates/rust-analyzer/src/handlers.rs

index eee430af135c396a842da56a4518f4a38c6c9379..7532d00b833a853853ffa0034a728c4c699eafcc 100644 (file)
@@ -475,7 +475,7 @@ fn original_range_opt(
     let single = skip_trivia_token(node.value.first_token()?, Direction::Next)?
         == skip_trivia_token(node.value.last_token()?, Direction::Prev)?;
 
-    Some(node.value.descendants().find_map(|it| {
+    node.value.descendants().find_map(|it| {
         let first = skip_trivia_token(it.first_token()?, Direction::Next)?;
         let first = ascend_call_token(db, &expansion, node.with_value(first))?;
 
@@ -487,7 +487,7 @@ fn original_range_opt(
         }
 
         Some(first.with_value(first.value.text_range().cover(last.value.text_range())))
-    })?)
+    })
 }
 
 fn ascend_call_token(
index 3ce851fe82ce72fed4bf9ac2272e9e80cf9748bc..30bb1b687b1c31025b8bf852b184ebd49ae513d6 100644 (file)
@@ -154,5 +154,5 @@ fn send_request(
     req: Request,
 ) -> io::Result<Option<Response>> {
     req.write(&mut writer)?;
-    Ok(Response::read(&mut reader)?)
+    Response::read(&mut reader)
 }
index 6fb7da79cb149056b5550e3a77f727f0a1758524..b6f484e5157f74c976a812963beb025c3c06ef61 100644 (file)
@@ -1134,7 +1134,7 @@ pub(crate) fn handle_code_lens_resolve(
 ) -> Result<CodeLens> {
     let annotation = from_proto::annotation(&snap, code_lens)?;
 
-    Ok(to_proto::code_lens(&snap, snap.analysis.resolve_annotation(annotation)?)?)
+    to_proto::code_lens(&snap, snap.analysis.resolve_annotation(annotation)?)
 }
 
 pub(crate) fn handle_document_highlight(