From 75154b380b5bccbb9ec1d350dc23e5a41e2b8714 Mon Sep 17 00:00:00 2001
From: Miki Rozloznik <mikulas.rozloznik@eccam.com>
Date: Wed, 30 Oct 2019 09:02:18 +0100
Subject: [PATCH] [#146] Fix C++ field constructor for optional fields with
 simple types

---
 compiler/extensions/cpp/freemarker/CompoundField.inc.ftl | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/compiler/extensions/cpp/freemarker/CompoundField.inc.ftl b/compiler/extensions/cpp/freemarker/CompoundField.inc.ftl
index c2b2f1703..88ea4bae4 100644
--- a/compiler/extensions/cpp/freemarker/CompoundField.inc.ftl
+++ b/compiler/extensions/cpp/freemarker/CompoundField.inc.ftl
@@ -25,7 +25,7 @@ ${I}if (in.readBool())
 ${I}{
         <@compound_read_field_inner field, compoundName, indent + 1/>
 ${I}}
-${I}
+
 ${I}return ::zserio::NullOpt;
     <#else>
     <@compound_read_field_inner field, compoundName, indent/>
@@ -547,10 +547,10 @@ ${I}<@field_member_name field/>(::std::move(other.<@field_member_name field/>))<
 ${I}m_objectChoice(::std::forward<ZSERIO_T>(value))
     <#else>
 ${I}<@field_member_name field/>(<#rt>
-        <#if field.isSimpleType>
-            <@field_argument_name field/><#t>
-        <#else>
+        <#if !field.isSimpleType || field.optional??>
             ::std::forward<ZSERIO_T_${field.name}>(<@field_argument_name field/>)<#t>
+        <#else>
+            <@field_argument_name field/><#t>
         </#if>
         <#lt>)<#if hasNext>,</#if>
     </#if>