This makes the code cleaner by allowing the Misbehaving() overload to be used outside of net_processing.cpp.
Depends on D4150
Build Bitcoin-ABC / Diffs / Diff Testing started.
Build Bitcoin-ABC / Diffs / Diff Testing passed.
This seems to me to be gratuitously moving away from core for no good reason. What's the reason?
I'm not sure what this combination of qualifier does.
I think you're right.