diff options
| author | Igor Aleksanov <popzxc@yandex.ru> | 2020-08-12 14:08:55 +0300 |
|---|---|---|
| committer | Igor Aleksanov <popzxc@yandex.ru> | 2020-08-12 14:08:55 +0300 |
| commit | fcd4b0176f1544b389c9b028c547a1dfc92f9a56 (patch) | |
| tree | bcca1afb725766cdf1eeb75aed3618c3e517345d | |
| parent | d180b8bbe8c9e31953069bae387b5214fbb51d64 (diff) | |
| download | rust-fcd4b0176f1544b389c9b028c547a1dfc92f9a56.tar.gz rust-fcd4b0176f1544b389c9b028c547a1dfc92f9a56.zip | |
Revert style preference-related fixes
| -rw-r--r-- | crates/flycheck/src/lib.rs | 1 | ||||
| -rw-r--r-- | crates/ra_arena/src/map.rs | 10 | ||||
| -rw-r--r-- | crates/ra_parser/src/grammar/expressions/atom.rs | 8 | ||||
| -rw-r--r-- | crates/vfs/src/file_set.rs | 3 | ||||
| -rw-r--r-- | xtask/src/codegen/gen_syntax.rs | 20 |
5 files changed, 21 insertions, 21 deletions
diff --git a/crates/flycheck/src/lib.rs b/crates/flycheck/src/lib.rs index ec769459c15..7c38f5ef9d5 100644 --- a/crates/flycheck/src/lib.rs +++ b/crates/flycheck/src/lib.rs @@ -106,7 +106,6 @@ struct FlycheckActor { cargo_handle: Option<CargoHandle>, } -#[allow(clippy::large_enum_variant)] enum Event { Restart(Restart), CheckEvent(Option<cargo_metadata::Message>), diff --git a/crates/ra_arena/src/map.rs b/crates/ra_arena/src/map.rs index c1b58712c16..0f33907c0ae 100644 --- a/crates/ra_arena/src/map.rs +++ b/crates/ra_arena/src/map.rs @@ -13,18 +13,18 @@ pub struct ArenaMap<ID, V> { impl<T, V> ArenaMap<Idx<T>, V> { pub fn insert(&mut self, id: Idx<T>, t: V) { - let idx = Self::into_idx(id); + let idx = Self::to_idx(id); self.v.resize_with((idx + 1).max(self.v.len()), || None); self.v[idx] = Some(t); } pub fn get(&self, id: Idx<T>) -> Option<&V> { - self.v.get(Self::into_idx(id)).and_then(|it| it.as_ref()) + self.v.get(Self::to_idx(id)).and_then(|it| it.as_ref()) } pub fn get_mut(&mut self, id: Idx<T>) -> Option<&mut V> { - self.v.get_mut(Self::into_idx(id)).and_then(|it| it.as_mut()) + self.v.get_mut(Self::to_idx(id)).and_then(|it| it.as_mut()) } pub fn values(&self) -> impl Iterator<Item = &V> { @@ -39,7 +39,7 @@ impl<T, V> ArenaMap<Idx<T>, V> { self.v.iter().enumerate().filter_map(|(idx, o)| Some((Self::from_idx(idx), o.as_ref()?))) } - fn into_idx(id: Idx<T>) -> usize { + fn to_idx(id: Idx<T>) -> usize { u32::from(id.into_raw()) as usize } @@ -51,7 +51,7 @@ impl<T, V> ArenaMap<Idx<T>, V> { impl<T, V> std::ops::Index<Idx<V>> for ArenaMap<Idx<V>, T> { type Output = T; fn index(&self, id: Idx<V>) -> &T { - self.v[Self::into_idx(id)].as_ref().unwrap() + self.v[Self::to_idx(id)].as_ref().unwrap() } } diff --git a/crates/ra_parser/src/grammar/expressions/atom.rs b/crates/ra_parser/src/grammar/expressions/atom.rs index ca6569c9f25..0b01d3bc646 100644 --- a/crates/ra_parser/src/grammar/expressions/atom.rs +++ b/crates/ra_parser/src/grammar/expressions/atom.rs @@ -243,10 +243,12 @@ fn lambda_expr(p: &mut Parser) -> CompletedMarker { // test lambda_ret_block // fn main() { || -> i32 { 92 }(); } block_expr(p); - } else if p.at_ts(EXPR_FIRST) { - expr(p); } else { - p.error("expected expression"); + if p.at_ts(EXPR_FIRST) { + expr(p); + } else { + p.error("expected expression"); + } } m.complete(p, CLOSURE_EXPR) } diff --git a/crates/vfs/src/file_set.rs b/crates/vfs/src/file_set.rs index 9f11268eeed..e9196fcd2f9 100644 --- a/crates/vfs/src/file_set.rs +++ b/crates/vfs/src/file_set.rs @@ -19,9 +19,6 @@ impl FileSet { pub fn len(&self) -> usize { self.files.len() } - pub fn is_empty(&self) -> bool { - self.len() == 0 - } pub fn resolve_path(&self, anchor: FileId, path: &str) -> Option<FileId> { let mut base = self.paths[&anchor].clone(); base.pop(); diff --git a/xtask/src/codegen/gen_syntax.rs b/xtask/src/codegen/gen_syntax.rs index af9d63b06e7..cafad8070d2 100644 --- a/xtask/src/codegen/gen_syntax.rs +++ b/xtask/src/codegen/gen_syntax.rs @@ -91,16 +91,18 @@ fn generate_nodes(kinds: KindsSrc<'_>, grammar: &AstSrc) -> Result<String> { support::children(&self.syntax) } } - } else if let Some(token_kind) = field.token_kind() { - quote! { - pub fn #method_name(&self) -> Option<#ty> { - support::token(&self.syntax, #token_kind) - } - } } else { - quote! { - pub fn #method_name(&self) -> Option<#ty> { - support::child(&self.syntax) + if let Some(token_kind) = field.token_kind() { + quote! { + pub fn #method_name(&self) -> Option<#ty> { + support::token(&self.syntax, #token_kind) + } + } + } else { + quote! { + pub fn #method_name(&self) -> Option<#ty> { + support::child(&self.syntax) + } } } } |
