e2689792b0
There's no need for macro `toSymbolName` to convert fork enum values to their presentation texts (logging etc) then re-parse them back to a fork enum value. `asFork` is already used in the same function and works without these steps, so use it consistently. Same applies to `op.toSymbolName` and `asOp`. This makes the code simpler, and removes a text pattern-matching requirement. The patch has been checked to confirm it doesn't change the compiled code. Motivation: The forks list will be removed from VM because it is used outside the VM as well. Doing so highlighted vm2's `toSymbolName`. It's not needed, and it's best if the VM doesn't constrain text strings used outside the VM Signed-off-by: Jamie Lokier <jamie@shareable.org> |
||
|---|---|---|
| .. | ||
| op_handlers | ||
| utils | ||
| forks_list.md | ||
| forks_list.nim | ||
| forks_list.png | ||
| gas_costs.nim | ||
| gas_meter.nim | ||
| op_codes.nim | ||
| op_dispatcher.nim | ||
| op_handlers.nim | ||
