Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Contract Creation #26

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rodrigoherrerai
Copy link

Hi, just a small suggestion here, the "zero address" is super confusing because it seems that it means the actual 20 bytes zero address "address(0)" in solidity. As you know, a contract creation does not occur when the "to" address is set to the actual address(0), rather it gets executed when the "to" value is set to null, left empty or the "0x0". I know that the intention was to mean the "0x0" value but it is still confusing and a lot of new developers mistakenly think that creating a contract is by sending a transaction to the 0 address.

I think it is important to be more precise with this subject.

From the solidity docs:
"If the target account is not set (the transaction does not have a recipient or the recipient is set to null), the transaction creates a new contract."
link: https://docs.soliditylang.org/en/v0.8.19/introduction-to-smart-contracts.html#index-8

Hi, I think the term zero address can be extremely confusing as sending a transaction to the actual zero address ("address(0)" in Solidity) does not create a new contract (the zero address is a normal address from the eyes of the evm). 

From the solidity docs: 

"If the target account is not set (the transaction does not have a recipient or the recipient is set to null), the transaction creates a new contract."
Link: https://docs.soliditylang.org/en/v0.8.19/introduction-to-smart-contracts.html#index-8

I think this makes it clearer for new developers as a lot of developers in the ecosystem mistakenly think that creating a contract is by sending the bytecode to the address(0).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant