Avoid code duplication in {MetaItem,MetaItemKind}::value_str.

nnethercote · nnethercote · commit 7d1e5a485c66 · 2022-08-16T11:17:14.000+10:00
The two methods are almost identical.
diff --git a/compiler/rustc_ast/src/attr/mod.rs b/compiler/rustc_ast/src/attr/mod.rs
@@ -182,13 +182,7 @@ impl MetaItem {
     }
 
     pub fn value_str(&self) -> Option<Symbol> {
-        match self.kind {
-            MetaItemKind::NameValue(ref v) => match v.kind {
-                LitKind::Str(ref s, _) => Some(*s),
-                _ => None,
-            },
-            _ => None,
-        }
+        self.kind.value_str()
     }
 
     pub fn meta_item_list(&self) -> Option<&[NestedMetaItem]> {

Original file line number	Diff line number	Diff line change
`@@ -182,13 +182,7 @@ impl MetaItem {`
`182`	`182`	`}`
`183`	`183`
`184`	`184`	`pub fn value_str(&self) -> Option<Symbol> {`
`185`		`- match self.kind {`
`186`		`- MetaItemKind::NameValue(ref v) => match v.kind {`
`187`		`- LitKind::Str(ref s, _) => Some(*s),`
`188`		`- _ => None,`
`189`		`- },`
`190`		`- _ => None,`
`191`		`- }`
	`185`	`+ self.kind.value_str()`
`192`	`186`	`}`
`193`	`187`
`194`	`188`	`pub fn meta_item_list(&self) -> Option<&[NestedMetaItem]> {`