clang-tidy

Signed-off-by: iceseer <[email protected]>

core/parachain/approval/knowledge.hpp

@@ -11,6 +11,7 @@
 #include "common/visitor.hpp"
 #include "consensus/timeline/types.hpp"
 #include "outcome/outcome.hpp"
+#include "parachain/approval/approval.hpp"
 #include "parachain/approval/state.hpp"
 #include "parachain/types.hpp"
 
@@ -86,7 +87,7 @@ namespace kagome::parachain::approval {
 
     // Generate the knowledge keys for querying if an approval is known by peer.
     static std::pair<MessageSubject, MessageKind> generate_approval_key(
-        const approval::IndirectSignedApprovalVoteV2 &approval) {
+        const IndirectSignedApprovalVoteV2 &approval) {
       return {
           std::make_tuple(approval.payload.payload.block_hash,
                           approval.payload.payload.candidate_indices,

core/scale/encoder/concepts.hpp

@@ -6,11 +6,12 @@
 
 #pragma once
 
+#include <cstdint>
 #include <type_traits>
 
 namespace kagome::scale {
   template <typename F>
-  concept Invocable = std::is_invocable_v<F, const uint8_t *const, size_t>;
+  concept Invocable = std::is_invocable_v<F, const uint8_t *const, std::size_t>;
 
   template <typename T>
   concept IsEnum = std::is_enum_v<std::decay_t<T>>;

core/scale/encoder/primitives.hpp

@@ -10,6 +10,7 @@
 #include <any>
 #include <deque>
 #include <iostream>
+#include <list>
 #include <map>
 #include <optional>
 #include <scale/outcome/outcome_throw.hpp>
@@ -19,6 +20,7 @@
 #include <type_traits>
 #include <vector>
 #include "crypto/ecdsa_types.hpp"
+#include "primitives/math.hpp"
 #include "scale/encoder/concepts.hpp"
 #include "utils/struct_to_tuple.hpp"
 
@@ -52,6 +54,7 @@ namespace kagome::scale {
                         const std::array<T, size> &c);
 
   template <typename T, size_t N>
+  // NOLINTNEXTLINE(cppcoreguidelines-avoid-c-arrays)
   constexpr void encode(const Invocable auto &func, const T (&c)[N]);
 
   template <typename K, typename V>
@@ -134,6 +137,7 @@ namespace kagome::scale {
     kagome::scale::encode(func, args...);
   }
 
+  // NOLINTBEGIN(cppcoreguidelines-pro-bounds-pointer-arithmetic)
   template <typename... Args>
   outcome::result<std::vector<uint8_t>> encode(const Args &...args) {
     std::vector<uint8_t> res;
@@ -185,6 +189,7 @@ namespace kagome::scale {
     }
     return counter;
   }
+  // NOLINTEND(cppcoreguidelines-pro-bounds-pointer-arithmetic)
 
   constexpr void putByte(const Invocable auto &func,
                          const uint8_t *const val,
@@ -206,6 +211,7 @@ namespace kagome::scale {
   }
 
   template <typename T, size_t N>
+  // NOLINTNEXTLINE(cppcoreguidelines-avoid-c-arrays)
   constexpr void encode(const Invocable auto &func, const T (&c)[N]) {
     using E = std::decay_t<T>;
     if constexpr (std::is_integral_v<E> && sizeof(E) == 1u) {
@@ -472,15 +478,12 @@ namespace kagome::scale {
 
   void encode(const Invocable auto &func, const crypto::EcdsaSignature &data) {
     kagome::scale::encode(
-        func,
-        static_cast<const crypto::EcdsaSignature::Base &>(data));
+        func, static_cast<const crypto::EcdsaSignature::Base &>(data));
   }
 
   void encode(const Invocable auto &func, const crypto::EcdsaPublicKey &data) {
     kagome::scale::encode(
-        func,
-        static_cast<
-            const crypto::EcdsaPublicKey::Base &>(data));
+        func, static_cast<const crypto::EcdsaPublicKey::Base &>(data));
   }
 
 }  // namespace kagome::scale

core/scale/kagome_scale.hpp

@@ -241,7 +241,8 @@ namespace kagome::scale {
   }
 
   constexpr void encode(const Invocable auto &func, const network::Roles &c) {
-    kagome::scale::encode(func, c.value);
+    kagome::scale::encode(
+        func, c.value);  // NOLINT(cppcoreguidelines-pro-type-union-access)
   }
 
   constexpr void encode(const Invocable auto &func,