diff options
author | chriseth <chris@ethereum.org> | 2017-10-18 01:14:49 +0800 |
---|---|---|
committer | chriseth <chris@ethereum.org> | 2017-10-18 01:14:49 +0800 |
commit | 8a8a71de84f5bd71fcea4d31d5a53fde7820ead6 (patch) | |
tree | e0e0bc76d0472ed2e05673e2341ce669cd1233a4 | |
parent | c99d2aae042643d9d26a4b952e07ca90f11a83c3 (diff) | |
download | dexon-solidity-8a8a71de84f5bd71fcea4d31d5a53fde7820ead6.tar.gz dexon-solidity-8a8a71de84f5bd71fcea4d31d5a53fde7820ead6.tar.zst dexon-solidity-8a8a71de84f5bd71fcea4d31d5a53fde7820ead6.zip |
Only check tuples for valid rational numbers if they have more than one element.
-rw-r--r-- | libsolidity/analysis/TypeChecker.cpp | 2 | ||||
-rw-r--r-- | test/libsolidity/SolidityNameAndTypeResolution.cpp | 18 |
2 files changed, 18 insertions, 2 deletions
diff --git a/libsolidity/analysis/TypeChecker.cpp b/libsolidity/analysis/TypeChecker.cpp index 054912dd..746e762e 100644 --- a/libsolidity/analysis/TypeChecker.cpp +++ b/libsolidity/analysis/TypeChecker.cpp @@ -1295,7 +1295,7 @@ bool TypeChecker::visit(TupleExpression const& _tuple) types.push_back(type(*components[i])); // Note: code generation will visit each of the expression even if they are not assigned from. - if (types[i]->category() == Type::Category::RationalNumber) + if (types[i]->category() == Type::Category::RationalNumber && components.size() > 1) if (!dynamic_cast<RationalNumberType const&>(*types[i]).mobileType()) m_errorReporter.fatalTypeError(components[i]->location(), "Invalid rational number."); diff --git a/test/libsolidity/SolidityNameAndTypeResolution.cpp b/test/libsolidity/SolidityNameAndTypeResolution.cpp index e9066c32..9b0647bf 100644 --- a/test/libsolidity/SolidityNameAndTypeResolution.cpp +++ b/test/libsolidity/SolidityNameAndTypeResolution.cpp @@ -7106,7 +7106,7 @@ BOOST_AUTO_TEST_CASE(invalid_literal_in_tuple) } } )"; - CHECK_ERROR(text, TypeError, "Invalid rational number."); + CHECK_ERROR(text, TypeError, "is not implicitly convertible to expected type"); text = R"( contract C { function f() pure public { @@ -7125,6 +7125,22 @@ BOOST_AUTO_TEST_CASE(invalid_literal_in_tuple) } )"; CHECK_ERROR(text, TypeError, "Invalid rational number."); + text = R"( + contract C { + function f() pure public { + (2**270, 1); + } + } + )"; + CHECK_ERROR(text, TypeError, "Invalid rational number."); + text = R"( + contract C { + function f() pure public { + ((2**270) / 2**100, 1); + } + } + )"; + CHECK_SUCCESS(text); } BOOST_AUTO_TEST_CASE(warn_about_sha3) |