Fix ExecutionState::reset()

[chfast]

Reset previously omitted fields related to EOF execution: deploy_container and call_stack.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.89%. Comparing base (00af48c) to head (88129ee).
Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #1004   +/-   ##
=======================================
  Coverage   93.89%   93.89%           
=======================================
  Files         146      146           
  Lines       15485    15487    +2     
=======================================
+ Hits        14539    14541    +2     
  Misses        946      946           

Flag	Coverage Δ
eof_execution_spec_tests	17.49% <0.00%> (-0.01%)	⬇️
ethereum_tests	27.76% <0.00%> (-0.01%)	⬇️
ethereum_tests_silkpre	19.45% <0.00%> (-0.01%)	⬇️
execution_spec_tests	19.02% <0.00%> (-0.01%)	⬇️
unittests	88.98% <100.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
lib/evmone/execution_state.hpp	94.82% <100.00%> (+0.18%)	⬆️

[pdobacz]

maybe rearrange the reset statements to reflect order of declarations (also move call_stack up?).

[chfast]

maybe rearrange the reset statements to reflect order of declarations (also move call_stack up?).

oh, I thought they are in order. I'll check.

[chfast]

The reset handles the fields in the order the fields are declared in the class. This is as if it is some kind of constructor.
BTW, I wonder if this is some kind of design pattern.