aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorchriseth <chris@ethereum.org>2017-10-18 01:14:49 +0800
committerchriseth <chris@ethereum.org>2017-10-18 01:14:49 +0800
commit8a8a71de84f5bd71fcea4d31d5a53fde7820ead6 (patch)
treee0e0bc76d0472ed2e05673e2341ce669cd1233a4
parentc99d2aae042643d9d26a4b952e07ca90f11a83c3 (diff)
downloaddexon-solidity-8a8a71de84f5bd71fcea4d31d5a53fde7820ead6.tar
dexon-solidity-8a8a71de84f5bd71fcea4d31d5a53fde7820ead6.tar.gz
dexon-solidity-8a8a71de84f5bd71fcea4d31d5a53fde7820ead6.tar.bz2
dexon-solidity-8a8a71de84f5bd71fcea4d31d5a53fde7820ead6.tar.lz
dexon-solidity-8a8a71de84f5bd71fcea4d31d5a53fde7820ead6.tar.xz
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.cpp2
-rw-r--r--test/libsolidity/SolidityNameAndTypeResolution.cpp18
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)