[pavanky]

That's a bit harder to read isn't it? I think finding the location in first line and using distance in the ternary operator makes it more readable.

[nly]

Like this?

    auto it = std::find(begin(replicas), end(replicas), my_ip);
    return (end(replicas) == it ? -1 : std::distance(begin(replicas), it));

Yeah, i'll go with that.

[pavanky]

Yeah that's what I was thinking.