diff options
author | evelkin <evelkin@yandex-team.ru> | 2022-02-10 16:49:25 +0300 |
---|---|---|
committer | Daniil Cherednik <dcherednik@yandex-team.ru> | 2022-02-10 16:49:25 +0300 |
commit | f9f914a9d510572fa0d68959aaea41be5839ff3d (patch) | |
tree | a9fbf49df619eef57f022218df3a1962d611cacf /library/cpp/protobuf | |
parent | 1e0b5488cdce503f4d970e8163823bba9126aedc (diff) | |
download | ydb-f9f914a9d510572fa0d68959aaea41be5839ff3d.tar.gz |
Restoring authorship annotation for <evelkin@yandex-team.ru>. Commit 1 of 2.
Diffstat (limited to 'library/cpp/protobuf')
-rw-r--r-- | library/cpp/protobuf/json/proto2json_printer.cpp | 28 | ||||
-rw-r--r-- | library/cpp/protobuf/json/ut/proto2json_ut.cpp | 16 | ||||
-rw-r--r-- | library/cpp/protobuf/json/ut/test.proto | 16 |
3 files changed, 30 insertions, 30 deletions
diff --git a/library/cpp/protobuf/json/proto2json_printer.cpp b/library/cpp/protobuf/json/proto2json_printer.cpp index 6123eab0f2..26f29547fc 100644 --- a/library/cpp/protobuf/json/proto2json_printer.cpp +++ b/library/cpp/protobuf/json/proto2json_printer.cpp @@ -442,8 +442,8 @@ namespace NProtobufJson { const FieldDescriptor& field, IJsonOutput& json, const TStringBuf key) { - - + + if (field.is_repeated()) PrintRepeatedField(proto, field, json, key); else @@ -456,21 +456,21 @@ namespace NProtobufJson { json.BeginObject(); - // Iterate over all non-extension fields + // Iterate over all non-extension fields for (int f = 0, endF = descriptor->field_count(); f < endF; ++f) { const FieldDescriptor* field = descriptor->field(f); Y_ASSERT(field); PrintField(proto, *field, json); } - // Check extensions via ListFields - std::vector<const FieldDescriptor*> fields; - auto* ref = proto.GetReflection(); - ref->ListFields(proto, &fields); - - for (const FieldDescriptor* field : fields) { - Y_ASSERT(field); - if (field->is_extension()) { + // Check extensions via ListFields + std::vector<const FieldDescriptor*> fields; + auto* ref = proto.GetReflection(); + ref->ListFields(proto, &fields); + + for (const FieldDescriptor* field : fields) { + Y_ASSERT(field); + if (field->is_extension()) { switch (GetConfig().ExtensionFieldNameMode) { case TProto2JsonConfig::ExtFldNameFull: PrintField(proto, *field, json, field->full_name()); @@ -479,9 +479,9 @@ namespace NProtobufJson { PrintField(proto, *field, json); break; } - } - } - + } + } + if (closeMap) { json.EndObject(); } diff --git a/library/cpp/protobuf/json/ut/proto2json_ut.cpp b/library/cpp/protobuf/json/ut/proto2json_ut.cpp index 07e52d7f2f..dc14531941 100644 --- a/library/cpp/protobuf/json/ut/proto2json_ut.cpp +++ b/library/cpp/protobuf/json/ut/proto2json_ut.cpp @@ -1006,17 +1006,17 @@ Y_UNIT_TEST(TestStringifyLongNumbers) { #undef TEST_SINGLE } // TestStringifyLongNumbers -Y_UNIT_TEST(TestExtension) { - TExtensionField proto; - proto.SetExtension(bar, 1); - - Y_ASSERT(proto.HasExtension(bar)); - UNIT_ASSERT_EQUAL(Proto2Json(proto, TProto2JsonConfig()), "{\"NProtobufJsonTest.bar\":1}"); +Y_UNIT_TEST(TestExtension) { + TExtensionField proto; + proto.SetExtension(bar, 1); + + Y_ASSERT(proto.HasExtension(bar)); + UNIT_ASSERT_EQUAL(Proto2Json(proto, TProto2JsonConfig()), "{\"NProtobufJsonTest.bar\":1}"); TProto2JsonConfig cfg; cfg.SetExtensionFieldNameMode(TProto2JsonConfig::ExtFldNameShort); UNIT_ASSERT_EQUAL(Proto2Json(proto, cfg), "{\"bar\":1}"); -} // TestExtension - +} // TestExtension + } // TProto2JsonTest diff --git a/library/cpp/protobuf/json/ut/test.proto b/library/cpp/protobuf/json/ut/test.proto index 0fa996fd41..54253e4186 100644 --- a/library/cpp/protobuf/json/ut/test.proto +++ b/library/cpp/protobuf/json/ut/test.proto @@ -193,11 +193,11 @@ message TSingleRepeatedString { message TSingleRepeatedInt { repeated int32 RepeatedInt = 1; } - -message TExtensionField { - extensions 100 to 199; -} - -extend TExtensionField { - optional int32 bar = 123; -}
\ No newline at end of file + +message TExtensionField { + extensions 100 to 199; +} + +extend TExtensionField { + optional int32 bar = 123; +}
\ No newline at end of file |