Restoring authorship annotation for Alexey Salmin <alexey.salmin@gmail.com>. Commit 1 of 2.

library/cpp/balloc/balloc.cpp

@@ -76,7 +76,7 @@ namespace NBalloc {
     static void Y_FORCE_INLINE Enable() {
         tls.Mode = ToBeEnabled;
     }
-
+ 
     static bool Y_FORCE_INLINE IsDisabled() {
         return tls.Mode == Disabled;
     }
@@ -273,11 +273,11 @@ extern "C" void* realloc(void* oldPtr, size_t newSize) {
 }
 #endif
 
-// Only for testing purposes. Never use in production.
+// Only for testing purposes. Never use in production. 
 extern "C" bool IsOwnedByBalloc(void* ptr) {
-    return NBalloc::IsOwnedByBalloc(ptr);
-}
-
+    return NBalloc::IsOwnedByBalloc(ptr); 
+} 
+ 
 extern "C" bool BallocDisabled() {
     return NBalloc::IsDisabled();
 }
@@ -286,10 +286,10 @@ extern "C" void DisableBalloc() {
     NBalloc::Disable();
 }
 
-extern "C" void EnableBalloc() {
-    NBalloc::Enable();
-}
-
+extern "C" void EnableBalloc() { 
+    NBalloc::Enable(); 
+} 
+ 
 extern "C" void* memalign(size_t alignment, size_t size) {
     void* ptr;
     int res = posix_memalign(&ptr, alignment, size);

library/cpp/balloc/malloc-info.cpp

@@ -5,18 +5,18 @@
 using namespace NMalloc;
 
 extern "C" void DisableBalloc();
-extern "C" void EnableBalloc();
+extern "C" void EnableBalloc(); 
 extern "C" bool BallocDisabled();
 
 namespace {
     bool SetAllocParam(const char* name, const char* value) {
         if (strcmp(name, "disable") == 0) {
             if (value == nullptr || strcmp(value, "false") != 0) {
-                // all values other than "false" are considred to be "true" for compatibility
-                DisableBalloc();
-            } else {
-                EnableBalloc();
-            }
+                // all values other than "false" are considred to be "true" for compatibility 
+                DisableBalloc(); 
+            } else { 
+                EnableBalloc(); 
+            } 
             return true;
         }
         return false;

library/cpp/balloc/optional/operators.h

@@ -10,7 +10,7 @@ inline bool BallocEnabled() {
 inline void ThreadDisableBalloc() {
     ::NMalloc::MallocInfo().SetParam("disable", "true");
 }
-
-inline void ThreadEnableBalloc() {
-    ::NMalloc::MallocInfo().SetParam("disable", "false");
-}
+ 
+inline void ThreadEnableBalloc() { 
+    ::NMalloc::MallocInfo().SetParam("disable", "false"); 
+} 

library/cpp/cgiparam/cgiparam.h

@@ -88,13 +88,13 @@ public:
     void InsertEscaped(const TStringBuf name, const TStringBuf value);
 
 #if !defined(__GLIBCXX__)
-    template <typename TName, typename TValue>
-    inline void InsertUnescaped(TName&& name, TValue&& value) {
+    template <typename TName, typename TValue> 
+    inline void InsertUnescaped(TName&& name, TValue&& value) { 
         // TStringBuf use as TName or TValue is C++17 actually.
         // There is no pair constructor available in C++14 when required type
         // is not implicitly constructible from given type.
         // But libc++ pair allows this with C++14.
-        emplace(std::forward<TName>(name), std::forward<TValue>(value));
+        emplace(std::forward<TName>(name), std::forward<TValue>(value)); 
     }
 #else
     template <typename TName, typename TValue>
@@ -103,19 +103,19 @@ public:
     }
 #endif
 
-    // replace all values for a given key with new values
-    template <typename TIter>
-    void ReplaceUnescaped(const TStringBuf key, TIter valuesBegin, const TIter valuesEnd);
-
-    void ReplaceUnescaped(const TStringBuf key, std::initializer_list<TStringBuf> values) {
-        ReplaceUnescaped(key, values.begin(), values.end());
-    }
+    // replace all values for a given key with new values 
+    template <typename TIter> 
+    void ReplaceUnescaped(const TStringBuf key, TIter valuesBegin, const TIter valuesEnd); 
 
-    void ReplaceUnescaped(const TStringBuf key, const TStringBuf value) {
+    void ReplaceUnescaped(const TStringBuf key, std::initializer_list<TStringBuf> values) { 
+        ReplaceUnescaped(key, values.begin(), values.end()); 
+    } 
+ 
+    void ReplaceUnescaped(const TStringBuf key, const TStringBuf value) { 
         ReplaceUnescaped(key, {value});
-    }
-
-    // join multiple values into a single one using a separator
+    } 
+ 
+    // join multiple values into a single one using a separator 
     // if val is a [possibly empty] non-NULL string, append it as well
     void JoinUnescaped(const TStringBuf key, char sep, TStringBuf val = TStringBuf());
 
@@ -132,29 +132,29 @@ public:
         return it->second.data();
     }
 };
-
-template <typename TIter>
-void TCgiParameters::ReplaceUnescaped(const TStringBuf key, TIter valuesBegin, const TIter valuesEnd) {
-    const auto oldRange = equal_range(key);
-    auto current = oldRange.first;
-
-    // reuse as many existing nodes as possible (probably none)
+ 
+template <typename TIter> 
+void TCgiParameters::ReplaceUnescaped(const TStringBuf key, TIter valuesBegin, const TIter valuesEnd) { 
+    const auto oldRange = equal_range(key); 
+    auto current = oldRange.first; 
+ 
+    // reuse as many existing nodes as possible (probably none) 
     for (; valuesBegin != valuesEnd && current != oldRange.second; ++valuesBegin, ++current) {
-        current->second = *valuesBegin;
-    }
-
-    // if there were more nodes than we need to insert then erase remaining ones
+        current->second = *valuesBegin; 
+    } 
+ 
+    // if there were more nodes than we need to insert then erase remaining ones 
     for (; current != oldRange.second; erase(current++)) {
-    }
-
-    // if there were less nodes than we need to insert then emplace the rest of the range
+    } 
+ 
+    // if there were less nodes than we need to insert then emplace the rest of the range 
     if (valuesBegin != valuesEnd) {
         const TString keyStr = TString(key);
         for (; valuesBegin != valuesEnd; ++valuesBegin) {
             emplace_hint(oldRange.second, keyStr, TString(*valuesBegin));
         }
-    }
-}
+    } 
+} 
 
 /** TQuickCgiParam is a faster non-editable version of TCgiParameters.
  * Care should be taken when replacing:

library/cpp/cgiparam/cgiparam_ut.cpp

@@ -132,56 +132,56 @@ Y_UNIT_TEST_SUITE(TCgiParametersTest) {
 
     Y_UNIT_TEST(TestReplaceUnescaped1) {
         TCgiParameters c;
-        c.ScanAddAll("many_keys=1&aaa=1&many_keys=2&bbb&many_keys=3");
-        c.ReplaceUnescaped("many_keys", "new_value");
-
-        UNIT_ASSERT_VALUES_EQUAL(c.Print(), "aaa=1&bbb=&many_keys=new_value");
-    }
-
+        c.ScanAddAll("many_keys=1&aaa=1&many_keys=2&bbb&many_keys=3"); 
+        c.ReplaceUnescaped("many_keys", "new_value"); 
+ 
+        UNIT_ASSERT_VALUES_EQUAL(c.Print(), "aaa=1&bbb=&many_keys=new_value"); 
+    } 
+ 
     Y_UNIT_TEST(TestReplaceUnescaped2) {
-        TCgiParameters c;
-        c.ScanAddAll("par=1&only_one=1&par=2&bbb&par=3");
-        c.ReplaceUnescaped("only_one", "new_value");
-
-        UNIT_ASSERT_VALUES_EQUAL(c.Print(), "bbb=&only_one=new_value&par=1&par=2&par=3");
-    }
-
+        TCgiParameters c; 
+        c.ScanAddAll("par=1&only_one=1&par=2&bbb&par=3"); 
+        c.ReplaceUnescaped("only_one", "new_value"); 
+ 
+        UNIT_ASSERT_VALUES_EQUAL(c.Print(), "bbb=&only_one=new_value&par=1&par=2&par=3"); 
+    } 
+ 
     Y_UNIT_TEST(TestReplaceUnescaped3) {
-        TCgiParameters c;
+        TCgiParameters c; 
         c.ScanAddAll("par=1&aaa=1&par=2&bbb&par=3");
-        c.ReplaceUnescaped("no_such_key", "new_value");
-
-        UNIT_ASSERT_VALUES_EQUAL(c.Print(), "aaa=1&bbb=&no_such_key=new_value&par=1&par=2&par=3");
-    }
+        c.ReplaceUnescaped("no_such_key", "new_value"); 
 
+        UNIT_ASSERT_VALUES_EQUAL(c.Print(), "aaa=1&bbb=&no_such_key=new_value&par=1&par=2&par=3"); 
+    } 
+ 
     Y_UNIT_TEST(TestReplaceUnescapedRange1) {
-        TCgiParameters c;
-        c.ScanAddAll("par=1&aaa=1&par=2&bbb&par=3");
-        c.ReplaceUnescaped("par", {"x", "y", "z"}); // 3 old values, 3 new values
-
-        UNIT_ASSERT_VALUES_EQUAL(c.Print(), "aaa=1&bbb=&par=x&par=y&par=z");
-    }
-
+        TCgiParameters c; 
+        c.ScanAddAll("par=1&aaa=1&par=2&bbb&par=3"); 
+        c.ReplaceUnescaped("par", {"x", "y", "z"}); // 3 old values, 3 new values 
+ 
+        UNIT_ASSERT_VALUES_EQUAL(c.Print(), "aaa=1&bbb=&par=x&par=y&par=z"); 
+    } 
+ 
     Y_UNIT_TEST(TestReplaceUnescapedRange2) {
-        TCgiParameters c;
-        c.ScanAddAll("par=1&aaa=1&par=2&bbb");
-        c.ReplaceUnescaped("par", {"x", "y", "z"}); // 2 old values, 3 new values
-
-        UNIT_ASSERT_VALUES_EQUAL(c.Print(), "aaa=1&bbb=&par=x&par=y&par=z");
-    }
-
+        TCgiParameters c; 
+        c.ScanAddAll("par=1&aaa=1&par=2&bbb"); 
+        c.ReplaceUnescaped("par", {"x", "y", "z"}); // 2 old values, 3 new values 
+ 
+        UNIT_ASSERT_VALUES_EQUAL(c.Print(), "aaa=1&bbb=&par=x&par=y&par=z"); 
+    } 
+ 
     Y_UNIT_TEST(TestReplaceUnescapedRange3) {
-        TCgiParameters c;
-        c.ScanAddAll("par=1&aaa=1&par=2&bbb&par=3");
-        c.ReplaceUnescaped("par", {"x", "y"}); // 3 old values, 2 new values
-
-        UNIT_ASSERT_VALUES_EQUAL(c.Print(), "aaa=1&bbb=&par=x&par=y");
-    }
-
+        TCgiParameters c; 
+        c.ScanAddAll("par=1&aaa=1&par=2&bbb&par=3"); 
+        c.ReplaceUnescaped("par", {"x", "y"}); // 3 old values, 2 new values 
+ 
+        UNIT_ASSERT_VALUES_EQUAL(c.Print(), "aaa=1&bbb=&par=x&par=y"); 
+    } 
+ 
     Y_UNIT_TEST(TestNumOfValues) {
-        TCgiParameters c;
-        c.ScanAddAll("par=1&aaa=1&par=2&bbb&par=3");
-
+        TCgiParameters c; 
+        c.ScanAddAll("par=1&aaa=1&par=2&bbb&par=3"); 
+ 
         UNIT_ASSERT_VALUES_EQUAL(c.NumOfValues("par"), 3u);
     }
 

library/cpp/containers/sorted_vector/sorted_vector.h

@@ -323,7 +323,7 @@ namespace NSorted {
         using TBase::erase;
     };
 
-    // The simplified map (a.k.a TFlatMap, flat_map), which is implemented by the sorted-vector.
+    // The simplified map (a.k.a TFlatMap, flat_map), which is implemented by the sorted-vector. 
     // This structure has the side-effect: if you keep a reference to an existing element
     // and then inserts a new one, the existing reference can be broken (due to reallocation).
     // Please keep this in mind when using this structure.
@@ -397,7 +397,7 @@ namespace NSorted {
         }
     };
 
-    // The simplified set (a.k.a TFlatSet, flat_set), which is implemented by the sorted-vector.
+    // The simplified set (a.k.a TFlatSet, flat_set), which is implemented by the sorted-vector. 
     // This structure has the same side-effect as TSimpleMap.
     // The value type must have TValueType(TKeyType) constructor in order to use [] operator
     template <typename TValueType, typename TKeyType = TValueType, class TKeyExtractor = TIdentity,

library/cpp/deprecated/mapped_file/mapped_file.cpp

@@ -11,7 +11,7 @@ TMappedFile::TMappedFile(TFileMap* map, const char* dbgName) {
     if (Hi32(len) != 0 && sizeof(size_t) <= sizeof(ui32))
         ythrow yexception() << "File '" << dbgName << "' mapping error: " << len << " too large";
 
-    Map_->Map(0, static_cast<size_t>(len));
+    Map_->Map(0, static_cast<size_t>(len)); 
 }
 
 TMappedFile::TMappedFile(const TFile& file, TFileMap::EOpenMode om, const char* dbgName)

library/cpp/deprecated/mapped_file/ut/mapped_file_ut.cpp

@@ -1,8 +1,8 @@
 #include <library/cpp/deprecated/mapped_file/mapped_file.h>
 #include <library/cpp/testing/unittest/registar.h>
 
-#include <util/system/fs.h>
-
+#include <util/system/fs.h> 
+ 
 Y_UNIT_TEST_SUITE(TMappedFileTest) {
     static const char* FileName_("./mappped_file");
     Y_UNIT_TEST(TestFileMapEmpty) {
@@ -13,6 +13,6 @@ Y_UNIT_TEST_SUITE(TMappedFileTest) {
         map.init(FileName_);
         map.getData(0);
 
-        NFs::Remove(FileName_);
+        NFs::Remove(FileName_); 
     }
 };

library/cpp/getopt/last_getopt_demo/demo.cpp

@@ -92,7 +92,7 @@ protected:
                      {"OPTIONS", "describe the communication options for the target resource"},
                      {"TRACE", "perform a message loop-back test"},
                      {"PATCH", "apply partial modifications to the specified resource"}}));
-
+ 
         opts.AddLongOption('U', "user-agent")
             .RequiredArgument("agent-string")
             .DefaultValue("LastGetoptDemo/1.0.0")
@@ -202,8 +202,8 @@ protected:
                 << colors.Cyan()
                 << "    $ last_getopt_demo -b -U 'Wget/1.0.0' https://wordpress.org/latest.zip"
                 << colors.Reset());
-    }
-
+    } 
+ 
     int DoRun(NLastGetopt::TOptsParseResult&& parsedOptions) override {
         using namespace NColorizer;
 

library/cpp/getopt/print.cpp

@@ -31,7 +31,7 @@ namespace NLastGetoptPrivate {
     TString& ShortVersionString();
 
     struct TInit {
-        TInit() {
+        TInit() { 
             VersionString() = InitVersionString();
             ShortVersionString() = InitShortVersionString();
         }