Simplify a condition in gives_check()

Now that aligned() is quite fast we can skip
some logic.

No functional change.
Rename CASTLE to CASTLING

It is call 'castling move', not 'castle move'

Noticed while reading DiscoCheck sources.

No functional change.
Merge remote-tracking branch 'upstream/master'