extend peer.Set with the Remove method

[vgonkivs]

Currently peer.Set allows only to add peers but it would be nice to have a remove functionality.

[marten-seemann]

Actually, I'd be in favor of removing the peer.Set altogether. The only consumer I can find is kad-dht (here and here) and the peerstore, where the peerstore isn't even using it in an efficient way.

Therefore, this code should probably live (unexported) inside kad.

[vgonkivs]

Hi @marten-seemann. Thanks for the explanation. Is it possible to temporary add this functionality(I can open a PR) as it's pretty logic that if there is a possibility to add a peer, then it might have possibility to delete a peer or could suggest what can I use instead of peer.Set(with add/remove peer functionality) if it will be deleted at all? I want to avoid copy pasting in my code

[marten-seemann]

This seems to be a trivial piece of code, so I'd argue that copy-pasting the parts that you need would actually be the right thing to do.

Also note that the API here is kind of confusing: You can set a size, but that size is not enforced by Add (only by TryAdd).

[vgonkivs]

ok, thank you.