Report minimum ping time in getpeerinfo

[TheBlueMatt]

No description provided.

[jonasschnelli]

utACK

[paveljanik]

ACK

[fanquake]

utACK

On Friday, September 4, 2015, paveljanik notifications@github.com wrote:

ACK

—
Reply to this email directly or view it on GitHub
#6633 (comment).

[maflcko]

minping seems to be zero in most cases. Being connected to 3 "incoming peers" on localhost and 8 "outgoing peers" on testnet I get

$ getpeerinfo|grep minping
    "minping": 0,
    "minping": 0,
    "minping": 0,
    "minping": 0.107754,
    "minping": 0,
    "minping": 0,
    "minping": 0,
    "minping": 0,
    "minping": 0.219713,
    "minping": 9.6e-05,
    "minping": 0,

[laanwj]

@MarcoFalke Good catch: nMinPingUsecTime is left uninitialized on CNode creation. And as it is a moving minimum, it should probably default to std::numeric_limits<int64_t>::max() initially, not zero.

[laanwj]

ACK on this pull.