Skip to content
This repository has been archived by the owner on Sep 27, 2020. It is now read-only.

Fix assembly assignments parsing #46

Open
LogvinovLeon opened this issue Feb 20, 2019 · 4 comments
Open

Fix assembly assignments parsing #46

LogvinovLeon opened this issue Feb 20, 2019 · 4 comments

Comments

@LogvinovLeon
Copy link

Solidity accepts both options. This parser fails on first one and only accepts the second one.
I know it's minor nit, but would be still good to accept both.

screen shot 2019-02-20 at 2 48 14 pm

@federicobond
Copy link
Owner

Solidity accepts both options.

@federicobond
Copy link
Owner

@LogvinovLeon good catch! Is this documented behaviour? Or just an oversight on the part of the Solidity team? Is there any mention of this in the issues or docs for ethereum/solidity?

@LogvinovLeon
Copy link
Author

@federicobond I don't think it's documented. And I think it's a mistake it supports it. Maybe this should be an issue on a solidity repo instead

@federicobond
Copy link
Owner

Yeah, let's open an issue in the Solidity repo and see if they are willing to tighten things up a bit. I don't think many users expect it to work that way.

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

2 participants