Skip to content
This repository was archived by the owner on Oct 28, 2021. It is now read-only.

Commit 9c99c5b

Browse files
authored
Merge pull request #5061 from lrettig/fix-jsontrace-with-vmtrace
Fix jsontrace conflict with vmtrace
2 parents 3884181 + b115d40 commit 9c99c5b

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

libethereum/State.cpp

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -611,7 +611,13 @@ std::pair<ExecutionResult, TransactionReceipt> State::execute(EnvInfo const& _en
611611

612612
auto onOp = _onOp;
613613
#if ETH_VMTRACE
614-
onOp = e.simpleTrace(); // override tracer
614+
// Run the existing onOp function and the tracer
615+
onOp = [&_onOp, &e](uint64_t _steps, uint64_t PC, Instruction inst, bigint
616+
newMemSize, bigint gasCost, bigint gas, VMFace const* _vm,
617+
ExtVMFace const* voidExt) {
618+
_onOp(_steps, PC, inst, newMemSize, gasCost, gas, _vm, voidExt);
619+
e.simpleTrace();
620+
};
615621
#endif
616622

617623
u256 const startGasUsed = _envInfo.gasUsed();

0 commit comments

Comments
 (0)