Skip to content

Commit 1fe9e8d

Browse files
pulimsraws-sdk-cpp-automation
authored andcommitted
Revert "Fix sparse trait codegen for nested collections in JSON and CBOR deserializers (#3756)"
This reverts commit cd93e74.
1 parent cd93e74 commit 1fe9e8d

File tree

3 files changed

+0
-24
lines changed

3 files changed

+0
-24
lines changed

tools/code-generation/generator/src/main/resources/com/amazonaws/util/awsclientgenerator/velocity/cpp/cbor/CborDecodeListValue.vm

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -166,11 +166,7 @@ if(peekType_${recursionDepth}.has_value()){
166166
#end
167167
#elseif($shapeMember.list)
168168
#set($template.nestedShapeMember = $shapeMember.listMember.shape)
169-
#if($shapeMember.sparse)
170-
auto nestedList_${recursionDepth} = Vector<Aws::Crt::Optional<$CppViewHelper.computeCppType($template.nestedShapeMember)>>();
171-
#else
172169
auto nestedList_${recursionDepth} = Vector<$CppViewHelper.computeCppType($template.nestedShapeMember)>();
173-
#end
174170
auto peekType_${recursionDepth} = decoder->PeekType();
175171
if (peekType_${recursionDepth}.has_value() && (peekType_${recursionDepth}.value() == CborType::ArrayStart || peekType_${recursionDepth}.value() == CborType::IndefArrayStart))
176172
{
@@ -217,11 +213,7 @@ if(peekType_${recursionDepth}.has_value()){
217213
}
218214
#elseif($shapeMember.map)
219215
#set($template.nestedShapeMember = $shapeMember.mapValue.shape)
220-
#if($shapeMember.sparse)
221-
auto nestedMap_${recursionDepth} = Map<Aws::String, Aws::Crt::Optional<$CppViewHelper.computeCppType($template.nestedShapeMember)>>();
222-
#else
223216
auto nestedMap_${recursionDepth} = Map<Aws::String, $CppViewHelper.computeCppType($template.nestedShapeMember)>();
224-
#end
225217
auto peekType_${recursionDepth} = decoder->PeekType();
226218
if (peekType_${recursionDepth}.has_value() && (peekType_${recursionDepth}.value() == CborType::MapStart || peekType_${recursionDepth}.value() == CborType::IndefMapStart))
227219
{

tools/code-generation/generator/src/main/resources/com/amazonaws/util/awsclientgenerator/velocity/cpp/cbor/CborDecodeMapValue.vm

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -169,11 +169,7 @@ if(peekType_${recursionDepth}){
169169
#end
170170
#elseif($shapeMember.map)
171171
#set($template.nestedShapeMember = $shapeMember.mapValue.shape)
172-
#if($shapeMember.sparse)
173-
auto nestedMap_${recursionDepth} = Map<Aws::String, Aws::Crt::Optional<$CppViewHelper.computeCppType($template.nestedShapeMember)>>();
174-
#else
175172
auto nestedMap_${recursionDepth} = Map<Aws::String, $CppViewHelper.computeCppType($template.nestedShapeMember)>();
176-
#end
177173
auto peekType_${recursionDepth} = decoder->PeekType();
178174
if (peekType_${recursionDepth}.has_value() && (peekType_${recursionDepth}.value() == CborType::MapStart || peekType_${recursionDepth}.value() == CborType::IndefMapStart))
179175
{
@@ -218,11 +214,7 @@ if(peekType_${recursionDepth}){
218214
}
219215
#elseif($shapeMember.list)
220216
#set($template.nestedShapeMember = $shapeMember.listMember.shape)
221-
#if($shapeMember.sparse)
222-
auto nestedList_${recursionDepth} = Vector<Aws::Crt::Optional<$CppViewHelper.computeCppType($template.nestedShapeMember)>>();
223-
#else
224217
auto nestedList_${recursionDepth} = Vector<$CppViewHelper.computeCppType($template.nestedShapeMember)>();
225-
#end
226218
auto peekType_${recursionDepth} = decoder->PeekType();
227219
if (peekType_${recursionDepth}.has_value() && (peekType_${recursionDepth}.value() == CborType::ArrayStart || peekType_${recursionDepth}.value() == CborType::IndefArrayStart))
228220
{

tools/code-generation/generator/src/main/resources/com/amazonaws/util/awsclientgenerator/velocity/cpp/json/ModelInternalMapOrListJsonDeserializer.vm

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -18,11 +18,7 @@
1818
#end
1919
#if($template.recursionDepth > 1)
2020
#set($template.containerVar = ${template.lowerCaseVarName} + "Map")
21-
#if($template.currentShape.sparse)
22-
${template.currentSpaces}Aws::Map<${CppViewHelper.computeCppType($template.currentShape.mapKey.shape)}, Aws::Crt::Optional<${CppViewHelper.computeCppType($template.currentShape.mapValue.shape)}>> ${template.lowerCaseVarName}Map;
23-
#else
2421
${template.currentSpaces}Aws::Map<${CppViewHelper.computeCppType($template.currentShape.mapKey.shape)}, ${CppViewHelper.computeCppType($template.currentShape.mapValue.shape)}> ${template.lowerCaseVarName}Map;
25-
#end
2622
#end
2723
${template.currentSpaces}for(auto& ${template.lowerCaseVarName}Item : ${template.lowerCaseVarName}JsonMap)
2824
${template.currentSpaces}{
@@ -88,11 +84,7 @@
8884
#end
8985
#if($template.recursionDepth > 1)
9086
#set($template.containerVar = ${template.lowerCaseVarName} + "List")
91-
#if($template.currentShape.sparse)
92-
${template.currentSpaces}Aws::Vector<Aws::Crt::Optional<${CppViewHelper.computeCppType($template.currentShape.listMember.shape)}>> ${template.lowerCaseVarName}List;
93-
#else
9487
${template.currentSpaces}Aws::Vector<${CppViewHelper.computeCppType($template.currentShape.listMember.shape)}> ${template.lowerCaseVarName}List;
95-
#end
9688
${template.currentSpaces}${template.lowerCaseVarName}List.reserve((size_t)${template.lowerCaseVarName}JsonList.GetLength());
9789
#end
9890
${template.currentSpaces}for(unsigned ${template.lowerCaseVarName}Index = 0; ${template.lowerCaseVarName}Index < ${template.lowerCaseVarName}JsonList.GetLength(); ++${template.lowerCaseVarName}Index)

0 commit comments

Comments
 (0)