diff --git a/AST.cpp b/AST.cpp index b05602639..605f53521 100644 --- a/AST.cpp +++ b/AST.cpp @@ -189,7 +189,7 @@ vector, FunctionTypePointer>> const& ContractDefinition::getIn for (ContractDefinition const* contract: getLinearizedBaseContracts()) { for (ASTPointer const& f: contract->getDefinedFunctions()) - if (f->isPublic() && !f->isConstructor() && !f->getName().empty() && functionsSeen.count(f->getName()) == 0) + if (functionsSeen.count(f->getName()) == 0 && f->isPartOfExternalInterface()) { functionsSeen.insert(f->getName()); FixedHash<4> hash(dev::sha3(f->getCanonicalSignature())); @@ -197,7 +197,7 @@ vector, FunctionTypePointer>> const& ContractDefinition::getIn } for (ASTPointer const& v: contract->getStateVariables()) - if (v->isPublic() && functionsSeen.count(v->getName()) == 0 && !v->isConstant()) + if (functionsSeen.count(v->getName()) == 0 && v->isPartOfExternalInterface()) { FunctionType ftype(*v); functionsSeen.insert(v->getName()); diff --git a/AST.h b/AST.h index e959ba248..6a269e154 100644 --- a/AST.h +++ b/AST.h @@ -156,6 +156,7 @@ public: /// contract types. virtual TypePointer getType(ContractDefinition const* m_currentContract = nullptr) const = 0; virtual bool isLValue() const { return false; } + virtual bool isPartOfExternalInterface() const { return false; }; protected: virtual Visibility getDefaultVisibility() const { return Visibility::Public; } @@ -415,6 +416,7 @@ public: getVisibility() >= Visibility::Internal; } virtual TypePointer getType(ContractDefinition const*) const override; + virtual bool isPartOfExternalInterface() const override { return isPublic() && !m_isConstructor && !getName().empty(); } /// Checks that all parameters have allowed types and calls checkTypeRequirements on the body. void checkTypeRequirements(); @@ -468,6 +470,8 @@ public: void setType(std::shared_ptr const& _type) { m_type = _type; } virtual bool isLValue() const override; + virtual bool isPartOfExternalInterface() const override { return isPublic() && !m_isConstant; } + void checkTypeRequirements(); bool isLocalVariable() const { return !!dynamic_cast(getScope()); } diff --git a/ExpressionCompiler.cpp b/ExpressionCompiler.cpp index ce5f184b4..9391bc2a7 100644 --- a/ExpressionCompiler.cpp +++ b/ExpressionCompiler.cpp @@ -859,6 +859,8 @@ void ExpressionCompiler::endVisit(Identifier const& _identifier) { if (!variable->isConstant()) setLValueFromDeclaration(*declaration, _identifier); + else + variable->getValue()->accept(*this); } else if (dynamic_cast(declaration)) { diff --git a/Parser.cpp b/Parser.cpp index 05c05f632..9efa30061 100644 --- a/Parser.cpp +++ b/Parser.cpp @@ -135,7 +135,6 @@ ASTPointer Parser::parseContractDefinition() } while (m_scanner->getCurrentToken() == Token::Comma); expectToken(Token::LBrace); - bool isDeclaredConst = false; while (true) { Token::Value currentToken = m_scanner->getCurrentToken(); @@ -153,21 +152,13 @@ ASTPointer Parser::parseContractDefinition() VarDeclParserOptions options; options.isStateVariable = true; options.allowInitialValue = true; - options.isDeclaredConst = isDeclaredConst; stateVariables.push_back(parseVariableDeclaration(options)); - isDeclaredConst = false; expectToken(Token::Semicolon); } else if (currentToken == Token::Modifier) modifiers.push_back(parseModifierDefinition()); else if (currentToken == Token::Event) events.push_back(parseEventDefinition()); - else if (currentToken == Token::Const) - { - solAssert(Token::isElementaryTypeName(m_scanner->peekNextToken()), ""); - isDeclaredConst = true; - m_scanner->next(); - } else BOOST_THROW_EXCEPTION(createParserError("Function, variable, struct or modifier declaration expected.")); } @@ -326,6 +317,7 @@ ASTPointer Parser::parseVariableDeclaration( nodeFactory.setEndPositionFromNode(type); } bool isIndexed = false; + bool isDeclaredConst = false; ASTPointer identifier; Token::Value token = m_scanner->getCurrentToken(); Declaration::Visibility visibility(Declaration::Visibility::Default); @@ -336,7 +328,13 @@ ASTPointer Parser::parseVariableDeclaration( isIndexed = true; m_scanner->next(); } + if (token == Token::Const) + { + m_scanner->next(); + isDeclaredConst = true; + } nodeFactory.markEndPosition(); + if (_options.allowEmptyName && m_scanner->getCurrentToken() != Token::Identifier) { identifier = make_shared(""); @@ -357,7 +355,7 @@ ASTPointer Parser::parseVariableDeclaration( } return nodeFactory.createNode(type, identifier, value, visibility, _options.isStateVariable, - isIndexed, _options.isDeclaredConst); + isIndexed, isDeclaredConst); } ASTPointer Parser::parseModifierDefinition() diff --git a/Parser.h b/Parser.h index 469e446fd..08c47c252 100644 --- a/Parser.h +++ b/Parser.h @@ -54,7 +54,6 @@ private: bool allowIndexed = false; bool allowEmptyName = false; bool allowInitialValue = false; - bool isDeclaredConst = false; }; ///@{ @@ -67,8 +66,7 @@ private: ASTPointer parseStructDefinition(); ASTPointer parseEnumDefinition(); ASTPointer parseEnumValue(); - ASTPointer parseVariableDeclaration( - VarDeclParserOptions const& _options = VarDeclParserOptions(), + ASTPointer parseVariableDeclaration(VarDeclParserOptions const& _options = VarDeclParserOptions(), ASTPointer const& _lookAheadArrayType = ASTPointer()); ASTPointer parseModifierDefinition(); ASTPointer parseEventDefinition();