diff options
| author | chriseth <c@ethdev.com> | 2017-03-07 19:34:32 +0800 |
|---|---|---|
| committer | chriseth <c@ethdev.com> | 2017-03-07 19:34:32 +0800 |
| commit | bb06569dc66adbe853bf79273f2eff04d8e36ab6 (patch) | |
| tree | 61d34de9f255d7ca7a2155063de5dd576f6e450b | |
| parent | 573b885337aca75a025c08eea80bb109041e669e (diff) | |
| download | dexon-solidity-bb06569dc66adbe853bf79273f2eff04d8e36ab6.tar.gz dexon-solidity-bb06569dc66adbe853bf79273f2eff04d8e36ab6.tar.zst dexon-solidity-bb06569dc66adbe853bf79273f2eff04d8e36ab6.zip | |
Disallow octal literals.
| -rw-r--r-- | docs/types.rst | 5 | ||||
| -rw-r--r-- | libsolidity/parsing/Scanner.cpp | 3 | ||||
| -rw-r--r-- | test/libsolidity/SolidityScanner.cpp | 18 |
3 files changed, 23 insertions, 3 deletions
diff --git a/docs/types.rst b/docs/types.rst index f7d1d54f..f89a8ee5 100644 --- a/docs/types.rst +++ b/docs/types.rst @@ -197,10 +197,9 @@ Rational and Integer Literals Integer literals are formed from a sequence of numbers in the range 0-9. They are interpreted as decimals. For example, ``69`` means sixty nine. -Octal literals do not exist in Solidity and leading zeros are ignored. -For example, ``0100`` means one hundred. +Octal literals do not exist in Solidity and leading zeros are invalid. -Decimal literals are formed by a ``.`` with at least one number on +Decimal fraction literals are formed by a ``.`` with at least one number on one side. Examples include ``1.``, ``.1`` and ``1.3``. Number literal expressions retain arbitrary precision until they are converted to a non-literal type (i.e. by diff --git a/libsolidity/parsing/Scanner.cpp b/libsolidity/parsing/Scanner.cpp index 3623f23f..0e60fd0b 100644 --- a/libsolidity/parsing/Scanner.cpp +++ b/libsolidity/parsing/Scanner.cpp @@ -758,6 +758,9 @@ Token::Value Scanner::scanNumber(char _charSeen) while (isHexDigit(m_char)) addLiteralCharAndAdvance(); } + else if (isDecimalDigit(m_char)) + // We do not allow octal numbers + return Token::Illegal; } // Parse decimal digits and allow trailing fractional part. if (kind == DECIMAL) diff --git a/test/libsolidity/SolidityScanner.cpp b/test/libsolidity/SolidityScanner.cpp index eb2f042c..3a5c6f24 100644 --- a/test/libsolidity/SolidityScanner.cpp +++ b/test/libsolidity/SolidityScanner.cpp @@ -97,6 +97,24 @@ BOOST_AUTO_TEST_CASE(hex_numbers) BOOST_CHECK_EQUAL(scanner.next(), Token::EOS); } +BOOST_AUTO_TEST_CASE(octal_numbers) +{ + Scanner scanner(CharStream("07")); + BOOST_CHECK_EQUAL(scanner.currentToken(), Token::Illegal); + scanner.reset(CharStream("007"), ""); + BOOST_CHECK_EQUAL(scanner.currentToken(), Token::Illegal); + scanner.reset(CharStream("-07"), ""); + BOOST_CHECK_EQUAL(scanner.currentToken(), Token::Sub); + BOOST_CHECK_EQUAL(scanner.next(), Token::Illegal); + scanner.reset(CharStream("-.07"), ""); + BOOST_CHECK_EQUAL(scanner.currentToken(), Token::Sub); + BOOST_CHECK_EQUAL(scanner.next(), Token::Number); + scanner.reset(CharStream("0"), ""); + BOOST_CHECK_EQUAL(scanner.currentToken(), Token::Number); + scanner.reset(CharStream("0.1"), ""); + BOOST_CHECK_EQUAL(scanner.currentToken(), Token::Number); +} + BOOST_AUTO_TEST_CASE(negative_numbers) { Scanner scanner(CharStream("var x = -.2 + -0x78 + -7.3 + 8.9;")); |