Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SyntaxError #73

Closed
xanderdunn opened this issue Apr 17, 2017 · 4 comments
Closed

SyntaxError #73

xanderdunn opened this issue Apr 17, 2017 · 4 comments

Comments

@xanderdunn
Copy link

xanderdunn commented Apr 17, 2017

solium 0.2.2

Anytime I attempt to lint the Numeraire contracts, I get this SyntaxError:

$ solium --file contracts/Numeraire_dependencies.sol
An error occurred while running the linter on contracts/Numeraire_dependencies.sol:
Error: An error occured while parsing the source code:
SyntaxError: Expected "!=", "!==", "%", "%=", "&", "&&", "&=", "(", "*", "**", "*=", "+", "++", "+=", ",", "-", "--", "-=", ".", "/", "/*", "//", "/=", ";", "<", "<<", "<<=", "<=", "=", "==", "===", ">", ">=", ">>", ">>=", ">>>", ">>>=", "?", "[", "^", "^=", "in", "instanceof", "|", "|=", "||", comment, end of line or whitespace but "{" found. Line: 130, Column: 93
    at EventEmitter.lint (/Users/admin/.nvm/versions/node/v6.9.2/lib/node_modules/solium/lib/solium.js:88:10)
    at lintString (/Users/admin/.nvm/versions/node/v6.9.2/lib/node_modules/solium/lib/cli.js:126:24)
    at lintFile (/Users/admin/.nvm/versions/node/v6.9.2/lib/node_modules/solium/lib/cli.js:157:9)
    at filesToLint.map.file (/Users/admin/.nvm/versions/node/v6.9.2/lib/node_modules/solium/lib/cli.js:176:45)
    at Array.map (native)
    at lint (/Users/admin/.nvm/versions/node/v6.9.2/lib/node_modules/solium/lib/cli.js:176:33)
    at Object.execute (/Users/admin/.nvm/versions/node/v6.9.2/lib/node_modules/solium/lib/cli.js:287:19)
    at Object.<anonymous> (/Users/admin/.nvm/versions/node/v6.9.2/lib/node_modules/solium/bin/solium.js:11:5)
    at Module._compile (module.js:570:32)
    at Object.Module._extensions..js (module.js:579:10)
No issues found.

The contracts compile fine with solc 0.4.10.

@duaraghav8
Copy link
Owner

Thanks for reporting this @xanderdunn

xanderdunn added a commit to erasureprotocol/NMR that referenced this issue Apr 24, 2017
@alexstep
Copy link

alexstep commented May 8, 2017

Maybe you can add something like https://github.com/Latrasis/linter-solidity/blob/master/lib/compiler.js ?

@travs
Copy link

travs commented Jun 13, 2017

Similar issue with solidity-parser was fixed over here, in case solium makes the switch.
Also relevant to #58

duaraghav8 pushed a commit that referenced this issue Jun 16, 2017
@duaraghav8
Copy link
Owner

Solium has now switched to solidity-parser as dependancy instead of solparse. So this won't give you trouble once v0.5.0 is released (which is very soon :) )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants