Accurate pv

A patch series that rearranges accurate pv to align to current coding styles and also simplifies the code, for instance we remove entirely PVEntry. 

I have split the code in 3 commits for an easier review.

It is a non functional change patch series.

Single line comments have no dot in SF

On Sat, Nov 15, 2014 at 7:37 AM, Arjun Temurnikar <notifications@github.com>
wrote:

> In src/search.cpp:
>
> > @@ -466,11 +466,8 @@ namespace {
> >      ss->ttMove = ttMove = RootNode ? RootMoves[PVIdx].pv[0] : tte ? tte->move() : MOVE_NONE;
> >      ttValue = tte ? value_from_tt(tte->value(), ss->ply) : VALUE_NONE;
> >
> > -    // At PV nodes we check for exact scores, whilst at non-PV nodes we check for
> > -    // a fail high/low. The biggest advantage to probing at PV nodes is to have a
> > -    // smooth experience in analysis mode. We don't probe at Root nodes otherwise
> > -    // we should also update RootMoveList to avoid bogus output.
> > -    if (   !PvNode
> > +    // At non-PV nodes we check for a fail high/low. We don't probe at PV nodes
>
> Minor grammar: Add full stop.
>
> We don't probe at PV nodes.
>
> :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/official-stockfish/Stockfish/pull/116/files#r20400584>
> .
>