fix: client input type member default init error on clang 14

Author: wravery

samples/client/multiple/MultipleQueriesClient.cpp

@@ -122,6 +122,10 @@ const peg::ast& GetRequestObject() noexcept
 }
 
 CompleteTaskInput::CompleteTaskInput() noexcept
+	: id {}
+	, testTaskState {}
+	, isComplete {}
+	, clientMutationId {}
 {
 	// Explicit definition to prevent ODR violations when LTO is enabled.
 }

samples/client/multiple/MultipleQueriesClient.h

@@ -140,10 +140,10 @@ struct [[nodiscard("unnecessary construction")]] CompleteTaskInput
 	CompleteTaskInput& operator=(const CompleteTaskInput& other);
 	CompleteTaskInput& operator=(CompleteTaskInput&& other) noexcept;
 
-	response::IdType id {};
-	std::optional<TaskState> testTaskState {};
-	std::optional<bool> isComplete {};
-	std::optional<std::string> clientMutationId {};
+	response::IdType id;
+	std::optional<TaskState> testTaskState;
+	std::optional<bool> isComplete;
+	std::optional<std::string> clientMutationId;
 };
 
 } // namespace multiple

samples/client/mutate/MutateClient.cpp

@@ -55,6 +55,10 @@ const peg::ast& GetRequestObject() noexcept
 }
 
 CompleteTaskInput::CompleteTaskInput() noexcept
+	: id {}
+	, testTaskState {}
+	, isComplete {}
+	, clientMutationId {}
 {
 	// Explicit definition to prevent ODR violations when LTO is enabled.
 }

samples/client/mutate/MutateClient.h

@@ -73,10 +73,10 @@ struct [[nodiscard("unnecessary construction")]] CompleteTaskInput
 	CompleteTaskInput& operator=(const CompleteTaskInput& other);
 	CompleteTaskInput& operator=(CompleteTaskInput&& other) noexcept;
 
-	response::IdType id {};
-	std::optional<TaskState> testTaskState {};
-	std::optional<bool> isComplete {};
-	std::optional<std::string> clientMutationId {};
+	response::IdType id;
+	std::optional<TaskState> testTaskState;
+	std::optional<bool> isComplete;
+	std::optional<std::string> clientMutationId;
 };
 
 } // namespace mutate

samples/client/nestedinput/NestedInputClient.cpp

@@ -49,6 +49,7 @@ const peg::ast& GetRequestObject() noexcept
 }
 
 InputA::InputA() noexcept
+	: a {}
 {
 	// Explicit definition to prevent ODR violations when LTO is enabled.
 }
@@ -87,6 +88,7 @@ InputA& InputA::operator=(InputA&& other) noexcept
 }
 
 InputB::InputB() noexcept
+	: b {}
 {
 	// Explicit definition to prevent ODR violations when LTO is enabled.
 }
@@ -125,6 +127,11 @@ InputB& InputB::operator=(InputB&& other) noexcept
 }
 
 InputABCD::InputABCD() noexcept
+	: d {}
+	, a {}
+	, b {}
+	, bc {}
+	, value {}
 {
 	// Explicit definition to prevent ODR violations when LTO is enabled.
 }
@@ -183,6 +190,8 @@ InputABCD& InputABCD::operator=(InputABCD&& other) noexcept
 }
 
 InputBC::InputBC() noexcept
+	: c {}
+	, b {}
 {
 	// Explicit definition to prevent ODR violations when LTO is enabled.
 }

samples/client/nestedinput/NestedInputClient.h

@@ -56,7 +56,7 @@ struct [[nodiscard("unnecessary construction")]] InputA
 	InputA& operator=(const InputA& other);
 	InputA& operator=(InputA&& other) noexcept;
 
-	bool a {};
+	bool a;
 };
 
 struct [[nodiscard("unnecessary construction")]] InputB
@@ -71,7 +71,7 @@ struct [[nodiscard("unnecessary construction")]] InputB
 	InputB& operator=(const InputB& other);
 	InputB& operator=(InputB&& other) noexcept;
 
-	double b {};
+	double b;
 };
 
 struct InputBC;
@@ -92,11 +92,11 @@ struct [[nodiscard("unnecessary construction")]] InputABCD
 	InputABCD& operator=(const InputABCD& other);
 	InputABCD& operator=(InputABCD&& other) noexcept;
 
-	std::string d {};
-	InputA a {};
-	InputB b {};
-	std::vector<InputBC> bc {};
-	int value {};
+	std::string d;
+	InputA a;
+	InputB b;
+	std::vector<InputBC> bc;
+	int value;
 };
 
 struct [[nodiscard("unnecessary construction")]] InputBC
@@ -112,8 +112,8 @@ struct [[nodiscard("unnecessary construction")]] InputBC
 	InputBC& operator=(const InputBC& other);
 	InputBC& operator=(InputBC&& other) noexcept;
 
-	response::IdType c {};
-	InputB b {};
+	response::IdType c;
+	InputB b;
 };
 
 } // namespace nestedinput

src/ClientGenerator.cpp

@@ -319,7 +319,7 @@ static_assert(graphql::internal::MinorVersion == )cpp"
 				headerFile << R"cpp(	)cpp"
 						   << _requestLoader.getInputCppType(inputField->type().lock())
 						   << R"cpp( )cpp" << SchemaLoader::getSafeCppName(inputField->name())
-						   << R"cpp( {};
+						   << R"cpp(;
 )cpp";
 			}
 
@@ -648,15 +648,28 @@ using namespace std::literals;
 
 			pendingSeparator.reset();
 
-			sourceFile << cppType << R"cpp(::)cpp" << cppType << R"cpp(() noexcept
+			sourceFile << cppType << R"cpp(::)cpp" << cppType << R"cpp(() noexcept)cpp";
+
+			bool firstField = true;
+
+			for (const auto& inputField : inputType.type->inputFields())
+			{
+				sourceFile << R"cpp(
+	)cpp" << (firstField ? R"cpp(:)cpp" : R"cpp(,)cpp")
+						   << R"cpp( )cpp" << SchemaLoader::getSafeCppName(inputField->name())
+						   << R"cpp( {})cpp";
+				firstField = false;
+			}
+
+			sourceFile << R"cpp(
 {
 	// Explicit definition to prevent ODR violations when LTO is enabled.
 }
 
 )cpp" << cppType << R"cpp(::)cpp"
 					   << cppType << R"cpp(()cpp";
 
-			bool firstField = true;
+			firstField = true;
 
 			for (const auto& inputField : inputType.type->inputFields())
 			{
@@ -854,8 +867,7 @@ response::Value Variable<)cpp"
 
 				sourceFile << R"cpp(template <>
 response::Value Variable<)cpp"
-						   << cppType << R"cpp(>::serialize()cpp" << cppType
-						   << R"cpp(&& inputValue)
+						   << cppType << R"cpp(>::serialize()cpp" << cppType << R"cpp(&& inputValue)
 {
 	response::Value result { response::Type::Map };