Skip to content

feat: Gas optimizations, style fixes, minor fixes#150

Open
QEDK wants to merge 2 commits intodevelopfrom
gas-opts
Open

feat: Gas optimizations, style fixes, minor fixes#150
QEDK wants to merge 2 commits intodevelopfrom
gas-opts

Conversation

@QEDK
Copy link
Copy Markdown
Member

@QEDK QEDK commented Feb 27, 2022

  • Allows hardhat to work without local variables by fixing the RPC config
  • Removes zero checks that consume gas
    • It is cheaper to do 0 ETH transfers than redo a reverted tx
    • Makes it more expensive for correct txs
  • Optimize uint256 total logic for ETH, we can do it in-loop
  • Conform event names to Solidity style guide (CapWords)
  • Bump compiler to latest Solidity and more opt runs
  • No change in hardhat_contracts.json

@QEDK QEDK added the enhancement New feature or request label Feb 27, 2022
@QEDK QEDK requested a review from hansmrtn February 27, 2022 13:24
@QEDK QEDK self-assigned this Feb 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant