Skip to content

Conversation

@iovoid
Copy link
Contributor

@iovoid iovoid commented Nov 10, 2025

Motivation

This clone shows up as 1.2% of key lookup time, and can be trivially removed.

Description

Removes the clone.

@iovoid iovoid requested a review from a team as a code owner November 10, 2025 13:49
@github-actions github-actions bot added the L1 Ethereum client label Nov 10, 2025
@github-actions
Copy link

Lines of code report

Total lines added: 0
Total lines removed: 1
Total lines changed: 1

Detailed view
+-------------------------------------------+-------+------+
| File                                      | Lines | Diff |
+-------------------------------------------+-------+------+
| ethrex/crates/storage/trie_db/layering.rs | 183   | -1   |
+-------------------------------------------+-------+------+

@github-project-automation github-project-automation bot moved this to In Review in ethrex_l1 Nov 10, 2025
@jrchatruc jrchatruc added this pull request to the merge queue Nov 10, 2025
Merged via the queue into main with commit 8627e77 Nov 10, 2025
43 checks passed
@jrchatruc jrchatruc deleted the remove_layering_key_clone branch November 10, 2025 15:47
@github-project-automation github-project-automation bot moved this from In Review to Done in ethrex_l1 Nov 10, 2025
xqft pushed a commit that referenced this pull request Nov 11, 2025
**Motivation**

This clone shows up as 1.2% of key lookup time, and can be trivially
removed.

**Description**

Removes the clone.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

L1 Ethereum client

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

6 participants