diff options
| author | chriseth <chris@ethereum.org> | 2018-02-27 17:22:17 +0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-02-27 17:22:17 +0800 |
| commit | 415ac2ae87761919d37bd281a9808592d15e544e (patch) | |
| tree | c0a81bfc65d3a991ddced5a954a5c6a63b6f39f3 /libsolidity/analysis/SyntaxChecker.cpp | |
| parent | 1f5eb4ba59891387c2dfcc10103c2f187bf2b4ee (diff) | |
| parent | 5c0d82059f50cb6ac67171ca075f394acae27228 (diff) | |
| download | dexon-solidity-415ac2ae87761919d37bd281a9808592d15e544e.tar dexon-solidity-415ac2ae87761919d37bd281a9808592d15e544e.tar.gz dexon-solidity-415ac2ae87761919d37bd281a9808592d15e544e.tar.bz2 dexon-solidity-415ac2ae87761919d37bd281a9808592d15e544e.tar.lz dexon-solidity-415ac2ae87761919d37bd281a9808592d15e544e.tar.xz dexon-solidity-415ac2ae87761919d37bd281a9808592d15e544e.tar.zst dexon-solidity-415ac2ae87761919d37bd281a9808592d15e544e.zip | |
Merge pull request #3605 from ethereum/deprecate-throw
Turn throw into a syntax error for 0.5.0
Diffstat (limited to 'libsolidity/analysis/SyntaxChecker.cpp')
| -rw-r--r-- | libsolidity/analysis/SyntaxChecker.cpp | 16 |
1 files changed, 12 insertions, 4 deletions
diff --git a/libsolidity/analysis/SyntaxChecker.cpp b/libsolidity/analysis/SyntaxChecker.cpp index 74834ba4..1dcfeb27 100644 --- a/libsolidity/analysis/SyntaxChecker.cpp +++ b/libsolidity/analysis/SyntaxChecker.cpp @@ -174,10 +174,18 @@ bool SyntaxChecker::visit(Break const& _breakStatement) bool SyntaxChecker::visit(Throw const& _throwStatement) { - m_errorReporter.warning( - _throwStatement.location(), - "\"throw\" is deprecated in favour of \"revert()\", \"require()\" and \"assert()\"." - ); + bool const v050 = m_sourceUnit->annotation().experimentalFeatures.count(ExperimentalFeature::V050); + + if (v050) + m_errorReporter.syntaxError( + _throwStatement.location(), + "\"throw\" is deprecated in favour of \"revert()\", \"require()\" and \"assert()\"." + ); + else + m_errorReporter.warning( + _throwStatement.location(), + "\"throw\" is deprecated in favour of \"revert()\", \"require()\" and \"assert()\"." + ); return true; } |