From 2e2bdac00092f8fcb96283da697a7a820a8c8978 Mon Sep 17 00:00:00 2001
From: Daniel <kingdread@gmx.de>
Date: Fri, 12 Nov 2021 14:08:35 +0100
Subject: Minor code smell fixes

---
 src/fexpr/grammar.lalrpop | 19 +++++--------------
 1 file changed, 5 insertions(+), 14 deletions(-)

(limited to 'src/fexpr/grammar.lalrpop')

diff --git a/src/fexpr/grammar.lalrpop b/src/fexpr/grammar.lalrpop
index 6333783..ce78396 100644
--- a/src/fexpr/grammar.lalrpop
+++ b/src/fexpr/grammar.lalrpop
@@ -102,7 +102,6 @@ Regex: Regex = {
         .map_err(|error| ParseError::User {
             error: FError {
                 location: l,
-                data: s.to_string(),
                 kind: error.into(),
             }
         }),
@@ -112,7 +111,6 @@ Regex: Regex = {
         .map_err(|error| ParseError::User {
             error: FError {
                 location: l,
-                data: s.to_string(),
                 kind: error.into(),
             }
         }),
@@ -122,8 +120,7 @@ FightOutcome: FightOutcome = {
     <l:@L> <w:word> =>? w.parse().map_err(|_| ParseError::User {
         error: FError {
             location: l,
-            data: w.into(),
-            kind: FErrorKind::InvalidFightOutcome,
+            kind: FErrorKind::FightOutcome,
         }
     }),
 }
@@ -132,8 +129,7 @@ Weekday: Weekday = {
     <l:@L> <w:word> =>? w.parse().map_err(|_| ParseError::User {
         error: FError {
             location: l,
-            data: w.into(),
-            kind: FErrorKind::InvalidWeekday,
+            kind: FErrorKind::Weekday,
         }
     }),
 }
@@ -142,16 +138,14 @@ Encounter: Encounter = {
     <l:@L> <w:word> =>? w.parse().map_err(|_| ParseError::User {
         error: FError {
             location: l,
-            data: w.into(),
-            kind: FErrorKind::InvalidBoss,
+            kind: FErrorKind::Boss,
         }
     }),
 
     <l:@L> <s:string> =>? s[1..s.len() -1].parse().map_err(|_| ParseError::User {
         error: FError {
             location: l,
-            data: s.into(),
-            kind: FErrorKind::InvalidBoss,
+            kind: FErrorKind::Boss,
         }
     }),
 }
@@ -160,8 +154,7 @@ PlayerClass: PlayerClass = {
     <l:@L> <w:word> =>? w.parse().map_err(|_| ParseError::User {
         error: FError {
             location: l,
-            data: w.into(),
-            kind: FErrorKind::InvalidClass,
+            kind: FErrorKind::Class,
         }
     }),
 }
@@ -171,7 +164,6 @@ Date: DateTime<Utc> = {
         .map_err(|error| ParseError::User {
             error: FError {
                 location: l,
-                data: d.into(),
                 kind: error.into(),
             }
         })
@@ -180,7 +172,6 @@ Date: DateTime<Utc> = {
         .map_err(|error| ParseError::User {
             error: FError {
                 location: l,
-                data: d.into(),
                 kind: error.into(),
             }
         })
-- 
cgit v1.2.3