feat[codegen]!: check returndatasize even when skip_contract_check is set

[charles-cooper]

What I did

How I did it

How to verify it

Commit message

previously, when `skip_contract_check=True` was set, the ABI decoding
routine could result in garbage memory in the case that no contract is
at the target address, based on the logic that, if we don't care about
whether a contract exists at the target, we can skip the returndatasize
check. however, it makes sense for the ABI decoder to in any case
include the returndatasize check, to prevent the garbage data case as
previously mentioned in the docs.

this is a breaking change, since previously calls with
`skip_contract_set=True` could result in garbage memory - and now they
would revert instead.

this represents a slight performance regression in that case, but it is
easier to reason about and also ensures the user always decodes valid
data.

Description for the changelog

Cute Animal Picture