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

Improve failure message when generating getter.

Browse Source
This commit is contained in:
mejsiej 2021-12-02 12:36:04 +01:00 committed by chriseth
parent 8da9239c83
commit ddd9a84141
2 changed files with 9 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
libsolidity/analysis/TypeChecker.cpp
View File

@ -592,8 +592,15 @@ bool TypeChecker::visit(VariableDeclaration const& _variable)
);
}
if (!getter.interfaceFunctionType())
{
solAssert(getter.returnParameterNames().size() == getter.returnParameterTypes().size());
solAssert(getter.parameterNames().size() == getter.parameterTypes().size());
if (getter.returnParameterTypes().empty() && getter.parameterTypes().empty())
m_errorReporter.typeError(5359_error, _variable.location(), "The struct has all its members omitted, therefore the getter cannot return any values.");
else
m_errorReporter.typeError(6744_error, _variable.location(), "Internal or recursive type is not allowed for public state variables.");
}
}
bool isStructMemberDeclaration = dynamic_cast<StructDefinition const*>(_variable.scope()) != nullptr;
if (isStructMemberDeclaration)

2
test/libsolidity/syntaxTests/nameAndTypeResolution/070_struct_accessor_one_array_only.sol
View File

@ -3,4 +3,4 @@ contract test {
Data public data;
}
// ----
// TypeError 6744: (58-74): Internal or recursive type is not allowed for public state variables.
// TypeError 5359: (58-74): The struct has all its members omitted, therefore the getter cannot return any values.