clarify non-error return statements

Doing `return e, err` when err was known to be nil made it quite confusing to read through `roundrobin.NextEndpoint`. Now it returns nil explicitly, which is clearer.