As per title.
- Group Reviewers
I'm torn. I think there's an argument for keeping it in consensus. The argument for moving it to defaults is because it can be overridden by miners, but tweaking this value can lead to chain instability if every miner has a different max set (allowing others to exploit this boundary, etc).
Otherwise, the diff looks fine to me. Getting many of these defaults out of validation.h to somewhere better described makes sense.
Thought about it some more. I don't think this fixes the root cause of the issue: validation.h/cpp is full of junk that shouldn't be there. Moving these defaults out looks like on the surface, but would be a non-issue if code was properly segmented into appropriate headers. That way, defaults would live where they are used.