Retire UCI_AnalyseMode option

It has been obsoleted out already some time ago
and currently there is no point in changing eval
score according to if we are in game or analyzing.

So retire the option.

No functional change.
Simplify Bitboards::pretty

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

Conflicts:
	src/ucioption.cpp
Rename xxx_to_char() -> to_char()

No functional change.
Remove limit of minimumSplitDepth

There is no reason why an user cannot set
it at a value less than 4.

No functional change.
Pass CastlingFlag argument only

Instead of Color and CastlingSide. Change functions API
accordingly.

No functional change.
Merge remote-tracking branch 'upstream/master'
Revert dynamic draw value

When tested with weaker engines did not
performed as expected, actually it was even
a regression from standard version.

bench: 8430785
Some minor cleanup stuff

I came across while browsing the code.

No functional change.
Merge remote-tracking branch 'upstream/master'
Fix dbg_mean_of() for negative numbers

Type should be int64_t instead of uint64_t

No functional change.
Merge remote-tracking branch 'upstream/master'
Rename castling flag to castling right

This is a more conventional naming as
reported also in:

http://chessprogramming.wikispaces.com/Castling+rights

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

Explicitly write the 'while' loops. This adds some
code but makes clear what's the code behind the
macro.

No functional change.
Simplify generate<EVASIONS>

No speed regression, tested with both perft and
bench.

No functional change.
Merge remote-tracking branch 'upstream/master'
Simplify pseudo_legal()

Big simplification of pawn move check.

Code has been tested with a brute force approach: for
every position reached during a bench search, the function
has been called for each combinations of Move(from, to)
and verified the result is the same of old code.

Actually this function is very critical becuase is the
one that ensures corrupted TT moves are discarded, so
to properly test it a simple bench is not enough.

Verified also speed is not changed.

No functional chnage.
Merge remote-tracking branch 'upstream/master'
Print dbg counters after bench

Print last debug counters update just
before to exit benchmark.

Suggested by Stephane Nicolet.

No functional change.