dapphub / dapphub/ds-auth

Use Local Memory Type Variable Instead of Global Storage Type Variable in Event to Save Gas

Open
#15 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Solidity
Stars
139
Forks
36
PR merge metrics
No merged PRs in 30d

Description

Hi, we recently have conducted a systematic study about Solidity event usage, evolution, and impact, and we are attempting to build a tool to improve the practice of Solidity event use based on our findings. We have tried our prototype tool on some of the most popular GitHub Solidity repositories, and for your repository, we find a potential optimization of gas consumption arisen from event use.

The point is that when we use emit operation to store the value of a certain variable, local memory type variable would be preferable to global storage type (state) variable if they hold the same value. The reason is that an extra SLOAD operation would be needed to access the variable if it is storage type, and the SLOAD operation costs 800 gas.

For your repository, we find that the following event use can be improved:

- [auth.sol](https://github.com/dapphub/ds-auth/blob/master/src/auth.sol)
**function name:** *setOwner*
**event name:**  *LogSetOwner*
**variable:**    *owner->owner_*
```Solidity
function setOwner(address owner_)
public
auth
{
owner = owner_;
emit LogSetOwner(owner);
}
```

Do you find our results useful? Your reply and invaluable suggestions would be greatly appreciated, and are vital for improving our tool. Thanks a lot for your time!

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with src/auth.sol, focusing on setOwner and the LogSetOwner emission identified in the issue. Check whether the event can use the already assigned value without another state read; done means the optimization is applied while preserving the event's value and existing behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
solidity
Domain
blockchain
Issue type
Refactor
Difficulty
1/5
Estimated time
Under an hour
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.