Skip to content

Commit

Permalink
Fix two typos in comments
Browse files Browse the repository at this point in the history
Note by snicolet: I use this non-functional change patch
as a pretext to correct the wrong bench number I introduced
in the message of the previous commit.

Bench: 4059356
  • Loading branch information
eduherminio authored and snicolet committed Sep 27, 2018
1 parent bbf9daa commit 8141bdd
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/endgame.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -216,7 +216,7 @@ Value Endgame<KRKP>::operator()(const Position& pos) const {
}


/// KR vs KB. This is very simple, and always returns drawish scores. The
/// KR vs KB. This is very simple, and always returns drawish scores. The
/// score is slightly bigger when the defending king is close to the edge.
template<>
Value Endgame<KRKB>::operator()(const Position& pos) const {
Expand Down
2 changes: 1 addition & 1 deletion src/search.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -905,7 +905,7 @@ namespace {
// Singular extension search (~60 Elo). If all moves but one fail low on a
// search of (alpha-s, beta-s), and just one fails high on (alpha, beta),
// then that move is singular and should be extended. To verify this we do
// a reduced search on on all the other moves but the ttMove and if the
// a reduced search on all the other moves but the ttMove and if the
// result is lower than ttValue minus a margin then we will extend the ttMove.
if ( depth >= 8 * ONE_PLY
&& move == ttMove
Expand Down

3 comments on commit 8141bdd

@IIvec
Copy link
Contributor

@IIvec IIvec commented on 8141bdd Oct 1, 2018

Choose a reason for hiding this comment

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

You are not using the same book as in the first test. Please stick to one book.

@IIvec
Copy link
Contributor

@IIvec IIvec commented on 8141bdd Oct 1, 2018

Choose a reason for hiding this comment

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

Sorry, my mistake.

@IIvec
Copy link
Contributor

@IIvec IIvec commented on 8141bdd Oct 5, 2018

Choose a reason for hiding this comment

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

So far, I approved first three tests to see a difference.

Please sign in to comment.