in D5967 ([[https://github.com/bitcoin/bitcoin/pull/15638/commits/4a75c9d6512a5580e60104103ea11d2cd9586354 | core commit]]) I've included policy/settings.h in policy/policy.h
so that our version of GetVirtualTransactionSize still had access to
`::nBytesPerSigOp`
recently while working on commit [[https://github.com/bitcoin/bitcoin/pull/15639/commits/b874747b51882a613895a100c4210c7f1dddde30 | b874747b51882a613895a100c4210c7f1dddde30]] of [[https://github.com/bitcoin/bitcoin/pull/15639 | PR15639]],
I realize, in hindsight, that looks wrong
this diff reverts the include I added to policy.h, moves our version of
GetVirtualTransactionSize to settings.h ahead of [[https://github.com/bitcoin/bitcoin/pull/15639 | PR15639]] and makes
the other necessary changes