Skip to content

Commit bc0578f

Browse files
committed
fix: match the standard Rust formatting && use .kind() instead of iter
1 parent cd49553 commit bc0578f

File tree

2 files changed

+14
-22
lines changed

2 files changed

+14
-22
lines changed

crates/hir/src/display.rs

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
//! HirDisplay implementations for various hir types.
22
use hir_def::{
3-
data::adt::VariantData,
3+
data::adt::{StructKind, VariantData},
44
generics::{
55
TypeOrConstParamData, TypeParamProvenance, WherePredicate, WherePredicateTypeTarget,
66
},
@@ -164,12 +164,9 @@ impl HirDisplay for Struct {
164164
let def_id = GenericDefId::AdtId(AdtId::StructId(self.id));
165165
write_generic_params(def_id, f)?;
166166

167-
let field_is_tuple_index =
168-
self.variant_data(f.db).fields().iter().any(|(_, f)| f.name.as_tuple_index().is_some());
169-
170-
if field_is_tuple_index {
167+
let variant_data = self.variant_data(f.db);
168+
if let StructKind::Tuple = variant_data.kind() {
171169
f.write_char('(')?;
172-
let variant_data = self.variant_data(f.db);
173170
let mut it = variant_data.fields().iter().peekable();
174171

175172
while let Some((id, _)) = it.next() {
@@ -188,7 +185,7 @@ impl HirDisplay for Struct {
188185

189186
let fields = self.fields(f.db);
190187
if !fields.is_empty() {
191-
f.write_str("\n{\n")?;
188+
f.write_str(" {\n")?;
192189
for field in self.fields(f.db) {
193190
f.write_str(" ")?;
194191
field.hir_fmt(f)?;
@@ -212,7 +209,7 @@ impl HirDisplay for Enum {
212209

213210
let variants = self.variants(f.db);
214211
if !variants.is_empty() {
215-
f.write_str("\n{\n")?;
212+
f.write_str(" {\n")?;
216213
for variant in variants {
217214
f.write_str(" ")?;
218215
variant.hir_fmt(f)?;
@@ -240,7 +237,7 @@ impl HirDisplay for Union {
240237
for field in self.fields(f.db) {
241238
f.write_str(" ")?;
242239
field.hir_fmt(f)?;
243-
f.write_char('\n')?;
240+
f.write_str(",\n")?;
244241
}
245242
f.write_str("}")?;
246243
}

crates/ide/src/hover/tests.rs

Lines changed: 8 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1136,8 +1136,7 @@ impl Thing {
11361136
```
11371137
11381138
```rust
1139-
struct Thing
1140-
{
1139+
struct Thing {
11411140
x: u32,
11421141
}
11431142
```
@@ -1158,8 +1157,7 @@ impl Thing {
11581157
```
11591158
11601159
```rust
1161-
struct Thing
1162-
{
1160+
struct Thing {
11631161
x: u32,
11641162
}
11651163
```
@@ -1180,8 +1178,7 @@ impl Thing {
11801178
```
11811179
11821180
```rust
1183-
enum Thing
1184-
{
1181+
enum Thing {
11851182
A,
11861183
}
11871184
```
@@ -1202,8 +1199,7 @@ impl Thing {
12021199
```
12031200
12041201
```rust
1205-
enum Thing
1206-
{
1202+
enum Thing {
12071203
A,
12081204
}
12091205
```
@@ -2017,8 +2013,7 @@ fn test_hover_layout_of_enum() {
20172013
```
20182014
20192015
```rust
2020-
enum Foo
2021-
{
2016+
enum Foo {
20222017
Variant1(u8, u16),
20232018
Variant2(i32, u8, i64),
20242019
} // size = 16 (0x10), align = 8, niches = 254
@@ -2362,7 +2357,7 @@ fn main() { let s$0t = S{ f1:0 }; }
23622357
focus_range: 7..8,
23632358
name: "S",
23642359
kind: Struct,
2365-
description: "struct S\n{\n f1: u32,\n}",
2360+
description: "struct S {\n f1: u32,\n}",
23662361
},
23672362
},
23682363
],
@@ -2395,7 +2390,7 @@ fn main() { let s$0t = S{ f1:Arg(0) }; }
23952390
focus_range: 24..25,
23962391
name: "S",
23972392
kind: Struct,
2398-
description: "struct S<T>\n{\n f1: T,\n}",
2393+
description: "struct S<T> {\n f1: T,\n}",
23992394
},
24002395
},
24012396
HoverGotoTypeData {
@@ -2454,7 +2449,7 @@ fn main() { let s$0t = S{ f1: S{ f1: Arg(0) } }; }
24542449
focus_range: 24..25,
24552450
name: "S",
24562451
kind: Struct,
2457-
description: "struct S<T>\n{\n f1: T,\n}",
2452+
description: "struct S<T> {\n f1: T,\n}",
24582453
},
24592454
},
24602455
HoverGotoTypeData {

0 commit comments

Comments
 (0)