Skip to content
This repository has been archived by the owner on May 24, 2022. It is now read-only.

Rename reserved #5

Open
kirushik opened this issue Jun 5, 2018 · 0 comments
Open

Rename reserved #5

kirushik opened this issue Jun 5, 2018 · 0 comments

Comments

@kirushik
Copy link

kirushik commented Jun 5, 2018

We're shadowing the Solidity built-in keyword reserved in https://github.com/parity-contracts/name-registry/blob/12282e0b20f9e6d611237a59014f33c317c9a648/contracts/SimpleRegistry.sol#L283

We should rename that function, possibly into isReseved.

Repercussions of such renaming in production (procy contracts, ABI usage) need to be taken in account.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant