From 231ba10f1ec3eec369d8150c266aa58a944eb2db Mon Sep 17 00:00:00 2001 From: gered Date: Wed, 5 Apr 2023 12:45:09 -0400 Subject: [PATCH] remove Copy trait from these event enums to be consistent with the rest since the TextInput event cannot implement Copy, necessitating the removal of the Copy trait from KeyboardEvent, it feels wrong to me to have the other event enums implement Copy, because it is inconsistent. so while this is less convenient potentially, it feels consistent. probably not a big deal in the future one way or the other. --- ggdt/src/system/event.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ggdt/src/system/event.rs b/ggdt/src/system/event.rs index f8da28e..257dc90 100644 --- a/ggdt/src/system/event.rs +++ b/ggdt/src/system/event.rs @@ -15,7 +15,7 @@ use crate::system::input_devices::keyboard::codes::Keycode; use crate::system::input_devices::keyboard::scancodes::Scancode; use crate::system::input_devices::mouse::buttons::{MouseButton, MouseButtons}; -#[derive(Debug, Copy, Clone, Eq, PartialEq, Hash)] +#[derive(Debug, Clone, Eq, PartialEq, Hash)] pub enum WindowEvent { Shown, Hidden, @@ -95,7 +95,7 @@ pub enum KeyboardEvent { }, } -#[derive(Debug, Copy, Clone, Eq, PartialEq, Hash)] +#[derive(Debug, Clone, Eq, PartialEq, Hash)] pub enum MouseEvent { MouseMotion { x: i32, //