chore: Removed From<Token> implementation of BareName

It feels that `BareName` should not know about `Token`.
Removed that implementation, it was only used in two places.

Author: ngeor

rusty_parser/src/specific/core/bare_name.rs

@@ -1,11 +1,4 @@
-use crate::pc::Token;
 use rusty_common::{CaseInsensitiveString, Positioned};
 
 pub type BareName = CaseInsensitiveString;
 pub type BareNamePos = Positioned<BareName>;
-
-impl From<Token> for BareName {
-    fn from(token: Token) -> Self {
-        Self::new(token.text)
-    }
-}

rusty_parser/src/specific/core/expression.rs

@@ -883,7 +883,7 @@ pub mod property {
                     first_expr_pos,
                     |prev_expr_pos, (name_token, opt_q_token)| {
                         let property_name = Name::new(
-                            BareName::from(name_token),
+                            BareName::new(name_token.text),
                             opt_q_token.as_ref().map(token_to_type_qualifier),
                         );
                         prev_expr_pos.map(|prev_expr| {

rusty_parser/src/specific/core/name.rs

@@ -178,7 +178,7 @@ mod type_tests {
 ///
 /// Use case: user defined type elements or types.
 pub fn bare_name_without_dots() -> impl Parser<RcStringView, Output = BareName> {
-    ensure_no_trailing_dot_or_qualifier(identifier()).map(BareName::from)
+    ensure_no_trailing_dot_or_qualifier(identifier()).map(|token| BareName::new(token.text))
 }
 
 /// Parses an identifier token.