From 5e2c5b6d41788d16ed7058acc9f48ac0a4cb1ae6 Mon Sep 17 00:00:00 2001
From: Stephen Seo <seo.disparate@gmail.com>
Date: Mon, 7 Mar 2022 13:14:57 +0900
Subject: [PATCH] Minor refactoring

Use "From<>" instead of "Into<>" for MainMenuMessage to GameState.
---
 front_end/src/state.rs          | 11 +++++++++++
 front_end/src/yew_components.rs | 11 -----------
 2 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/front_end/src/state.rs b/front_end/src/state.rs
index 6bb693b..8b8e2cc 100644
--- a/front_end/src/state.rs
+++ b/front_end/src/state.rs
@@ -1,3 +1,4 @@
+use crate::yew_components::MainMenuMessage;
 use std::cell::Cell;
 use std::fmt::Display;
 use std::rc::Rc;
@@ -17,6 +18,16 @@ impl Default for GameState {
     }
 }
 
+impl From<MainMenuMessage> for GameState {
+    fn from(msg: MainMenuMessage) -> Self {
+        match msg {
+            MainMenuMessage::SinglePlayer => GameState::SinglePlayer,
+            MainMenuMessage::LocalMultiplayer => GameState::LocalMultiplayer,
+            MainMenuMessage::NetworkedMultiplayer => GameState::NetworkedMultiplayer,
+        }
+    }
+}
+
 #[derive(Copy, Clone, Debug, PartialEq, Eq)]
 pub enum BoardState {
     Empty,
diff --git a/front_end/src/yew_components.rs b/front_end/src/yew_components.rs
index 8f92889..56f6e84 100644
--- a/front_end/src/yew_components.rs
+++ b/front_end/src/yew_components.rs
@@ -12,17 +12,6 @@ pub enum MainMenuMessage {
     NetworkedMultiplayer,
 }
 
-#[allow(clippy::from_over_into)]
-impl Into<GameState> for MainMenuMessage {
-    fn into(self) -> GameState {
-        match self {
-            MainMenuMessage::SinglePlayer => GameState::SinglePlayer,
-            MainMenuMessage::LocalMultiplayer => GameState::LocalMultiplayer,
-            MainMenuMessage::NetworkedMultiplayer => GameState::NetworkedMultiplayer,
-        }
-    }
-}
-
 impl Component for MainMenu {
     type Message = MainMenuMessage;
     type Properties = ();