argotorg / argotorg/solidity

Check for reuse of old error/warning codes

Open
#13,545 0 comments 0 reactions 0 assignees View on GitHub
low effort medium impact must have eventually needs design testing :hammer:
Dominant language
C++
Stars
25.7k
Forks
6.2k
Avg merge
2d 19h
Merged PRs (30d)
29

Description

Some error codes have been removed and are no longer emitted (for example, 8760 for shadowing of opcodes). There is currently a risk that those codes are reintroduced with a different meaning.

[`scripts/error_codes.py`](https://github.com/ethereum/solidity/blob/db571add54bb55f008bff1b51f3dafad7112b8d6/scripts/error_codes.py) should be amended to include a list of removed error codes and verify that they are not in use in the codebase.

Contributor guide

Open the contributing guide

Research direction

Start with scripts/error_codes.py and inspect how error codes such as 8760 are defined and referenced across the codebase. Add the removed-code list and a check for reuse with a different meaning. Done means the check covers the recorded removed codes and finds no active reuse.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp, python
Domain
compilers
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.