Mutations to dynamic field objects should not show up in effects if execution failed

[lxfind]

trafficstars

This is needed to bound the effects size in all cases.

[lxfind]

@gdanezis in case I missed anything

[tnowacki]

I was mistaken, this should not happen.

Move VM errors, or other execution errors, propagate via Rust Results. So any dynamic field information will be lost, as we want.

[stefan-mysten]

@lxfind is this still relevant? If not, let's close it. Thanks.

[lxfind]

Thanks. We can close this.