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

doc: remove "prior to Homestead" descriptions #1214

Closed
wants to merge 1 commit into from
Closed

Conversation

pirapira
Copy link
Member

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.

@axic
Copy link
Member

axic commented Oct 13, 2016

If we get #1117 implemented, this may become useful?

@pirapira
Copy link
Member Author

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.

@chriseth
Copy link
Contributor

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).
Copy link
Member

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.
Copy link
Member

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?

Copy link
Member Author

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.

@pirapira
Copy link
Member Author

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.

@pirapira pirapira closed this Oct 14, 2016
@axic axic deleted the doc-homestead branch October 20, 2016 12:34
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.

3 participants