ConsenSysMesh / solidity-parser

Solidity Parser in Javascript
138 stars 54 forks source link

Allow expressions in modifier arguments #28

Closed raineorshine closed 7 years ago

raineorshine commented 7 years ago

Fix for https://github.com/ConsenSys/solidity-parser/issues/27

e.g.

contract MyContract {
  function fun() mymodifier(foo.bar()) {}
}
tcoulter commented 7 years ago

Awesome @raineorshine. Can you add the above code example into https://github.com/ConsenSys/solidity-parser/blob/master/test/doc_examples.sol, which is the catch all for testing to assure we can parse everything in that file?

It's not a great way to test each feature, but it's good enough for now.

raineorshine commented 7 years ago

Ready to merge

tcoulter commented 7 years ago

Thanks @raineorshine!