Rewrite some bitboard init code

And move the static function Position::attacks_from() to
bitboard code renaming it attacks_bb()

No functional change.
Rename Bitboards print to pretty

To align to same named Position function and
avoid using std::cout directly.

Also remove some stale <iostream> include while
there.

No functional change.
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.
Assorted spelling fixes

No functional change.
Big assorted spelling fixes

No functional change.
Another round of spelling fixes

And also renamed a loop variable while there.

No functional change.
Re-fix a comment

No functional change.