search

ethereum / execution-specs

Specification for the Execution Layer. Tracking network upgrades.
Creative Commons Zero v1.0 Universal
869 stars 240 forks source link

The docstring of func `process_message` in `vm/interpreter.py` is wrong #992

Open Alleysira opened 3 months ago

Alleysira commented 3 months ago

Metadata

What was wrong?

Hello, developers! This is just a Inappropriate expression in docstring.

Take the func process_message() in vm/interpreter.py in Cancun as an example: https://github.com/ethereum/execution-specs/blob/fcd12750edd4443a91f138728689a1d0a503a7c1/src/ethereum/cancun/vm/interpreter.py#L204-L240

I think here process_message() is just a smart contract call that executes the code of message, instead of what is described in the func's docstring:

Executes a call to create a smart contract.

I checked all vm/interpreter.py from Frontier to Cancun, it seems that all forks have this issue. Perhaps a slight adjustment to the description could help clarify this.

Thanks for you attention!