Skip to content

fix: remove wrong Clone Trait in WrapDatabaseRef#2181

Merged
rakita merged 1 commit intobluealloy:mainfrom
jackwener:remove_copy
Mar 11, 2025
Merged

fix: remove wrong Clone Trait in WrapDatabaseRef#2181
rakita merged 1 commit intobluealloy:mainfrom
jackwener:remove_copy

Conversation

@jackwener
Copy link
Contributor

Removed the Copy traits from the WrapDatabaseRef struct's derive attributes.

WrapDatabaseRef shouldn't own Copy traits.

@jackwener jackwener changed the title refactor: remove wrong Clone Macro in WrapDatabaseRef fix: remove wrong Clone Macro in WrapDatabaseRef Mar 11, 2025
@jackwener jackwener changed the title fix: remove wrong Clone Macro in WrapDatabaseRef fix: remove wrong Clone Trait in WrapDatabaseRef Mar 11, 2025
@codspeed-hq
Copy link

codspeed-hq bot commented Mar 11, 2025

CodSpeed Performance Report

Merging #2181 will improve performances by 3.11%

Comparing jackwener:remove_copy (da046dd) with main (9011820)

Summary

⚡ 1 improvements
✅ 7 untouched benchmarks

Benchmarks breakdown

Benchmark BASE HEAD Change
precompile bench | ecrecover precompile 203.4 µs 197.3 µs +3.11%

Copy link
Member

@rakita rakita left a comment

Choose a reason for hiding this comment

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

true

@rakita rakita merged commit a21d64e into bluealloy:main Mar 11, 2025
29 checks passed
This was referenced Mar 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants