Skip to content

Commit

Permalink
Fix typo in tbprobe.cpp
Browse files Browse the repository at this point in the history
closes #4959

No functional change
  • Loading branch information
RainRat authored and Disservin committed Jan 4, 2024
1 parent cafbe8e commit 4930892
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/syzygy/tbprobe.cpp
Expand Up @@ -863,7 +863,7 @@ do_probe_table(const Position& pos, T* entry, WDLScore wdl, ProbeState* result)
int adjust2 = (squares[2] > squares[0]) + (squares[2] > squares[1]);

// First piece is below a1-h8 diagonal. MapA1D1D4[] maps the b1-d1-d3
// triangle to 0...5. There are 63 squares for second piece and and 62
// triangle to 0...5. There are 63 squares for second piece and 62

This comment has been minimized.

Copy link
@PedanticHacker

PedanticHacker Jan 4, 2024

Contributor

The way a range is written is by using two dots, so 0..5.

// (mapped to 0...61) for the third.

This comment has been minimized.

Copy link
@PedanticHacker

PedanticHacker Jan 4, 2024

Contributor

Same here, the range should be written as 0..61.

if (off_A1H8(squares[0]))
idx = (MapA1D1D4[squares[0]] * 63 + (squares[1] - adjust1)) * 62 + squares[2] - adjust2;
Expand Down

3 comments on commit 4930892

@Disservin
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@PedanticHacker these werent part of the pr, if you got a problem with those open a new pr which fixes these.

@PedanticHacker
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will do.

@PedanticHacker
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here it is: #4965

Please sign in to comment.