Skip to content

Conversation

@pirapira
Copy link
Member

Ξ's output was declared to be (σ', g', A, I) and especially the second element g' was a gas amount while Ξ was defined to return a machine state μ' as the second element.

This PR changes the definition of Ξ so that it returns a gas amount as the second element.

This PR is a reincarnation of #240.

@pirapira pirapira requested review from gavofyork and removed request for gavofyork March 29, 2017 15:09
Copy link
Contributor

@gavofyork gavofyork left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

suggestion for clarity

Paper.tex Outdated
\begin{eqnarray}
\Xi(\boldsymbol{\sigma}, g, I) & \equiv & X_{0,1,2,4}\big((\boldsymbol{\sigma}, \boldsymbol{\mu}, A^0, I)\big) \\
\Xi(\boldsymbol{\sigma}, g, I) & \equiv & (\boldsymbol{\sigma}'\!, \boldsymbol{\mu}'_g, A, \mathbf{o}) \\
(\boldsymbol{\sigma}, \boldsymbol{\mu}'\!, A, I'\!, \mathbf{o}) & \equiv & X\big((\boldsymbol{\sigma}, \boldsymbol{\mu}, A^0\!, I)\big) \\
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

consider I'\! -> ... to make clear that the value isn't being used.

@pirapira pirapira force-pushed the take_the_gas_of_machine_state branch from 7bece16 to 56475f1 Compare March 30, 2017 10:55
@gavofyork gavofyork merged commit 6878240 into ethereum:master Mar 30, 2017
@pirapira pirapira deleted the take_the_gas_of_machine_state branch May 10, 2017 13:09
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.

2 participants