Skip to content

Update EIP-7708: Clarify selfdestruct event#11127

Merged
eth-bot merged 5 commits into
ethereum:masterfrom
etan-status:7708-sendall
Jan 21, 2026
Merged

Update EIP-7708: Clarify selfdestruct event#11127
eth-bot merged 5 commits into
ethereum:masterfrom
etan-status:7708-sendall

Conversation

@etan-status
Copy link
Copy Markdown
Contributor

No description provided.

@etan-status etan-status requested a review from eth-bot as a code owner January 21, 2026 09:05
@etan-status etan-status marked this pull request as draft January 21, 2026 09:05
@github-actions github-actions Bot added c-update Modifies an existing proposal s-draft This EIP is a Draft t-core labels Jan 21, 2026
@eth-bot
Copy link
Copy Markdown
Collaborator

eth-bot commented Jan 21, 2026

✅ All reviewers have approved.

Comment thread EIPS/eip-7708.md Outdated
| `topics[1]` | Closed `contract_address` (zero prefixed to fill uint256) |
| `data` | `amount` in Wei (big endian uint256)

The Selfdestruct log is emitted when (1) a `SELFDESTRUCT` to self is triggered, or when (2) a smart contract receives unspent ETH after it is already flagged for `SELFDESTRUCT`.
Copy link
Copy Markdown
Contributor

@jwasinger jwasinger Jan 21, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For 2, a balance transfer is emitted at the time the contract receives balance post-selfdestruct and another selfdestruct log is emitted when the account is removed at finalization.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Corrected my above comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense, yeah! Revised wording, sounds good?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good.

Comment thread EIPS/eip-7708.md Outdated
@jwasinger
Copy link
Copy Markdown
Contributor

SGTM

@etan-status etan-status marked this pull request as ready for review January 21, 2026 09:38
@etan-status
Copy link
Copy Markdown
Contributor Author

@eth-bot rerun

@eth-bot eth-bot enabled auto-merge (squash) January 21, 2026 09:39
Copy link
Copy Markdown
Collaborator

@eth-bot eth-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All Reviewers Have Approved; Performing Automatic Merge...

@eth-bot eth-bot merged commit b6ddf5a into ethereum:master Jan 21, 2026
17 checks passed
@etan-status etan-status deleted the 7708-sendall branch January 21, 2026 11:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c-update Modifies an existing proposal s-draft This EIP is a Draft t-core

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants