diff options
author | chriseth <chris@ethereum.org> | 2017-12-12 17:10:29 +0800 |
---|---|---|
committer | chriseth <chris@ethereum.org> | 2017-12-12 17:27:28 +0800 |
commit | 2d171c25e57e821c242b2a6ffa9d6e890e8b207e (patch) | |
tree | 6a12f54f269b9eee51f60570b1519e0b9adf969a /libsolidity/analysis | |
parent | eebeb52aed6edb3896137f796aca344eb831cafc (diff) | |
download | dexon-solidity-2d171c25e57e821c242b2a6ffa9d6e890e8b207e.tar.gz dexon-solidity-2d171c25e57e821c242b2a6ffa9d6e890e8b207e.tar.zst dexon-solidity-2d171c25e57e821c242b2a6ffa9d6e890e8b207e.zip |
Limit number of secondary source locations.
Diffstat (limited to 'libsolidity/analysis')
-rw-r--r-- | libsolidity/analysis/TypeChecker.cpp | 19 |
1 files changed, 5 insertions, 14 deletions
diff --git a/libsolidity/analysis/TypeChecker.cpp b/libsolidity/analysis/TypeChecker.cpp index a578ad0f..090e5159 100644 --- a/libsolidity/analysis/TypeChecker.cpp +++ b/libsolidity/analysis/TypeChecker.cpp @@ -171,13 +171,7 @@ void TypeChecker::checkContractDuplicateFunctions(ContractDefinition const& _con ssl.append("Another declaration is here:", (*it)->location()); string msg = "More than one constructor defined."; - size_t occurrences = ssl.infos.size(); - if (occurrences > 32) - { - ssl.infos.resize(32); - msg += " Truncated from " + boost::lexical_cast<string>(occurrences) + " to the first 32 occurrences."; - } - + ssl.limitSize(msg); m_errorReporter.declarationError( functions[_contract.name()].front()->location(), ssl, @@ -219,12 +213,7 @@ void TypeChecker::findDuplicateDefinitions(map<string, vector<T>> const& _defini if (ssl.infos.size() > 0) { - size_t occurrences = ssl.infos.size(); - if (occurrences > 32) - { - ssl.infos.resize(32); - _message += " Truncated from " + boost::lexical_cast<string>(occurrences) + " to the first 32 occurrences."; - } + ssl.limitSize(_message); m_errorReporter.declarationError( overloads[i]->location(), @@ -1679,10 +1668,12 @@ void TypeChecker::endVisit(NewExpression const& _newExpression) SecondarySourceLocation ssl; for (auto function: contract->annotation().unimplementedFunctions) ssl.append("Missing implementation:", function->location()); + string msg = "Trying to create an instance of an abstract contract."; + ssl.limitSize(msg); m_errorReporter.typeError( _newExpression.location(), ssl, - "Trying to create an instance of an abstract contract." + msg ); } if (!contract->constructorIsPublic()) |