diff options
| author | Federico Bond <federicobond@gmail.com> | 2017-06-17 23:21:46 +0800 |
|---|---|---|
| committer | Federico Bond <federicobond@gmail.com> | 2017-06-17 23:21:46 +0800 |
| commit | d170ceaf3d8f63536a730ccc6388183a7a483514 (patch) | |
| tree | 3c91cb2455eae5fd5faeea8658a4e2ae259e4ae2 | |
| parent | e0b9589e5a5b961541aefe783045b93fac347773 (diff) | |
| download | dexon-solidity-d170ceaf3d8f63536a730ccc6388183a7a483514.tar.gz dexon-solidity-d170ceaf3d8f63536a730ccc6388183a7a483514.tar.zst dexon-solidity-d170ceaf3d8f63536a730ccc6388183a7a483514.zip | |
Display error if payable or constant is specified multiple times
| -rw-r--r-- | libsolidity/parsing/Parser.cpp | 6 | ||||
| -rw-r--r-- | test/libsolidity/SolidityParser.cpp | 18 |
2 files changed, 24 insertions, 0 deletions
diff --git a/libsolidity/parsing/Parser.cpp b/libsolidity/parsing/Parser.cpp index 88b41f20..ec6d11f4 100644 --- a/libsolidity/parsing/Parser.cpp +++ b/libsolidity/parsing/Parser.cpp @@ -323,11 +323,17 @@ Parser::FunctionHeaderParserResult Parser::parseFunctionHeader(bool _forceEmptyN Token::Value token = m_scanner->currentToken(); if (token == Token::Const) { + if (result.isDeclaredConst) + parserError(string("Multiple \"constant\" specifiers.")); + result.isDeclaredConst = true; m_scanner->next(); } else if (m_scanner->currentToken() == Token::Payable) { + if (result.isPayable) + parserError(string("Multiple \"payable\" specifiers.")); + result.isPayable = true; m_scanner->next(); } diff --git a/test/libsolidity/SolidityParser.cpp b/test/libsolidity/SolidityParser.cpp index 27231b9b..78edd4d1 100644 --- a/test/libsolidity/SolidityParser.cpp +++ b/test/libsolidity/SolidityParser.cpp @@ -901,6 +901,24 @@ BOOST_AUTO_TEST_CASE(multiple_visibility_specifiers) CHECK_PARSE_ERROR(text, "Visibility already specified"); } +BOOST_AUTO_TEST_CASE(multiple_payable_specifiers) +{ + char const* text = R"( + contract c { + function f() payable payable {} + })"; + CHECK_PARSE_ERROR(text, "Multiple \"payable\" specifiers."); +} + +BOOST_AUTO_TEST_CASE(multiple_constant_specifiers) +{ + char const* text = R"( + contract c { + function f() constant constant {} + })"; + CHECK_PARSE_ERROR(text, "Multiple \"constant\" specifiers."); +} + BOOST_AUTO_TEST_CASE(literal_constants_with_ether_subdenominations) { char const* text = R"( |