reap the benefits of the simpler API, and remove uneccessary const

lgtm, just a few suggestions for conciseness

address comments: add a QUORUM_NODE_COUNT=16 to test, return veector of peers instead of passing a reference

@bot build-tsan

@bot build-tsan

specialize Misbehaving messages: unsolicited-avahello, unsolicited-avapoll and unsolicited-avaresponse

Also update .arclint accordingly

minor optimization: don't do and additional check for each message

fix functional test by sorting the list of node ids before comparing with range(1, 17)

I think this should be abandoned in favor of D9298

change icon to something a bit more readable

It is not very readable. But I created the diff anyways to start discussing solutions. I know nothing about HTML.

apparently the additional NodeSet index is not necessary. You can do a partial search on only the first key in a composite_key.