Avoid a tricky line in shelter_storm()

kf = (kf == FILE_A) ? kf++ : ....

is tricky becuase kf is updated twice and it happens
to do the right thing just by accident.

Rewrite in a better way.

Spotted by pdimov

No functional change.
This commit is contained in:
Marco Costalba 2013-02-23 19:27:32 +01:00
parent ccf21f5595
commit dbd28bc7f8

View file

@ -231,7 +231,7 @@ Value Entry::shelter_storm(const Position& pos, Square ksq) {
Rank rkUs, rkThem;
File kf = file_of(ksq);
kf = (kf == FILE_A) ? kf++ : (kf == FILE_H) ? kf-- : kf;
kf = (kf == FILE_A) ? FILE_B : (kf == FILE_H) ? FILE_G : kf;
for (int f = kf - 1; f <= kf + 1; f++)
{