diff options
| author | bors <bors@rust-lang.org> | 2022-12-12 05:16:50 +0000 |
|---|---|---|
| committer | bors <bors@rust-lang.org> | 2022-12-12 05:16:50 +0000 |
| commit | 53ca2598d3156b420931e5bcedbf49baeba0a68c (patch) | |
| tree | da759ca5c9722beae66a4b5c4bd73f9e9566f8dc | |
| parent | 2119f7d7d1f06059dfe5059fb62362efa56d4069 (diff) | |
| parent | c91dd22870d40315c3e0e5b45077516286e918fd (diff) | |
| download | rust-53ca2598d3156b420931e5bcedbf49baeba0a68c.tar.gz rust-53ca2598d3156b420931e5bcedbf49baeba0a68c.zip | |
Auto merge of #105160 - nnethercote:rm-Lit-token_lit, r=petrochenkov
Remove `token::Lit` from `ast::MetaItemLit`. Currently `ast::MetaItemLit` represents the literal kind twice. This PR removes that redundancy. Best reviewed one commit at a time. r? `@petrochenkov`
| -rw-r--r-- | src/attr.rs | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/src/attr.rs b/src/attr.rs index 2ac703b957b..c503eeeb9b3 100644 --- a/src/attr.rs +++ b/src/attr.rs @@ -260,7 +260,9 @@ impl Rewrite for ast::NestedMetaItem { fn rewrite(&self, context: &RewriteContext<'_>, shape: Shape) -> Option<String> { match self { ast::NestedMetaItem::MetaItem(ref meta_item) => meta_item.rewrite(context, shape), - ast::NestedMetaItem::Lit(ref l) => rewrite_literal(context, l.token_lit, l.span, shape), + ast::NestedMetaItem::Lit(ref l) => { + rewrite_literal(context, l.as_token_lit(), l.span, shape) + } } } } @@ -308,18 +310,18 @@ impl Rewrite for ast::MetaItem { }), )? } - ast::MetaItemKind::NameValue(ref literal) => { + ast::MetaItemKind::NameValue(ref lit) => { let path = rewrite_path(context, PathContext::Type, &None, &self.path, shape)?; // 3 = ` = ` let lit_shape = shape.shrink_left(path.len() + 3)?; - // `rewrite_literal` returns `None` when `literal` exceeds max + // `rewrite_literal` returns `None` when `lit` exceeds max // width. Since a literal is basically unformattable unless it // is a string literal (and only if `format_strings` is set), // we might be better off ignoring the fact that the attribute // is longer than the max width and continue on formatting. // See #2479 for example. - let value = rewrite_literal(context, literal.token_lit, literal.span, lit_shape) - .unwrap_or_else(|| context.snippet(literal.span).to_owned()); + let value = rewrite_literal(context, lit.as_token_lit(), lit.span, lit_shape) + .unwrap_or_else(|| context.snippet(lit.span).to_owned()); format!("{} = {}", path, value) } }) |
