search

ConsenSysMesh / solidity-parser

Solidity Parser in Javascript
137 stars 53 forks source link

Updated version of #11 #12

Closed tcoulter closed 8 years ago

tcoulter commented 8 years ago

See #11.

It turns out that the using statement didn't like array types. Changed type to Type to support this.

duaraghav8 commented 8 years ago

For understanding Using statement, http://solidity.readthedocs.io/en/latest/contracts.html?#using-for was my primary resource. Will keep the tests in mind next time.

Yes, I missed out the Array Declaration my bad. In fact, Type is more appropriate for this. Please go ahead and merge. Thanks for pointing this error out!

duaraghav8 commented 8 years ago

For understanding Using statement, http://solidity.readthedocs.io/en/latest/contracts.html?#using-for was my primary resource. Will keep the tests in mind next time.

Yes, I missed out the Array Declaration my bad. In fact, Type is more appropriate for this. Please go ahead and merge. Thanks for pointing this error out!

tcoulter commented 8 years ago

Merged. Thanks!