[ripple/rippled] Add a ‘domain’ field to validator lists: (#2619)

nbougalis commented on this pull request.

> {
— ++iNew;
— ++iOld;
+ if (it->second <= 0) + it = keyListings_.erase(it); + else + ++it; I share your concern generally, but I don't think this code is invoked frequently enough to really matter. And the new code is, as you say, much more readable and (imo) less error prone. I had to read the existing code a couple of times to _really_ convince myself it does what it claims to.

Добавить комментарий