From 6415ef05d34be35bdcfbd7a9c8edf82485136928 Mon Sep 17 00:00:00 2001 From: Russell Chreptyk Date: Thu, 10 Jun 2021 15:52:19 -0400 Subject: [PATCH] [Python] [Codegen] Fixes nested structs with underscore names (#6686) Fixes #6648 --- src/idl_gen_python.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/idl_gen_python.cpp b/src/idl_gen_python.cpp index e7d6c3c81..62e6d84c8 100644 --- a/src/idl_gen_python.cpp +++ b/src/idl_gen_python.cpp @@ -478,7 +478,7 @@ class PythonGenerator : public BaseGenerator { // a nested struct, prefix the name with the field name. auto subprefix = nameprefix; if (has_field_name) { - subprefix += NormalizedName(field) + fieldname_suffix; + subprefix += MakeLowerCamel(field) + fieldname_suffix; } StructBuilderArgs(*field.value.type.struct_def, subprefix, namesuffix, has_field_name, fieldname_suffix, code_ptr); @@ -518,7 +518,7 @@ class PythonGenerator : public BaseGenerator { indent + " builder.Pad(" + NumToString(field.padding) + ")\n"; if (IsStruct(field_type)) { StructBuilderBody(*field_type.struct_def, - (nameprefix + (NormalizedName(field) + "_")).c_str(), + (nameprefix + (MakeLowerCamel(field) + "_")).c_str(), code_ptr, index, in_array); } else { const auto index_var = "_idx" + NumToString(index); @@ -531,7 +531,7 @@ class PythonGenerator : public BaseGenerator { if (IsStruct(type)) { StructBuilderBody( *field_type.struct_def, - (nameprefix + (NormalizedName(field) + "_")).c_str(), code_ptr, + (nameprefix + (MakeLowerCamel(field) + "_")).c_str(), code_ptr, index + 1, in_array); } else { code += IsArray(field_type) ? " " : "";