Clippy suggested improvements
authorJustin Worthe <justin@worthe-it.co.za>
Sun, 2 Sep 2018 07:29:14 +0000 (09:29 +0200)
committerJustin Worthe <justin@worthe-it.co.za>
Sun, 2 Sep 2018 07:29:14 +0000 (09:29 +0200)
src/engine/command.rs
src/engine/geometry.rs
src/input/json.rs
src/strategy/monte_carlo.rs

index b34553f..76cfaee 100644 (file)
@@ -20,7 +20,7 @@ impl fmt::Display for Command {
 }
 
 impl Command {
-    pub fn cant_build_yet(&self, energy: u16) -> bool {
+    pub fn cant_build_yet(self, energy: u16) -> bool {
         use self::Command::*;
 
         match self {
@@ -52,7 +52,7 @@ impl BuildingType {
         if id <= 4 && id != 3 { Some(unsafe { mem::transmute(id) }) } else { None }
     }
 
-    pub fn cant_build_yet(&self, energy: u16) -> bool {
+    pub fn cant_build_yet(self, energy: u16) -> bool {
         use self::BuildingType::*;
 
         let required = match self {
index b9b556a..b8b38dd 100644 (file)
@@ -32,11 +32,11 @@ impl Point {
         }
     }
 
-    pub fn x(&self) -> u8 {
+    pub fn x(self) -> u8 {
         self.index % SINGLE_MAP_WIDTH
     }
 
-    pub fn y(&self) -> u8 {
+    pub fn y(self) -> u8 {
         self.index / SINGLE_MAP_WIDTH
     }
 }
@@ -51,7 +51,7 @@ impl Point {
      * This involves mirroring the x dimension for the opponent's side
      */
 
-    pub fn to_either_bitfield(&self) -> u64 {
+    pub fn to_either_bitfield(self) -> u64 {
         1u64 << self.index
     }
 }
index 843f228..4e10f9a 100644 (file)
@@ -127,7 +127,7 @@ impl State {
                         &mut opponent
                     };
 
-                    for mut tier in bitwise_buildings.missiles.iter_mut() {
+                    for mut tier in &mut bitwise_buildings.missiles {
                         let setting = (!tier.0 & left, !tier.1 & right);
                         tier.0 |= setting.0;
                         tier.1 |= setting.1;
index 347af96..0587e31 100644 (file)
@@ -58,11 +58,11 @@ pub fn choose_move(state: &BitwiseGameState, start_time: PreciseTime, max_time:
         
         println!("NOTHING");
         println!("{}", command_scores.iter().find(|c| c.command == Command::Nothing).map(|s| s.win_ratio()).unwrap_or(0));
-        println!("");
+        println!();
 
         println!("IRON CURTAIN");
         println!("{}", command_scores.iter().find(|c| c.command == Command::IronCurtain).map(|s| s.win_ratio()).unwrap_or(0));
-        println!("");
+        println!();
     }
 
     command
@@ -83,7 +83,7 @@ fn debug_print_choices<F: FnMut(&CommandScore) -> Option<(Point, i32)>>(label: &
         }
         println!(" |");
     }
-    println!("");
+    println!();
 }
 
 #[cfg(not(feature = "discard-poor-performers"))]
@@ -174,7 +174,7 @@ fn random_move<R: Rng>(player: &Player, opponent: &Player, rng: &mut R) -> Comma
             let mut m = [Command::Nothing; NUMBER_OF_POSSIBLE_MOVES];
             m[1] = Command::IronCurtain;
             let mut i = 2;
-            for b in [BuildingType::Energy, BuildingType::Defence, BuildingType::Attack, BuildingType::Tesla].iter() {
+            for b in &[BuildingType::Energy, BuildingType::Defence, BuildingType::Attack, BuildingType::Tesla] {
                 for p in 0..NUMBER_OF_MAP_POSITIONS as u8 {
                 let point = Point::new_index(p);
                     m[i] = Command::Build(point, *b);
@@ -284,7 +284,7 @@ fn random_move<R: Rng>(player: &Player, opponent: &Player, rng: &mut R) -> Comma
     // TODO can this be a more efficient lookup?
     let index = cdf.iter().position(|&c| c > choice).expect("Random number has exceeded cumulative distribution");
 
-    MOVES[index].clone()
+    MOVES[index]
 }
 
 #[cfg(not(feature = "heuristic-random"))]