Docs: Specify blobhash() behaviour when blob does not exist
#15210
Conversation
|
Thank you for your contribution to the Solidity compiler! A team member will follow up shortly. If you haven't read our contributing guidelines and our review checklist before, please do it now, this makes the reviewing process and accepting your contribution smoother. If you have any questions or need our help, feel free to post them in the PR or talk to us directly on the #solidity-dev channel on Matrix. |
|
Thanks @ToonVanHove, I'm already approving, but please remove the trailing whitespace as reported here: https://app.circleci.com/pipelines/github/ethereum/solidity/34603/workflows/0f99fcfa-1229-432c-9a19-d59a9ac4b4bb/jobs/1568548 |
|
Please, also rebase and squash the commits ;) |
|
Sorry about that, should be fixed! |
| @@ -77,6 +77,7 @@ Block and Transaction Properties | |||
| - ``blobhash(uint index) returns (bytes32)``: versioned hash of the ``index``-th blob associated with the current transaction. | |||
| A versioned hash consists of a single byte representing the version (currently ``0x01``), followed by the last 31 bytes | |||
| of the SHA256 hash of the KZG commitment (`EIP-4844 <https://eips.ethereum.org/EIPS/eip-4844>`_). | |||
| Returns zero if no blob with the given index exists. | |||
There was a problem hiding this comment.
We should probably include that info in the opcode table as well: https://docs.soliditylang.org/en/develop/yul.html#evm-dialect
The builtin also listed in the cheatsheet: https://docs.soliditylang.org/en/develop/cheatsheet.html#block-and-transaction-properties. Not sure if we should add it there or not though. Cheatsheet is meant to be concise. On the other hand the description is the same as here so it would make sense to have it in both places as well.
|
I've added it to both the opcode table and the cheatsheet. Let me know if you'd like any further changes! |
Add description of
blobhash()behaviour when the given blob does not exist.This is currently not specified.