From 2ce6d3bceec703e0d33e0ca7923072b18b5f2e91 Mon Sep 17 00:00:00 2001 From: chriseth Date: Wed, 15 Jul 2015 19:13:42 +0200 Subject: [PATCH] Fix for initialising storage strings. --- ExpressionCompiler.cpp | 17 ++++++++++++++--- 1 file changed, 14 insertions(+), 3 deletions(-) diff --git a/ExpressionCompiler.cpp b/ExpressionCompiler.cpp index 6a2e185c5..470fd7c58 100644 --- a/ExpressionCompiler.cpp +++ b/ExpressionCompiler.cpp @@ -48,12 +48,23 @@ void ExpressionCompiler::appendStateVariableInitialization(VariableDeclaration c { if (!_varDecl.getValue()) return; - solAssert(!!_varDecl.getValue()->getType(), "Type information not available."); + TypePointer type = _varDecl.getValue()->getType(); + solAssert(!!type, "Type information not available."); CompilerContext::LocationSetter locationSetter(m_context, _varDecl); _varDecl.getValue()->accept(*this); - utils().convertType(*_varDecl.getValue()->getType(), *_varDecl.getType(), true); - StorageItem(m_context, _varDecl).storeValue(*_varDecl.getType(), _varDecl.getLocation(), true); + if (_varDecl.getType()->dataStoredIn(DataLocation::Storage)) + { + // reference type, only convert value to mobile type and do final conversion in storeValue. + utils().convertType(*type, *type->mobileType()); + type = type->mobileType(); + } + else + { + utils().convertType(*type, *_varDecl.getType()); + type = _varDecl.getType(); + } + StorageItem(m_context, _varDecl).storeValue(*type, _varDecl.getLocation(), true); } void ExpressionCompiler::appendStateVariableAccessor(VariableDeclaration const& _varDecl)