Code review comment for lp:~widelands-dev/widelands/net-boost-asio

Revision history for this message
GunChleoc (gunchleoc) wrote :

Well, this should work:

    for (auto& client: clients_) {
        ...
        ConnectionId id_to_remove = client.first;
        ...
        client.second.deserializer.read_data(buffer, length);
        ...
    }

You don't need an explicit iterator to access the first and second element of the pair.

« Back to merge proposal