Fix clang-tidy issue

apache · Jul 3, 2023 · df38cbd · df38cbd
1 parent 487ed9b
commit df38cbd
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/libminifi/src/c2/ControllerSocketProtocol.cpp b/libminifi/src/c2/ControllerSocketProtocol.cpp
@@ -100,7 +100,7 @@ asio::awaitable<void> ControllerSocketProtocol::startAccept() {
 }
 
 asio::awaitable<void> ControllerSocketProtocol::startAcceptSsl(std::shared_ptr<minifi::controllers::SSLContextService> ssl_context_service) {
-  while (true) {
+  while (true) {  // NOLINT(clang-analyzer-core.NullDereference) suppressing asio library linter warning
     auto [accept_error, socket] = co_await acceptor_->async_accept(utils::net::use_nothrow_awaitable);
     if (accept_error) {
       logger_->log_error("Controller socket accept failed with the following message: '%s'", accept_error.message());

diff --git a/libminifi/test/unit/BackTraceTests.cpp b/libminifi/test/unit/BackTraceTests.cpp
@@ -83,7 +83,7 @@ TEST_CASE("BT2", "[TPT2]") {
   std::vector<BackTrace> traces = pool.getTraces();
   for (const auto &trace : traces) {
     std::cerr << "Thread name: " << trace.getName() << std::endl;
-    const auto &trace_strings = trace.getTraces();
+    [[maybe_unused]] const auto &trace_strings = trace.getTraces();
 #ifdef HAS_EXECINFO
     REQUIRE(trace_strings.size() > 2);
     for (const auto& trace_string : trace_strings) {