diff options
author | Alex Beregszaszi <alex@rtfs.hu> | 2018-07-24 01:59:08 +0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-07-24 01:59:08 +0800 |
commit | b3c8e14952f86ab658ac456f5cce71609498a348 (patch) | |
tree | 0ece76b37c3a63e68924bb541cc5eab39abf6df8 | |
parent | f887d5162744d25d54ba22033a48d398ce88edbd (diff) | |
parent | 9f35f0b80571927ec89d77ec6827332c2ab60eba (diff) | |
download | dexon-solidity-b3c8e14952f86ab658ac456f5cce71609498a348.tar dexon-solidity-b3c8e14952f86ab658ac456f5cce71609498a348.tar.gz dexon-solidity-b3c8e14952f86ab658ac456f5cce71609498a348.tar.bz2 dexon-solidity-b3c8e14952f86ab658ac456f5cce71609498a348.tar.lz dexon-solidity-b3c8e14952f86ab658ac456f5cce71609498a348.tar.xz dexon-solidity-b3c8e14952f86ab658ac456f5cce71609498a348.tar.zst dexon-solidity-b3c8e14952f86ab658ac456f5cce71609498a348.zip |
Merge pull request #4411 from ethereum/v050-functions-with-modifiers-no-impl
[BREAKING] Disallow functions without implementation to use modifiers.
4 files changed, 6 insertions, 11 deletions
diff --git a/Changelog.md b/Changelog.md index c2aa3f17..59d0aa59 100644 --- a/Changelog.md +++ b/Changelog.md @@ -51,6 +51,7 @@ Breaking Changes: * Type Checker: Address members are not included in contract types anymore. An explicit conversion is now required before invoking an ``address`` member from a contract. * Remove obsolete ``std`` directory from the Solidity repository. This means accessing ``https://github.com/ethereum/soldity/blob/develop/std/*.sol`` (or ``https://github.com/ethereum/solidity/std/*.sol`` in Remix) will not be possible. * References Resolver: Turn missing storage locations into an error. This was already the case in the experimental 0.5.0 mode. + * Syntax Checker: Disallow functions without implementation to use modifiers. This was already the case in the experimental 0.5.0 mode. * Syntax Checker: Named return values in function types are an error. * Syntax Checker: Strictly require visibility specifier for functions. This was already the case in the experimental 0.5.0 mode. * Syntax Checker: Disallow unary ``+``. This was already the case in the experimental 0.5.0 mode. diff --git a/libsolidity/analysis/SyntaxChecker.cpp b/libsolidity/analysis/SyntaxChecker.cpp index 5d16a33f..77492499 100644 --- a/libsolidity/analysis/SyntaxChecker.cpp +++ b/libsolidity/analysis/SyntaxChecker.cpp @@ -213,8 +213,6 @@ bool SyntaxChecker::visit(ContractDefinition const& _contract) bool SyntaxChecker::visit(FunctionDefinition const& _function) { - bool const v050 = m_sourceUnit->annotation().experimentalFeatures.count(ExperimentalFeature::V050); - if (_function.noVisibilitySpecified()) { string suggestedVisibility = _function.isFallback() || m_isInterface ? "external" : "public"; @@ -225,12 +223,8 @@ bool SyntaxChecker::visit(FunctionDefinition const& _function) } if (!_function.isImplemented() && !_function.modifiers().empty()) - { - if (v050) - m_errorReporter.syntaxError(_function.location(), "Functions without implementation cannot have modifiers."); - else - m_errorReporter.warning(_function.location(), "Modifiers of functions without implementation are ignored." ); - } + m_errorReporter.syntaxError(_function.location(), "Functions without implementation cannot have modifiers."); + return true; } diff --git a/test/libsolidity/syntaxTests/functionTypes/function_type_constructor.sol b/test/libsolidity/syntaxTests/functionTypes/function_type_constructor.sol index 6549eb48..51f0b10d 100644 --- a/test/libsolidity/syntaxTests/functionTypes/function_type_constructor.sol +++ b/test/libsolidity/syntaxTests/functionTypes/function_type_constructor.sol @@ -3,5 +3,5 @@ contract C { constructor() public x; } // ---- -// Warning: (83-106): Modifiers of functions without implementation are ignored. +// SyntaxError: (83-106): Functions without implementation cannot have modifiers. // DeclarationError: (104-105): Undeclared identifier. diff --git a/test/libsolidity/syntaxTests/modifiers/modifiers_on_abstract_functions_no_parser_error.sol b/test/libsolidity/syntaxTests/modifiers/modifiers_on_abstract_functions_no_parser_error.sol index e18c5cf9..2e86fcc1 100644 --- a/test/libsolidity/syntaxTests/modifiers/modifiers_on_abstract_functions_no_parser_error.sol +++ b/test/libsolidity/syntaxTests/modifiers/modifiers_on_abstract_functions_no_parser_error.sol @@ -9,5 +9,5 @@ contract C function bar() public only_owner; } // ---- -// Warning: (203-236): Modifiers of functions without implementation are ignored. -// Warning: (241-274): Modifiers of functions without implementation are ignored. +// SyntaxError: (203-236): Functions without implementation cannot have modifiers. +// SyntaxError: (241-274): Functions without implementation cannot have modifiers. |