Merge #789

789: remove useless hash r=matklad a=matklad



Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

crates/ra_hir/src/macros.rs

@@ -17,7 +17,7 @@ use ra_syntax::{
 use crate::{MacroCallId, PersistentHirDatabase};
 
 // Hard-coded defs for now :-(
-#[derive(Debug, Clone, PartialEq, Eq, Hash)]
+#[derive(Debug, Clone, PartialEq, Eq)]
 pub enum MacroDef {
     Vec,
     QueryGroup,

crates/ra_hir/src/module_tree.rs

@@ -115,7 +115,7 @@ pub struct ModuleTree {
     links: Arena<LinkId, LinkData>,
 }
 
-#[derive(Debug, PartialEq, Eq, Hash)]
+#[derive(Debug, PartialEq, Eq)]
 pub struct ModuleData {
     file_id: HirFileId,
     /// Points to `ast::Module`, `None` for the whole file.

crates/ra_mbe/src/lib.rs

@@ -26,7 +26,7 @@ pub use tt::{Delimiter, Punct};
 
 pub use crate::syntax_bridge::ast_to_token_tree;
 
-/// This struct contains AST for a single `macro_rules` defenition. What might
+/// This struct contains AST for a single `macro_rules` definition. What might
 /// be very confusing is that AST has almost exactly the same shape as
 /// `tt::TokenTree`, but there's a crucial difference: in macro rules, `$ident`
 /// and `$()*` have special meaning (see `Var` and `Repeat` data structures)

crates/ra_mbe/src/mbe_expander.rs

@@ -1,4 +1,4 @@
-/// This module takes a (parsed) defenition of `macro_rules` invocation, a
+/// This module takes a (parsed) definition of `macro_rules` invocation, a
 /// `tt::TokenTree` representing an argument of macro invocation, and produces a
 /// `tt::TokenTree` for the result of the expansion.
 use rustc_hash::FxHashMap;