mirror of
https://github.com/ethereum/solidity
synced 2023-10-03 13:03:40 +00:00
Add structs and enums to contract types.
This commit is contained in:
parent
938ed70935
commit
54e3637d23
@ -1042,12 +1042,14 @@ MemberList::MemberMap ContractType::nativeMembers(ContractDefinition const*) con
|
||||
}
|
||||
}
|
||||
else
|
||||
{
|
||||
for (auto const& it: m_contract.interfaceFunctions())
|
||||
members.push_back(MemberList::Member(
|
||||
it.second->declaration().name(),
|
||||
it.second->asMemberFunction(m_contract.isLibrary()),
|
||||
&it.second->declaration()
|
||||
));
|
||||
}
|
||||
return members;
|
||||
}
|
||||
|
||||
@ -1858,6 +1860,12 @@ MemberList::MemberMap TypeType::nativeMembers(ContractDefinition const* _current
|
||||
if (m_actualType->category() == Category::Contract)
|
||||
{
|
||||
ContractDefinition const& contract = dynamic_cast<ContractType const&>(*m_actualType).contractDefinition();
|
||||
bool isBase = false;
|
||||
if (_currentScope != nullptr)
|
||||
{
|
||||
auto const& currentBases = _currentScope->annotation().linearizedBaseContracts;
|
||||
isBase = (find(currentBases.begin(), currentBases.end(), &contract) != currentBases.end());
|
||||
}
|
||||
if (contract.isLibrary())
|
||||
for (auto const& it: contract.interfaceFunctions())
|
||||
members.push_back(MemberList::Member(
|
||||
@ -1865,14 +1873,19 @@ MemberList::MemberMap TypeType::nativeMembers(ContractDefinition const* _current
|
||||
it.second->asMemberFunction(true), // use callcode
|
||||
&it.second->declaration()
|
||||
));
|
||||
else if (_currentScope != nullptr)
|
||||
if (isBase)
|
||||
{
|
||||
auto const& currentBases = _currentScope->annotation().linearizedBaseContracts;
|
||||
if (find(currentBases.begin(), currentBases.end(), &contract) != currentBases.end())
|
||||
// We are accessing the type of a base contract, so add all public and protected
|
||||
// members. Note that this does not add inherited functions on purpose.
|
||||
for (Declaration const* decl: contract.inheritableMembers())
|
||||
members.push_back(MemberList::Member(decl->name(), decl->type(), decl));
|
||||
// We are accessing the type of a base contract, so add all public and protected
|
||||
// members. Note that this does not add inherited functions on purpose.
|
||||
for (Declaration const* decl: contract.inheritableMembers())
|
||||
members.push_back(MemberList::Member(decl->name(), decl->type(), decl));
|
||||
}
|
||||
else
|
||||
{
|
||||
for (auto const& stru: contract.definedStructs())
|
||||
members.push_back(MemberList::Member(stru->name(), stru->type(), stru));
|
||||
for (auto const& enu: contract.definedEnums())
|
||||
members.push_back(MemberList::Member(enu->name(), enu->type(), enu));
|
||||
}
|
||||
}
|
||||
else if (m_actualType->category() == Category::Enum)
|
||||
|
@ -913,14 +913,16 @@ void ExpressionCompiler::endVisit(MemberAccess const& _memberAccess)
|
||||
|
||||
if (dynamic_cast<ContractType const*>(type.actualType().get()))
|
||||
{
|
||||
auto const& funType = dynamic_cast<FunctionType const&>(*_memberAccess.annotation().type);
|
||||
if (funType.location() != FunctionType::Location::Internal)
|
||||
m_context << funType.externalIdentifier();
|
||||
else
|
||||
if (auto funType = dynamic_cast<FunctionType const*>(_memberAccess.annotation().type.get()))
|
||||
{
|
||||
auto const* function = dynamic_cast<FunctionDefinition const*>(_memberAccess.annotation().referencedDeclaration);
|
||||
solAssert(!!function, "Function not found in member access");
|
||||
m_context << m_context.functionEntryLabel(*function).pushTag();
|
||||
if (funType->location() != FunctionType::Location::Internal)
|
||||
m_context << funType->externalIdentifier();
|
||||
else
|
||||
{
|
||||
auto const* function = dynamic_cast<FunctionDefinition const*>(_memberAccess.annotation().referencedDeclaration);
|
||||
solAssert(!!function, "Function not found in member access");
|
||||
m_context << m_context.functionEntryLabel(*function).pushTag();
|
||||
}
|
||||
}
|
||||
}
|
||||
else if (auto enumType = dynamic_cast<EnumType const*>(type.actualType().get()))
|
||||
|
@ -5377,6 +5377,21 @@ BOOST_AUTO_TEST_CASE(library_stray_values)
|
||||
BOOST_CHECK(callContractFunction("f(uint256)", u256(33)) == encodeArgs(u256(42)));
|
||||
}
|
||||
|
||||
BOOST_AUTO_TEST_CASE(cross_contract_types)
|
||||
{
|
||||
char const* sourceCode = R"(
|
||||
contract Lib { struct S {uint a; uint b; } }
|
||||
contract Test {
|
||||
function f() returns (uint r) {
|
||||
var x = Lib.S({a: 2, b: 3});
|
||||
r = x.b;
|
||||
}
|
||||
}
|
||||
)";
|
||||
compileAndRun(sourceCode, 0, "Test");
|
||||
BOOST_CHECK(callContractFunction("f()") == encodeArgs(u256(3)));
|
||||
}
|
||||
|
||||
BOOST_AUTO_TEST_CASE(simple_throw)
|
||||
{
|
||||
char const* sourceCode = R"(
|
||||
|
@ -1348,6 +1348,21 @@ BOOST_AUTO_TEST_CASE(enum_member_access)
|
||||
BOOST_CHECK(success(text));
|
||||
}
|
||||
|
||||
BOOST_AUTO_TEST_CASE(enum_member_access_accross_contracts)
|
||||
{
|
||||
char const* text = R"(
|
||||
contract Interface {
|
||||
enum MyEnum { One, Two }
|
||||
}
|
||||
contract Impl {
|
||||
function test() returns (Interface.MyEnum) {
|
||||
return Interface.MyEnum.One;
|
||||
}
|
||||
}
|
||||
)";
|
||||
BOOST_CHECK(success(text));
|
||||
}
|
||||
|
||||
BOOST_AUTO_TEST_CASE(enum_invalid_member_access)
|
||||
{
|
||||
char const* text = R"(
|
||||
|
Loading…
Reference in New Issue
Block a user