mirror of
https://github.com/ethereum/solidity
synced 2023-10-03 13:03:40 +00:00
Do not report overload conflicts for every line excessively
This commit is contained in:
parent
f2412da800
commit
5ae9b8dab1
@ -173,13 +173,17 @@ void TypeChecker::checkContractDuplicateFunctions(ContractDefinition const& _con
|
|||||||
for (auto const& it: functions)
|
for (auto const& it: functions)
|
||||||
{
|
{
|
||||||
vector<FunctionDefinition const*> const& overloads = it.second;
|
vector<FunctionDefinition const*> const& overloads = it.second;
|
||||||
for (size_t i = 0; i < overloads.size(); ++i)
|
set<size_t> reported;
|
||||||
|
for (size_t i = 0; i < overloads.size() && !reported.count(i); ++i)
|
||||||
{
|
{
|
||||||
SecondarySourceLocation ssl;
|
SecondarySourceLocation ssl;
|
||||||
|
|
||||||
for (size_t j = i + 1; j < overloads.size(); ++j)
|
for (size_t j = i + 1; j < overloads.size(); ++j)
|
||||||
if (FunctionType(*overloads[i]).hasEqualArgumentTypes(FunctionType(*overloads[j])))
|
if (FunctionType(*overloads[i]).hasEqualArgumentTypes(FunctionType(*overloads[j])))
|
||||||
|
{
|
||||||
ssl.append("Other declaration is here:", overloads[j]->location());
|
ssl.append("Other declaration is here:", overloads[j]->location());
|
||||||
|
reported.insert(j);
|
||||||
|
}
|
||||||
|
|
||||||
if (ssl.infos.size() > 0)
|
if (ssl.infos.size() > 0)
|
||||||
m_errorReporter.declarationError(
|
m_errorReporter.declarationError(
|
||||||
|
Loading…
Reference in New Issue
Block a user