cerc-io/solidity
16
0
Fork 0
mirror of https://github.com/ethereum/solidity synced 2023-10-03 13:03:40 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Added error for interface function with modifiers; test case

Browse Source
This commit is contained in:
hrkrshnn 2020-04-02 20:30:43 +05:30
parent 8d28089abc
commit ef2bef9ddc
2 changed files with 9 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
libsolidity/analysis/SyntaxChecker.cpp
View File

@ -303,7 +303,9 @@ bool SyntaxChecker::visit(FunctionDefinition const& _function)
); );
} }
if (!_function.isImplemented() && !_function.modifiers().empty()) if (m_isInterface && !_function.modifiers().empty())
m_errorReporter.syntaxError(_function.location(), "Functions in interfaces cannot have modifiers.");
else if (!_function.isImplemented() && !_function.modifiers().empty())
m_errorReporter.syntaxError(_function.location(), "Functions without implementation cannot have modifiers."); m_errorReporter.syntaxError(_function.location(), "Functions without implementation cannot have modifiers.");
return true; return true;

6
test/libsolidity/syntaxTests/nameAndTypeResolution/588_interface_function_modifier.sol Normal file
View File

@ -0,0 +1,6 @@
interface I {
function f() external m pure returns (uint);
modifier m() { _; }
}
// ----
// SyntaxError: (16-60): Functions in interfaces cannot have modifiers.