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 | |
parent | 1e0b5488cdce503f4d970e8163823bba9126aedc (diff) | |
download | ydb-f9f914a9d510572fa0d68959aaea41be5839ff3d.tar.gz |
Restoring authorship annotation for <evelkin@yandex-team.ru>. Commit 1 of 2.
Diffstat (limited to 'library')
-rw-r--r-- | library/cpp/enumbitset/enumbitset.h | 50 | ||||
-rw-r--r-- | library/cpp/object_factory/object_factory.h | 34 | ||||
-rw-r--r-- | library/cpp/object_factory/object_factory_ut.cpp | 56 | ||||
-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 |
6 files changed, 100 insertions, 100 deletions
diff --git a/library/cpp/enumbitset/enumbitset.h b/library/cpp/enumbitset/enumbitset.h index 41864c3a04..1fdeaa2744 100644 --- a/library/cpp/enumbitset/enumbitset.h +++ b/library/cpp/enumbitset/enumbitset.h @@ -2,7 +2,7 @@ #include <util/ysaveload.h> #include <util/generic/bitmap.h> -#include <util/generic/serialized_enum.h> +#include <util/generic/serialized_enum.h> #include <util/generic/yexception.h> #include <util/string/cast.h> #include <util/string/printf.h> @@ -216,14 +216,14 @@ public: return TThis::operator^(TThis(c)); } - auto operator[] (TEnum e) { - return TParent::operator[](this->Pos(e)); - } - - auto operator[] (TEnum e) const { - return TParent::operator[](this->Pos(e)); - } - + auto operator[] (TEnum e) { + return TParent::operator[](this->Pos(e)); + } + + auto operator[] (TEnum e) const { + return TParent::operator[](this->Pos(e)); + } + using TParent::Count; using TParent::Empty; @@ -485,22 +485,22 @@ public: return ebs; } }; - -/* For Enums with GENERATE_ENUM_SERIALIZATION_WITH_HEADER */ -template <typename TEnum> -class TGeneratedEnumBitSet : public TEnumBitSet<TEnum, 0, GetEnumItemsCount<TEnum>()> { -public: - using TParent = TEnumBitSet<TEnum, 0, GetEnumItemsCount<TEnum>()>; - - TGeneratedEnumBitSet() - : TParent() - { - } - + +/* For Enums with GENERATE_ENUM_SERIALIZATION_WITH_HEADER */ +template <typename TEnum> +class TGeneratedEnumBitSet : public TEnumBitSet<TEnum, 0, GetEnumItemsCount<TEnum>()> { +public: + using TParent = TEnumBitSet<TEnum, 0, GetEnumItemsCount<TEnum>()>; + + TGeneratedEnumBitSet() + : TParent() + { + } + explicit TGeneratedEnumBitSet(const TParent& p) - : TParent(p) - { - } + : TParent(p) + { + } explicit TGeneratedEnumBitSet(TEnum c1) : TParent(c1) @@ -512,4 +512,4 @@ public: : TParent(c1, c2, r...) { } -}; +}; diff --git a/library/cpp/object_factory/object_factory.h b/library/cpp/object_factory/object_factory.h index 96cc11bcfd..e77d0c28fc 100644 --- a/library/cpp/object_factory/object_factory.h +++ b/library/cpp/object_factory/object_factory.h @@ -54,20 +54,20 @@ namespace NObjectFactory { public: template <class TDerivedProduct> - void Register(const TKey& key, IFactoryObjectCreator<TProduct, TArgs...>* creator) { + void Register(const TKey& key, IFactoryObjectCreator<TProduct, TArgs...>* creator) { if (!creator) - ythrow yexception() << "Please specify non-null creator for " << key; + ythrow yexception() << "Please specify non-null creator for " << key; TWriteGuard guard(CreatorsLock); if (!Creators.insert(typename ICreators::value_type(key, creator)).second) ythrow yexception() << "Product with key " << key << " already registered"; } - template <class TDerivedProduct> - void Register(const TKey& key) { - Register<TDerivedProduct>(key, new TFactoryObjectCreator<TProduct, TDerivedProduct, TArgs...>); - } - + template <class TDerivedProduct> + void Register(const TKey& key) { + Register<TDerivedProduct>(key, new TFactoryObjectCreator<TProduct, TDerivedProduct, TArgs...>); + } + void GetKeys(TSet<TKey>& keys) const { TReadGuard guard(CreatorsLock); keys.clear(); @@ -165,14 +165,14 @@ namespace NObjectFactory { template <class Product> class TRegistrator { public: - TRegistrator(const TKey& key, IFactoryObjectCreator<TProduct, void>* creator) { + TRegistrator(const TKey& key, IFactoryObjectCreator<TProduct, void>* creator) { Singleton<TObjectFactory<TProduct, TKey>>()->template Register<Product>(key, creator); } - TRegistrator(const TKey& key) { - Singleton<TObjectFactory<TProduct, TKey>>()->template Register<Product>(key); - } - + TRegistrator(const TKey& key) { + Singleton<TObjectFactory<TProduct, TKey>>()->template Register<Product>(key); + } + TRegistrator() : TRegistrator(Product::GetTypeName()) { @@ -221,14 +221,14 @@ namespace NObjectFactory { template <class Product> class TRegistrator { public: - TRegistrator(const TKey& key, IFactoryObjectCreator<TProduct, TArgs...>* creator) { + TRegistrator(const TKey& key, IFactoryObjectCreator<TProduct, TArgs...>* creator) { Singleton<TParametrizedObjectFactory<TProduct, TKey, TArgs...>>()->template Register<Product>(key, creator); } - TRegistrator(const TKey& key) { - Singleton<TParametrizedObjectFactory<TProduct, TKey, TArgs...>>()->template Register<Product>(key); - } - + TRegistrator(const TKey& key) { + Singleton<TParametrizedObjectFactory<TProduct, TKey, TArgs...>>()->template Register<Product>(key); + } + TRegistrator() : TRegistrator(Product::GetTypeName()) { diff --git a/library/cpp/object_factory/object_factory_ut.cpp b/library/cpp/object_factory/object_factory_ut.cpp index 06fb0739ff..5bb2b75bf2 100644 --- a/library/cpp/object_factory/object_factory_ut.cpp +++ b/library/cpp/object_factory/object_factory_ut.cpp @@ -123,25 +123,25 @@ using TMoveableOnly2Factory = TParametrizedObjectFactory<IMoveableOnlyInterface, static TMoveableOnly2Factory::TRegistrator<TMoveableOnly2> MoveableOnly2Reg("move2"); -class TDirectOrderDifferentSignature : public TDirectOrder { -public: - TDirectOrderDifferentSignature(const TString& provider, TArgument& argument) : - TDirectOrder(provider, 0.01f, argument) - { - } - -}; - -struct TDirectOrderDSCreator: public IFactoryObjectCreator<ICommonInterface, const TString&, float, TArgument&> { - ICommonInterface* Create(const TString& provider, float factor, TArgument& argument) const override { - Y_UNUSED(factor); - return new TDirectOrderDifferentSignature(provider, argument); - } -}; - - -static TTestFactory::TRegistrator<TDirectOrderDifferentSignature> DirectDs("direct_ds", new TDirectOrderDSCreator); - +class TDirectOrderDifferentSignature : public TDirectOrder { +public: + TDirectOrderDifferentSignature(const TString& provider, TArgument& argument) : + TDirectOrder(provider, 0.01f, argument) + { + } + +}; + +struct TDirectOrderDSCreator: public IFactoryObjectCreator<ICommonInterface, const TString&, float, TArgument&> { + ICommonInterface* Create(const TString& provider, float factor, TArgument& argument) const override { + Y_UNUSED(factor); + return new TDirectOrderDifferentSignature(provider, argument); + } +}; + + +static TTestFactory::TRegistrator<TDirectOrderDifferentSignature> DirectDs("direct_ds", new TDirectOrderDSCreator); + Y_UNIT_TEST_SUITE(TestObjectFactory) { Y_UNIT_TEST(TestParametrized) { TArgument directArg{"Name", nullptr}; @@ -177,13 +177,13 @@ Y_UNIT_TEST_SUITE(TestObjectFactory) { UNIT_ASSERT(moveableOnly2->GetValue() == "value2"); } - - Y_UNIT_TEST(TestDifferentSignature) { - TArgument directArg{"Name", nullptr}; - THolder<ICommonInterface> directDs(TTestFactory::Construct("direct_ds", "prov", 0.42, directArg)); - - UNIT_ASSERT(!!directDs); - - UNIT_ASSERT_EQUAL(directDs->GetValue(), "prov0.01Name"); - } + + Y_UNIT_TEST(TestDifferentSignature) { + TArgument directArg{"Name", nullptr}; + THolder<ICommonInterface> directDs(TTestFactory::Construct("direct_ds", "prov", 0.42, directArg)); + + UNIT_ASSERT(!!directDs); + + UNIT_ASSERT_EQUAL(directDs->GetValue(), "prov0.01Name"); + } } 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 |