diff options
| author | chriseth <chris@ethereum.org> | 2017-06-27 22:32:48 +0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2017-06-27 22:32:48 +0800 |
| commit | 708d17d13fce9095178962c2ebc3fa17303b72f3 (patch) | |
| tree | 774625d27d8f372e10132760248b349016247f94 /libsolidity | |
| parent | 36044c8c95890bfc25a199510e32a0481e8082d0 (diff) | |
| parent | 70fd5c17704119a3f52aba5c67eda96c2a222d2d (diff) | |
| download | dexon-solidity-708d17d13fce9095178962c2ebc3fa17303b72f3.tar.gz dexon-solidity-708d17d13fce9095178962c2ebc3fa17303b72f3.tar.zst dexon-solidity-708d17d13fce9095178962c2ebc3fa17303b72f3.zip | |
Merge pull request #2464 from federicobond/deprecate-function-type-names
Warn deprecated usage of parameter names in function types
Diffstat (limited to 'libsolidity')
| -rw-r--r-- | libsolidity/analysis/SyntaxChecker.cpp | 12 | ||||
| -rw-r--r-- | libsolidity/analysis/SyntaxChecker.h | 2 |
2 files changed, 14 insertions, 0 deletions
diff --git a/libsolidity/analysis/SyntaxChecker.cpp b/libsolidity/analysis/SyntaxChecker.cpp index 35d71d85..02e2fdcf 100644 --- a/libsolidity/analysis/SyntaxChecker.cpp +++ b/libsolidity/analysis/SyntaxChecker.cpp @@ -148,3 +148,15 @@ bool SyntaxChecker::visit(PlaceholderStatement const&) return true; } +bool SyntaxChecker::visit(FunctionTypeName const& _node) +{ + for (auto const& decl: _node.parameterTypeList()->parameters()) + if (!decl->name().empty()) + m_errorReporter.warning(decl->location(), "Naming function type parameters is deprecated."); + + for (auto const& decl: _node.returnParameterTypeList()->parameters()) + if (!decl->name().empty()) + m_errorReporter.warning(decl->location(), "Naming function type return parameters is deprecated."); + + return true; +} diff --git a/libsolidity/analysis/SyntaxChecker.h b/libsolidity/analysis/SyntaxChecker.h index b1857aa5..ec6ac434 100644 --- a/libsolidity/analysis/SyntaxChecker.h +++ b/libsolidity/analysis/SyntaxChecker.h @@ -63,6 +63,8 @@ private: virtual bool visit(PlaceholderStatement const& _placeholderStatement) override; + virtual bool visit(FunctionTypeName const& _node) override; + ErrorReporter& m_errorReporter; /// Flag that indicates whether a function modifier actually contains '_'. |