⚡ Save sload in Owned.sol

[z0r0z]

Since setOwner() is onlyOwner restricted, and there is already a check that msg.sender is owner, we can just pass msg.sender into change event rather than loading owner again.

[hensha256]

Please can you run yarn snapshots and push the resultant gas snapshots

[z0r0z]

Please can you run yarn snapshots and push the resultant gas snapshots

Just ran. Not sure how to capture forge snapshot difference in this codebase.

[hensha256]

Just ran. Not sure how to capture forge snapshot difference in this codebase.

We can't yet sadly - we had some issues with CI so had to remove it temporarily. Hoping to bring it back soon

[hensha256]

Looks great! Feel free to merge when youre ready :)

[z0r0z]

great @hensha256 I think we're gtg but looks like might need approval bump cuz ocd merged main branch into mine 😅

[hensha256]

Hey @z0r0z ! Really sorry I didnt notice this earlier - we require commits to be signed on this repo for security reasons. Please can you sign your commits 🙏

[hensha256]

Its your snapshots commit thats not signed

[hensha256]

Hi! Still waiting on:

• Signing your old commit that wasnt signed - this will mean pushing new commits to the branch
• yarn snapshots needs to be run again - when you merged in main they became outdated

[snreynolds]

Hey happy to merge these changes. Can you update the branch?

[marktoda]

replacing with https://github.com/Uniswap/v4-core/pull/440