Remove clang-tidy checks in source code for testing

VarunNagaraju · VarunNagaraju · commit a87706757cb2 · 2025-06-03T15:38:29.000+05:30
diff --git a/libs/mysql/binlog/event/compression/buffer/managed_buffer.h b/libs/mysql/binlog/event/compression/buffer/managed_buffer.h
@@ -124,11 +124,7 @@ class Managed_buffer : public buffer::Rw_buffer<Char_tp> {
   // Nolint: clang-tidy does not recognize that m_owns_default_buffer
   // is initialized, despite it is initialized in the targed
   // constructor.
-  // NOLINTBEGIN(cppcoreguidelines-pro-type-member-init)
-  explicit Managed_buffer(
-      const Memory_resource_t &memory_resource = Memory_resource_t())
-      : Managed_buffer(Size_t(0), memory_resource) {}
-  // NOLINTEND(cppcoreguidelines-pro-type-member-init)
+  explicit Managed_buffer(const Memory_resource_t &memory_resource = Memory_resource_t()) : Managed_buffer(Size_t(0), memory_resource) {}
 
   /// Construct a new object that owns a default buffer.
   ///
diff --git a/libs/mysql/binlog/event/compression/payload_event_buffer_istream.cpp b/libs/mysql/binlog/event/compression/payload_event_buffer_istream.cpp
@@ -144,17 +144,10 @@ void Payload_event_buffer_istream::update_buffer() {
 #ifndef NDEBUG
     // "nolint": clang-tidy reports an unnecessary warning since 'if'
     // and 'else' branches may compile to the same. Suppressing that.
-    // NOLINTBEGIN(bugprone-branch-clone)
     if (m_managed_buffer_ptr.use_count() == 0)
       BAPI_LOG("info", "Allocating managed buffer for the first time.");
     else
-      BAPI_LOG("info",
-               "Allocating new managed buffer. "
-               "The previous one cannot be reused since there are "
-                   << m_managed_buffer_ptr.use_count()
-                   << ">1 shared pointer references to "
-                      "it.");
-      // NOLINTEND(bugprone-branch-clone)
+      BAPI_LOG("info", "Allocating new managed buffer. The previous one cannot be reused since there are " << m_managed_buffer_ptr.use_count() << ">1 shared pointer references to it.");
 #endif
     try {
       auto allocator = Allocator_t<Managed_buffer_t>(m_memory_resource);
diff --git a/libs/mysql/binlog/event/compression/payload_event_buffer_istream.h b/libs/mysql/binlog/event/compression/payload_event_buffer_istream.h
@@ -96,16 +96,8 @@ class Payload_event_buffer_istream {
   /// @param memory_resource @c Memory_resource object used to
   /// allocate memory.
   template <class String_char_t>
-  Payload_event_buffer_istream(
-      const String_char_t *compressed_buffer, Size_t compressed_buffer_size,
-      type compression_algorithm, Size_t default_buffer_size = 0,
-      const Memory_resource_t &memory_resource = Memory_resource_t())
-      : m_memory_resource(memory_resource),
-        m_compressed_buffer(
-            reinterpret_cast<const Char_t *>(compressed_buffer)),
-        m_compressed_buffer_size(compressed_buffer_size),
-        m_compression_algorithm(compression_algorithm),
-        m_default_buffer_size(default_buffer_size) {
+  Payload_event_buffer_istream(const String_char_t *compressed_buffer, Size_t compressed_buffer_size, type compression_algorithm, Size_t default_buffer_size = 0, const Memory_resource_t &memory_resource = Memory_resource_t())
+      : m_memory_resource(memory_resource), m_compressed_buffer(reinterpret_cast<const Char_t *>(compressed_buffer)), m_compressed_buffer_size(compressed_buffer_size), m_compression_algorithm(compression_algorithm), m_default_buffer_size(default_buffer_size) {
     initialize();
   }
 
@@ -130,7 +122,6 @@ class Payload_event_buffer_istream {
   // Nolint: clang-tidy does not recognize that m_default_buffer_size
   // is initialized, despite it is initialized in the targed
   // constructor.
-  // NOLINTBEGIN(cppcoreguidelines-pro-type-member-init)
   template <class String_char_t>
   Payload_event_buffer_istream(
       const std::basic_string<String_char_t> &compressed_data,
@@ -139,7 +130,6 @@ class Payload_event_buffer_istream {
       : Payload_event_buffer_istream(
             compressed_data.data(), compressed_data.size(),
             compression_algorithm, default_buffer_size, memory_resource) {}
-  // NOLINTEND(cppcoreguidelines-pro-type-member-init)
 
   /// Construct the stream from a (non-owned) Payload Event.
   ///
@@ -346,7 +336,7 @@ class Payload_event_buffer_istream {
   /// Grow calculator for the Managed_buffer.
   Grow_calculator_t m_grow_calculator;
   /// Default buffer size for the Managed_buffer.
-  Size_t m_default_buffer_size;
+  Size_t  m_default_buffer_size;
   /// Shared pointer to Managed_buffer that holds the output.  This
   /// will be shared with API clients.  Therefore, API clients can use
   /// the returned buffer as long as they like.  The next time this
diff --git a/sql/raii/thread_stage_guard.h b/sql/raii/thread_stage_guard.h
@@ -111,9 +111,9 @@ class Thread_stage_guard {
 /// @param new_stage The new stage.  `thd` will use this stage until
 /// the end of the scope where the macro is invoked.  At that point,
 /// the stage is reverted to what it was before invoking this macro.
-#define NAMED_THD_STAGE_GUARD(name, thd, new_stage)  \
-  raii::Thread_stage_guard name {                    \
-    (thd), (new_stage), __func__, __FILE__, __LINE__ \
+#define  NAMED_THD_STAGE_GUARD(name, thd, new_stage)  \
+  raii::Thread_stage_guard name {                   \
+    (thd),  (new_stage), __func__, __FILE__, __LINE__ \
   }
 
 /// Set the thread stage for the given thread, and make it restore the
@@ -127,8 +127,6 @@ class Thread_stage_guard {
 #define THD_STAGE_GUARD(thd, new_stage) \
   NAMED_THD_STAGE_GUARD(_thread_stage_guard_##new_stage, (thd), (new_stage))
 
-// NOLINTEND(cppcoreguidelines-macro-usage)
-
 }  // namespace raii
 
 #endif  /// THREAD_STAGE_GUARD_H_INCLUDED
diff --git a/unittest/gunit/binlogevents/managed_buffer_sequence-t.cc b/unittest/gunit/binlogevents/managed_buffer_sequence-t.cc
@@ -80,7 +80,6 @@ using Difference_t = mysql::binlog::event::compression::buffer::
 // make the program stop with assertion.
 [[maybe_unused]] static int n_assertions = 0;
 static bool _shall_stop_after_assertion = false;
-// NOLINTBEGIN(cppcoreguidelines-macro-usage)
 #define ASSERTION_TAIL                                                   \
   << debug_output(fileline) << (_shall_stop_after_assertion = true, ""), \
       assert(!_shall_stop_after_assertion)
@@ -94,7 +93,6 @@ static bool _shall_stop_after_assertion = false;
     ASSERT_NE(v1, v2) ASSERTION_TAIL; \
     ++n_assertions;                   \
   } while (0)
-// NOLINTEND(cppcoreguidelines-macro-usage)
 
 // Requirements:
 //
@@ -251,10 +249,8 @@ class Grow_tester {
                  Size_t extra_container_capacity, Size_t position,
                  Size_t capacity, Size_t max_capacity,
                  Size_t requested_capacity, Size_t requested_position) {
-    // NOLINTBEGIN(cppcoreguidelines-macro-usage)
 #define CHECK_SIZES(POSITION, CAPACITY) \
   check_sizes(FILELINE(), debug_output, mbs, buffer_size, POSITION, CAPACITY)
-    // NOLINTEND(cppcoreguidelines-macro-usage)
 
     // This does the following:
     //
diff --git a/unittest/gunit/binlogevents/payload_event_buffer_istream-t.cc b/unittest/gunit/binlogevents/payload_event_buffer_istream-t.cc
@@ -364,7 +364,7 @@ class PayloadEventBufferStreamTest {
       // clang-tidy warns about it. But this is an allocator so it is
       // appropriate to use malloc and therefore we suppress the check.
       // NOLINTNEXTLINE(cppcoreguidelines-no-malloc)
-      return std::malloc(n);
+      return  std::malloc(n);
     };
     Memory_resource_t failing_memory_resource(failing_allocator, std::free);
     auto debug_func = [&] {
