Simplify magic number with type constant (#876)

SunnyWar · web-flow · commit 13f7eeaef57c · 2026-04-12T01:07:25.000+03:00
No functional change.

Bench: 2798043
diff --git a/src/tools/perft.rs b/src/tools/perft.rs
@@ -108,12 +108,12 @@ fn perft_internal<F: Fn(&Board) -> MoveList>(move_gen: &F, depth: usize, board:
 
 fn is_legal_movegen(board: &Board) -> MoveList {
     let mut moves = MoveList::new();
-    for i in 0..0x10000 {
+    for i in 0..=u16::MAX {
         let j = i >> 12;
         if j == 0b0011 || j == 0b0110 || j == 0b0111 {
             continue;
         }
-        let mv: Move = unsafe { std::mem::transmute(i as u16) };
+        let mv: Move = unsafe { std::mem::transmute(i) };
         if mv.is_present() && board.is_legal(mv) {
             moves.push(mv.from(), mv.to(), mv.kind());
         }

Original file line number	Diff line number	Diff line change
`@@ -108,12 +108,12 @@ fn perft_internal<F: Fn(&Board) -> MoveList>(move_gen: &F, depth: usize, board:`
`108`	`108`
`109`	`109`	`fn is_legal_movegen(board: &Board) -> MoveList {`
`110`	`110`	`let mut moves = MoveList::new();`
`111`		`- for i in 0..0x10000 {`
	`111`	`+ for i in 0..=u16::MAX {`
`112`	`112`	`let j = i >> 12;`
`113`	`113`	`if j == 0b0011 \|\| j == 0b0110 \|\| j == 0b0111 {`
`114`	`114`	`continue;`
`115`	`115`	`}`
`116`		`- let mv: Move = unsafe { std::mem::transmute(i as u16) };`
	`116`	`+ let mv: Move = unsafe { std::mem::transmute(i) };`
`117`	`117`	`if mv.is_present() && board.is_legal(mv) {`
`118`	`118`	`moves.push(mv.from(), mv.to(), mv.kind());`
`119`	`119`	`}`