JoinColony / solcover

Code coverage for solidity
MIT License
64 stars 8 forks source link

Support for var(x, y) #25

Closed area closed 7 years ago

area commented 7 years ago

Solidity parser doesn't support this (see https://github.com/ConsenSys/solidity-parser/issues/9), but solparse does. I'm tempted to switch back for now.

cgewecke commented 7 years ago

This is hopefully getting fixed in solidity-parser by solparse merging their code in (next week?).

area commented 7 years ago

Cool, I'll hold off for now then.

cgewecke commented 7 years ago

Tuples fix now very close to being in. They will probably rebuild the parser tomorrow and it should work on whatever ref that is. A couple other fixes were ported from solparse too. Only remaining issue with upgrading to these changes is handling the new node type: StateVariableDeclaration.

area commented 7 years ago

Great, that'll make me much happier!