Coding style fix (grep ' =[^ =]' *.h *.cc).
re-apply r33185

    Consolidate connect-scan status and errno checking

    Previously, the return value and socket errors from the connect() call
    were checked in two different places: immediately after the call, and
    then later as the sockets were select()ed over. This led to a divergence
    of logic, so the immediate checking failed to set state reasons or
    contribute to timing. This commit puts all such checking into a single
    function, handleConnectResult, so further improvements will not diverge
    again.
Fix assertion failure in do_one_select_round

Since the refactor in r33185, getting a good response for a host during
a ping scan can remove all outstanding probes for that host, since we
already know the host is up. This broke the existing iteration in some
cases. I could force the crash with: nmap -sn -PS80-89 scanme.nmap.org

Now we check explicitly for an empty list each time through the loop.