[PVS] An unconditional 'return' within a loop.

[tico-tico]

PVS-Studio found this weird loop in next.cpp

int cEngine::NextCapture(MOVES *m) {

    int move;

    while (m->next < m->last) {
        move = SelectBest(m);
        return move;
    }
    return 0;
}

I don't know if this a typo (while instead of if) or some logical error.

[nescitus]

Long time ago there was some filtering of bad captures done within that loop; I moved it to Quiesce() gaininig some speed, but forgot to make te loop look normal.