Added a bunch of style rules

Author: marcomagdy

.clang-tidy

@@ -13,5 +13,29 @@ CheckOptions:
     value:           '1'
   - key:             misc-non-private-member-variables-in-classes.IgnoreClassesWithAllMemberVariablesBeingPublic
     value:           '1'
+  - key:             readability-identifier-naming.ClassCase
+    value:           'lower_case'
+  - key:             readability-identifier-naming.StructCase
+    value:           'lower_case'
+  - key:             readability-identifier-naming.StructCase
+    value:           'lower_case'
+  - key:             readability-identifier-naming.ParameterCase
+    value:           'lower_case'
+  - key:             readability-identifier-naming.PrivateMemberCase
+    value:           'lower_case'
+  - key:             readability-identifier-naming.LocalVariableCase
+    value:           'lower_case'
+  - key:             readability-identifier-naming.TypeAliasCase
+    value:           'lower_case'
+  - key:             readability-identifier-naming.UnionCase
+    value:           'lower_case'
+  - key:             readability-identifier-naming.FunctionCase
+    value:           'lower_case'
+  - key:             readability-identifier-naming.NamespaceCase
+    value:           'lower_case'
+  - key:             readability-identifier-naming.GlobalConstantCase
+    value:           'UPPER_CASE'
+  - key:             readability-identifier-naming.PrivateMemberPrefix
+    value:           'm_'
 
 ...

include/aws/lambda-runtime/outcome.h

@@ -23,13 +23,13 @@ namespace lambda_runtime {
 template <typename TResult, typename TFailure>
 class outcome {
 public:
-    outcome(TResult const& s) : s(s), success(true) {}
+    outcome(TResult const& s) : s(s), m_success(true) {}
 
-    outcome(TFailure const& f) : f(f), success(false) {}
+    outcome(TFailure const& f) : f(f), m_success(false) {}
 
-    outcome(outcome&& other) noexcept : success(other.success)
+    outcome(outcome&& other) noexcept : m_success(other.m_success)
     {
-        if (success) {
+        if (m_success) {
             s = std::move(other.s);
         }
         else {
@@ -39,7 +39,7 @@ class outcome {
 
     ~outcome()
     {
-        if (success) {
+        if (m_success) {
             s.~TResult();
         }
         else {
@@ -49,24 +49,24 @@ class outcome {
 
     TResult const& get_result() const
     {
-        assert(success);
+        assert(m_success);
         return s;
     }
 
     TFailure const& get_failure() const
     {
-        assert(!success);
+        assert(!m_success);
         return f;
     }
 
-    bool is_success() const { return success; }
+    bool is_success() const { return m_success; }
 
 private:
     union {
         TResult s;
         TFailure f;
     };
-    bool success;
+    bool m_success;
 };
 } // namespace lambda_runtime
 } // namespace aws

src/runtime.cpp

@@ -72,7 +72,7 @@ static size_t write_data(char* ptr, size_t size, size_t nmemb, void* userdata)
 // std::isspace has a few edge cases that would trigger UB. In particular, the documentation says:
 // "The behavior is undefined if the value of the input is not representable as unsigned char and is not equal to EOF."
 // So, this function does the simple obvious thing instead.
-static inline bool IsSpace(int ch)
+static inline bool is_whiltespace(int ch)
 {
     constexpr int space = 0x20;           // space (0x20, ' ')
     constexpr int form_feed = 0x0c;       // form feed (0x0c, '\f')
@@ -96,9 +96,9 @@ static inline bool IsSpace(int ch)
 static inline std::string trim(std::string s)
 {
     // trim right
-    s.erase(std::find_if(s.rbegin(), s.rend(), [](int ch) { return !IsSpace(ch); }).base(), s.end());
+    s.erase(std::find_if(s.rbegin(), s.rend(), [](int ch) { return !is_whiltespace(ch); }).base(), s.end());
     // trim left
-    s.erase(s.begin(), std::find_if(s.begin(), s.end(), [](int ch) { return !IsSpace(ch); }));
+    s.erase(s.begin(), std::find_if(s.begin(), s.end(), [](int ch) { return !is_whiltespace(ch); }));
     return s;
 }