Browse Source

Restoring authorship annotation for <chegoryu@yandex-team.ru>. Commit 2 of 2.

chegoryu 3 years ago
parent
commit
767f053568

+ 11 - 11
library/cpp/getopt/small/modchooser.cpp

@@ -206,7 +206,7 @@ int TModChooser::Run(const int argc, const char** argv) const {
 
     if (shiftArgs) {
         TString firstArg;
-        TVector<const char*> nargv(Reserve(argc)); 
+        TVector<const char*> nargv(Reserve(argc));
 
         if (PrintShortCommandInUsage) {
             firstArg = modeIter->second->Name;
@@ -219,26 +219,26 @@ int TModChooser::Run(const int argc, const char** argv) const {
         for (int i = 2; i < argc; ++i) {
             nargv.push_back(argv[i]);
         }
-        // According to the standard, "argv[argc] shall be a null pointer" (5.1.2.2.1). 
-        // http://www.open-std.org/JTC1/SC22/WG14/www/docs/n1336 
-        nargv.push_back(nullptr); 
+        // According to the standard, "argv[argc] shall be a null pointer" (5.1.2.2.1).
+        // http://www.open-std.org/JTC1/SC22/WG14/www/docs/n1336
+        nargv.push_back(nullptr);
 
-        return (*modeIter->second->Main)(nargv.size() - 1, nargv.data()); 
+        return (*modeIter->second->Main)(nargv.size() - 1, nargv.data());
     } else {
         return (*modeIter->second->Main)(argc, argv);
     }
 }
 
 int TModChooser::Run(const TVector<TString>& argv) const {
-    TVector<const char*> nargv(Reserve(argv.size() + 1)); 
+    TVector<const char*> nargv(Reserve(argv.size() + 1));
     for (auto& arg : argv) {
         nargv.push_back(arg.c_str());
     }
-    // According to the standard, "argv[argc] shall be a null pointer" (5.1.2.2.1). 
-    // http://www.open-std.org/JTC1/SC22/WG14/www/docs/n1336 
-    nargv.push_back(nullptr); 
- 
-    return Run(nargv.size() - 1, nargv.data()); 
+    // According to the standard, "argv[argc] shall be a null pointer" (5.1.2.2.1).
+    // http://www.open-std.org/JTC1/SC22/WG14/www/docs/n1336
+    nargv.push_back(nullptr);
+
+    return Run(nargv.size() - 1, nargv.data());
 }
 
 size_t TModChooser::TMode::CalculateFullNameLen() const {

+ 24 - 24
library/cpp/getopt/ut/modchooser_ut.cpp

@@ -4,33 +4,33 @@
 
 #include <util/stream/str.h>
 
-void ValidateArgcArgv(int argc, const char** argv) { 
-    UNIT_ASSERT_EQUAL(argc, 1); 
-    UNIT_ASSERT_EQUAL(argv[argc], nullptr); 
-} 
- 
-int One(int argc, const char** argv) { 
-    ValidateArgcArgv(argc, argv); 
+void ValidateArgcArgv(int argc, const char** argv) {
+    UNIT_ASSERT_EQUAL(argc, 1);
+    UNIT_ASSERT_EQUAL(argv[argc], nullptr);
+}
+
+int One(int argc, const char** argv) {
+    ValidateArgcArgv(argc, argv);
     return 1;
 }
 
-int Two(int argc, const char** argv) { 
-    ValidateArgcArgv(argc, argv); 
+int Two(int argc, const char** argv) {
+    ValidateArgcArgv(argc, argv);
     return 2;
 }
 
-int Three(int argc, const char** argv) { 
-    ValidateArgcArgv(argc, argv); 
+int Three(int argc, const char** argv) {
+    ValidateArgcArgv(argc, argv);
     return 3;
 }
 
-int Four(int argc, const char** argv) { 
-    ValidateArgcArgv(argc, argv); 
+int Four(int argc, const char** argv) {
+    ValidateArgcArgv(argc, argv);
     return 4;
 }
 
-int Five(int argc, const char** argv) { 
-    ValidateArgcArgv(argc, argv); 
+int Five(int argc, const char** argv) {
+    ValidateArgcArgv(argc, argv);
     return 5;
 }
 
@@ -46,25 +46,25 @@ Y_UNIT_TEST_SUITE(TModChooserTest) {
             chooser.AddMode(NAMES[idx], FUNCTIONS[idx], NAMES[idx]);
         }
 
-        // test argc, argv 
+        // test argc, argv
         for (size_t idx = 0; idx < Y_ARRAY_SIZE(NAMES); ++idx) {
             int argc = 2;
-            const char* argv[] = {"UNITTEST", NAMES[idx], nullptr}; 
+            const char* argv[] = {"UNITTEST", NAMES[idx], nullptr};
             UNIT_ASSERT_EQUAL(static_cast<int>(idx) + 1, chooser.Run(argc, argv));
         }
- 
-        // test TVector<TString> argv 
-        for (size_t idx = 0; idx < Y_ARRAY_SIZE(NAMES); ++idx) { 
-            const TVector<TString> argv = {"UNITTEST", NAMES[idx]}; 
-            UNIT_ASSERT_EQUAL(static_cast<int>(idx) + 1, chooser.Run(argv)); 
-        } 
+
+        // test TVector<TString> argv
+        for (size_t idx = 0; idx < Y_ARRAY_SIZE(NAMES); ++idx) {
+            const TVector<TString> argv = {"UNITTEST", NAMES[idx]};
+            UNIT_ASSERT_EQUAL(static_cast<int>(idx) + 1, chooser.Run(argv));
+        }
     }
 
     Y_UNIT_TEST(TestHelpMessage) {
         TModChooser chooser;
 
         int argc = 2;
-        const char* argv[] = {"UNITTEST", "-?", nullptr}; 
+        const char* argv[] = {"UNITTEST", "-?", nullptr};
 
         chooser.Run(argc, argv);
     }

+ 5 - 5
library/cpp/http/io/headers.h

@@ -94,11 +94,11 @@ public:
     /// заголовок на новый.
     void AddOrReplaceHeader(const THttpInputHeader& header);
 
-    template <typename ValueType> 
-    void AddOrReplaceHeader(TString name, const ValueType& value) { 
-        AddOrReplaceHeader(THttpInputHeader(std::move(name), ToString(value))); 
-    } 
- 
+    template <typename ValueType>
+    void AddOrReplaceHeader(TString name, const ValueType& value) {
+        AddOrReplaceHeader(THttpInputHeader(std::move(name), ToString(value)));
+    }
+
     // Проверяет, есть ли такой заголовок
     bool HasHeader(TStringBuf header) const;
 

+ 17 - 17
library/cpp/http/io/headers_ut.cpp

@@ -54,7 +54,7 @@ class THttpHeadersTest: public TTestBase {
 
 private:
     typedef void (*TAddHeaderFunction)(THttpHeaders&, TStringBuf name, TStringBuf value);
-    typedef void (*TAddOrReplaceHeaderFunction)(THttpHeaders&, TStringBuf name, TStringBuf value); 
+    typedef void (*TAddOrReplaceHeaderFunction)(THttpHeaders&, TStringBuf name, TStringBuf value);
 
 public:
     void TestAddOperation1Arg();
@@ -73,16 +73,16 @@ private:
         headers.AddHeader(TString(name), TString(value));
     }
 
-    static void AddOrReplaceHeaderImpl1Arg(THttpHeaders& headers, TStringBuf name, TStringBuf value) { 
-        headers.AddOrReplaceHeader(THttpInputHeader(TString(name), TString(value))); 
-    } 
- 
-    static void AddOrReplaceHeaderImpl2Args(THttpHeaders& headers, TStringBuf name, TStringBuf value) { 
-        headers.AddOrReplaceHeader(TString(name), TString(value)); 
-    } 
- 
+    static void AddOrReplaceHeaderImpl1Arg(THttpHeaders& headers, TStringBuf name, TStringBuf value) {
+        headers.AddOrReplaceHeader(THttpInputHeader(TString(name), TString(value)));
+    }
+
+    static void AddOrReplaceHeaderImpl2Args(THttpHeaders& headers, TStringBuf name, TStringBuf value) {
+        headers.AddOrReplaceHeader(TString(name), TString(value));
+    }
+
     void DoTestAddOperation(TAddHeaderFunction);
-    void DoTestAddOrReplaceOperation(TAddHeaderFunction, TAddOrReplaceHeaderFunction); 
+    void DoTestAddOrReplaceOperation(TAddHeaderFunction, TAddOrReplaceHeaderFunction);
 };
 
 UNIT_TEST_SUITE_REGISTRATION(THttpHeadersTest);
@@ -95,10 +95,10 @@ void THttpHeadersTest::TestAddOperation2Args() {
 }
 
 void THttpHeadersTest::TestAddOrReplaceOperation1Arg() {
-    DoTestAddOrReplaceOperation(AddHeaderImpl1Arg, AddOrReplaceHeaderImpl1Arg); 
+    DoTestAddOrReplaceOperation(AddHeaderImpl1Arg, AddOrReplaceHeaderImpl1Arg);
 }
 void THttpHeadersTest::TestAddOrReplaceOperation2Args() {
-    DoTestAddOrReplaceOperation(AddHeaderImpl2Args, AddOrReplaceHeaderImpl2Args); 
+    DoTestAddOrReplaceOperation(AddHeaderImpl2Args, AddOrReplaceHeaderImpl2Args);
 }
 
 void THttpHeadersTest::DoTestAddOperation(TAddHeaderFunction addHeader) {
@@ -124,19 +124,19 @@ void THttpHeadersTest::DoTestAddOperation(TAddHeaderFunction addHeader) {
 }
 
 // Sorry, but AddOrReplaceHeader replaces only first occurence
-void THttpHeadersTest::DoTestAddOrReplaceOperation(TAddHeaderFunction addHeader, TAddOrReplaceHeaderFunction addOrReplaceHeader) { 
+void THttpHeadersTest::DoTestAddOrReplaceOperation(TAddHeaderFunction addHeader, TAddOrReplaceHeaderFunction addOrReplaceHeader) {
     THttpHeaders h1;
 
     addHeader(h1, "h1", "v1");
 
-    addOrReplaceHeader(h1, "h2", "v1"); 
-    addOrReplaceHeader(h1, "h2", "v2"); 
-    addOrReplaceHeader(h1, "h2", "v3"); 
+    addOrReplaceHeader(h1, "h2", "v1");
+    addOrReplaceHeader(h1, "h2", "v2");
+    addOrReplaceHeader(h1, "h2", "v3");
     addHeader(h1, "h2", "v4");
 
     addHeader(h1, "h3", "v1");
     addHeader(h1, "h3", "v2");
-    addOrReplaceHeader(h1, "h3", "v3"); 
+    addOrReplaceHeader(h1, "h3", "v3");
 
     THeadersExistence h2;
 

+ 1 - 1
library/cpp/protobuf/json/json2proto.cpp

@@ -408,7 +408,7 @@ namespace NProtobufJson {
     void MergeJson2Proto(const TStringBuf& json, google::protobuf::Message& proto, const TJson2ProtoConfig& config) {
         NJson::TJsonReaderConfig jsonCfg;
         jsonCfg.DontValidateUtf8 = true;
-        jsonCfg.AllowComments = config.AllowComments; 
+        jsonCfg.AllowComments = config.AllowComments;
 
         NJson::TJsonValue jsonValue;
         ReadJsonTree(json, &jsonCfg, &jsonValue, /* throwOnError = */ true);

+ 8 - 8
library/cpp/protobuf/json/json2proto.h

@@ -93,11 +93,11 @@ namespace NProtobufJson {
             return *this;
         }
 
-        TSelf& SetAllowComments(bool value) { 
-            AllowComments = value; 
-            return *this; 
-        } 
- 
+        TSelf& SetAllowComments(bool value) {
+            AllowComments = value;
+            return *this;
+        }
+
         TSelf& SetAllowUnknownFields(bool value) {
             AllowUnknownFields = value;
             return *this;
@@ -141,9 +141,9 @@ namespace NProtobufJson {
 
         /// Custom spliter non array value to repeated fields.
         TValueVectorizer ValueVectorizer;
- 
-        /// Allow js-style comments (both // and /**/) 
-        bool AllowComments = false; 
+
+        /// Allow js-style comments (both // and /**/)
+        bool AllowComments = false;
     };
 
     /// @throw yexception

+ 24 - 24
library/cpp/protobuf/json/ut/json2proto_ut.cpp

@@ -1120,28 +1120,28 @@ Y_UNIT_TEST(TestEmptyStringForCastFromString) {
     UNIT_ASSERT_EQUAL("", proto.GetOneString());
 } // TestEmptyStringForCastFromString
 
-Y_UNIT_TEST(TestAllowComments) { 
-    constexpr TStringBuf json = R"( 
-{ 
-    "I32": 4, // comment1 
-/* 
-    comment2 
-    {} 
-    qwer 
-*/ 
-    "I64": 3423 
-} 
- 
-)"; 
- 
-    TJson2ProtoConfig config; 
-    TFlatOptional proto; 
-    UNIT_ASSERT_EXCEPTION_CONTAINS(Json2Proto(json, proto, config), yexception, "Error: Missing a name for object member"); 
- 
-    config.SetAllowComments(true); 
-    UNIT_ASSERT_NO_EXCEPTION(Json2Proto(json, proto, config)); 
-    UNIT_ASSERT_VALUES_EQUAL(proto.GetI32(), 4); 
-    UNIT_ASSERT_VALUES_EQUAL(proto.GetI64(), 3423); 
-} // TestAllowComments 
- 
+Y_UNIT_TEST(TestAllowComments) {
+    constexpr TStringBuf json = R"(
+{
+    "I32": 4, // comment1
+/*
+    comment2
+    {}
+    qwer
+*/
+    "I64": 3423
+}
+
+)";
+
+    TJson2ProtoConfig config;
+    TFlatOptional proto;
+    UNIT_ASSERT_EXCEPTION_CONTAINS(Json2Proto(json, proto, config), yexception, "Error: Missing a name for object member");
+
+    config.SetAllowComments(true);
+    UNIT_ASSERT_NO_EXCEPTION(Json2Proto(json, proto, config));
+    UNIT_ASSERT_VALUES_EQUAL(proto.GetI32(), 4);
+    UNIT_ASSERT_VALUES_EQUAL(proto.GetI64(), 3423);
+} // TestAllowComments
+
 } // TJson2ProtoTest

+ 1 - 1
util/system/info.cpp

@@ -55,7 +55,7 @@ static inline size_t CgroupCpus() {
             return 0;
         }
 
-        return Max<ssize_t>(1, (q + p / 2) / p); 
+        return Max<ssize_t>(1, (q + p / 2) / p);
     } catch (...) {
         return 0;
     }