fix: removed unnecessary print statement & refactored Color::negation…

…_multiplier
dannyhammer · Aug 15, 2024 · 249bd82 · 249bd82
1 parent f2e4d32
commit 249bd82
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 10 deletions.
diff --git a/brogle/src/engine.rs b/brogle/src/engine.rs
@@ -601,8 +601,6 @@ impl UciEngine for Engine {
             Duration::MAX
         };
 
-        eprintln!("Received duration of {time_remaining:?}");
-
         // Clone the arcs for whether we're searching and our found results
         let is_searching = Arc::clone(&self.is_searching);
         // let timeout = Duration::from_secs_f32(time_remaining.as_secs_f32() / 20.0); // 5% time remaining

diff --git a/brogle_core/brogle_types/src/color.rs b/brogle_core/brogle_types/src/color.rs
@@ -116,16 +116,16 @@ impl Color {
         // TODO: Which of these 3 is faster?
 
         // A: Match
-        // match self {
-        //     Self::White => 1,
-        //     Self::Black => -1,
-        // }
+        match self {
+            Self::White => 1,
+            Self::Black => -1,
+        }
 
-        // B: Multiply
-        // 1 - 2 * *self as i8
+        // B: Shift
+        // 1 - ((*self as i8) << 1)
 
-        // C: Shift
-        1 - ((*self as i8) << 1)
+        // C: Bitwise
+        // (*self as i8 ^ 1) - (*self as i8 & 1)
     }
 
     /// Returns this [`Color`]'s opposite / inverse / enemy.