diff --git a/crates/ide/src/ssr.rs b/crates/ide/src/ssr.rs
index 97ad66d358..57ec802614 100644
--- a/crates/ide/src/ssr.rs
+++ b/crates/ide/src/ssr.rs
@@ -13,7 +13,7 @@ pub(crate) fn ssr_assists(
     let mut ssr_assists = Vec::with_capacity(2);
 
     let (match_finder, comment_range) = match ide_ssr::ssr_from_comment(db, frange) {
-        Some((match_finder, comment_range)) => (match_finder, comment_range),
+        Some(ssr_data) => ssr_data,
         None => return ssr_assists,
     };
     let id = AssistId("ssr", AssistKind::RefactorRewrite);
diff --git a/crates/ide_assists/src/lib.rs b/crates/ide_assists/src/lib.rs
index 5a0047f037..723531078f 100644
--- a/crates/ide_assists/src/lib.rs
+++ b/crates/ide_assists/src/lib.rs
@@ -28,7 +28,7 @@ pub(crate) use crate::assist_context::{AssistContext, Assists};
 
 pub use assist_config::AssistConfig;
 
-#[derive(Debug, Clone, Copy, PartialEq, Eq, Hash)]
+#[derive(Debug, Clone, Copy, PartialEq, Eq)]
 pub enum AssistKind {
     // FIXME: does the None variant make sense? Probably not.
     None,
@@ -91,7 +91,7 @@ impl FromStr for AssistKind {
 
 /// Unique identifier of the assist, should not be shown to the user
 /// directly.
-#[derive(Debug, Clone, Copy, PartialEq, Eq, Hash)]
+#[derive(Debug, Clone, Copy, PartialEq, Eq)]
 pub struct AssistId(pub &'static str, pub AssistKind);
 
 /// A way to control how many asssist to resolve during the assist resolution.