Rename execution_result -> ExecutionResult

[chfast]

No description provided.

[axic]

Nice, I was about to ask why not rename it.

[chfast]

Move it to separate header? I will use it also in WasmEngine.

[codecov]

Codecov Report

Merging #428 into master will not change coverage.
The diff coverage is 95.45%.

@@           Coverage Diff           @@
##           master     #428   +/-   ##
=======================================
  Coverage   99.35%   99.35%           
=======================================
  Files          49       49           
  Lines       13242    13242           
=======================================
  Hits        13156    13156           
  Misses         86       86           

[axic]

Move it to separate header? I will use it also in WasmEngine.

I think that is a bad idea -- we would be trying to apply Fizzy's API to other engines. Why can't we leave WasmEngine as is, already uses optional?

Probably I could / will change my mind if we would expose host functions in WasmEngine.

[chfast]

I think that is a bad idea -- we would be trying to apply Fizzy's API to other engines. Why can't we leave WasmEngine as is, already uses optional?

Sounds good.