Merge #3492

3492: Simplify creation of `T[,]` r=matklad a=matklad



bors r+
🤖

Co-authored-by: Aleksey Kladov <[email protected]>

Author: bors[bot]

crates/ra_syntax/src/ast/edit.rs

@@ -139,7 +139,7 @@ impl ast::RecordFieldList {
         let mut to_insert: ArrayVec<[SyntaxElement; 4]> = ArrayVec::new();
         to_insert.push(space.into());
         to_insert.push(field.syntax().clone().into());
-        to_insert.push(tokens::comma().into());
+        to_insert.push(make::token(T![,]).into());
 
         macro_rules! after_l_curly {
             () => {{
@@ -160,7 +160,7 @@ impl ast::RecordFieldList {
                 {
                     InsertPosition::After(comma)
                 } else {
-                    to_insert.insert(0, tokens::comma().into());
+                    to_insert.insert(0, make::token(T![,]).into());
                     InsertPosition::After($anchor.syntax().clone().into())
                 }
             };

crates/ra_syntax/src/ast/make.rs

@@ -240,22 +240,13 @@ fn unroot(n: SyntaxNode) -> SyntaxNode {
 }
 
 pub mod tokens {
-    use crate::{ast, AstNode, Parse, SourceFile, SyntaxKind::*, SyntaxToken, T};
     use once_cell::sync::Lazy;
 
+    use crate::{ast, AstNode, Parse, SourceFile, SyntaxKind::*, SyntaxToken};
+
     pub(super) static SOURCE_FILE: Lazy<Parse<SourceFile>> =
         Lazy::new(|| SourceFile::parse("const C: <()>::Item = (1 != 1, 2 == 2, !true)\n;"));
 
-    pub fn comma() -> SyntaxToken {
-        SOURCE_FILE
-            .tree()
-            .syntax()
-            .descendants_with_tokens()
-            .filter_map(|it| it.into_token())
-            .find(|it| it.kind() == T![,])
-            .unwrap()
-    }
-
     pub fn single_space() -> SyntaxToken {
         SOURCE_FILE
             .tree()