-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
doc: remove "prior to Homestead" descriptions #1214
Conversation
If we get #1117 implemented, this may become useful? |
After #1117, in my opinion, EVM version differences should be explained in a special section about the differences. The other parts of the documentation can always talk about the "current" one. |
I think we should still leave that in, people might ask about old rules and opcodes. |
@@ -976,8 +975,7 @@ are all compiled as calls (``DELEGATECALL``) to an external | |||
contract/library. If you use libraries, take care that an | |||
actual external function call is performed. | |||
``msg.sender``, ``msg.value`` and ``this`` will retain their values | |||
in this call, though (prior to Homestead, ``msg.sender`` and | |||
``msg.value`` changed, though). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually I think we should extend this with an explanation: "because of the use of CALLCODE
and not DELEGATECALL
"
@@ -1079,7 +1077,7 @@ In both situations, all functions, even those where the | |||
type of the first parameter does not match the type of | |||
the object, are attached. The type is checked at the | |||
point the function is called and function overload | |||
resolution is performed. | |||
resolution is performed at the compilation time. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this related to this PR?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, but I didn't even split the commits. hmm.
78399bc
to
9f3a133
Compare
I thought these phrases are leftovers from the era when people remembered things before Homestead, but from the comments here, I guess that's not the case. My main concern was about undefined names like "Homestead" appearing in the middle of a text. |
In the documentation there were two sidenotes about "prior to Homestead" era. I think these notes are no longer necessary.
The other change is just a clarification about when the function overload resolution is performed.