From 177a183e85467390bff24df6d72dbe0c9831d26a Mon Sep 17 00:00:00 2001 From: Aleksey Kladov Date: Tue, 28 Dec 2021 16:51:17 +0300 Subject: [PATCH] minor: simplify --- crates/hir/src/semantics.rs | 16 +++++----------- 1 file changed, 5 insertions(+), 11 deletions(-) diff --git a/crates/hir/src/semantics.rs b/crates/hir/src/semantics.rs index f3d8550a87..1070a26c26 100644 --- a/crates/hir/src/semantics.rs +++ b/crates/hir/src/semantics.rs @@ -448,8 +448,8 @@ impl<'db> SemanticsImpl<'db> { } fn resolve_derive_macro(&self, attr: &ast::Attr) -> Option> { - let macro_call_ids = self.derive_macro_calls(attr)?; - let res = macro_call_ids + let res = self + .derive_macro_calls(attr)? .iter() .map(|&call| { let loc: MacroCallLoc = self.db.lookup_intern_macro_call(call); @@ -460,9 +460,8 @@ impl<'db> SemanticsImpl<'db> { } fn expand_derive_macro(&self, attr: &ast::Attr) -> Option> { - let macro_call_ids = self.derive_macro_calls(attr)?; - - let expansions: Vec<_> = macro_call_ids + let res: Vec<_> = self + .derive_macro_calls(attr)? .iter() .map(|call| call.as_file()) .flat_map(|file_id| { @@ -471,12 +470,7 @@ impl<'db> SemanticsImpl<'db> { Some(node) }) .collect(); - - if expansions.is_empty() { - None - } else { - Some(expansions) - } + Some(res) } fn derive_macro_calls(&self, attr: &ast::Attr) -> Option> {