Comment immediately following function's opening brace breaks instrumentation

[sc-forks]

Comment immediately following function's opening brace breaks instrumentation

[area]

Reopening issue because the interaction with comments is a legitimate bug for us to fix.

[area]

You might have got lucky here without providing further context. Do you happen to have something like:

returns (uint256) {//This is a comment on the function

[jeromerousselot]

You are right! I added a space and it passes.

I now hit a solidity bug about calling twice the same modifier (with two different values).
Should be fixed in 0.4.15 I think, and doesnt exist in 0.4.11.
solidity-coverage currently depends on 0.4.13.

ethereum/solidity#2621

,/home/jerome/src/eth/contracts/coverageEnv/contracts/HeroCoin.sol:356:5: DeclarationError: Modifier already used for this function.
    updateAccount(_to) //update receivers rake before transfer as well, to avoid over-attributing rake
    ^----------------^
,/home/jerome/src/eth/contracts/coverageEnv/contracts/HeroCoin.sol:372:5: DeclarationError: Modifier already used for this function.
    updateAccount(_to) //update receivers rake before transfer as well, to avoid over-attributing rake

[area]

That's only a dev dependency, needed for running our tests. During the tests of your project, your contracts should still be compiled by truffle, I think, so I don't think that is a solidity-coverage issue.

[jeromerousselot]

I agree, ultimately I think it is a package management issue.
I have a globally installed truffle using globally installed solc 0.4.11 which compiles my contracts.
Locally I have another truffle and solidity-coverage. When running solidity-coverage I believe it uses solc 0.4.13.
I'll try again when truffle uses latest solc.
Thanks!