Fix clang-tidy warnings

src/api/common/src/binance-common-api.cpp

@@ -10,9 +10,7 @@
 #include "cachedresult.hpp"
 #include "cct_json.hpp"
 #include "cct_log.hpp"
-#include "cct_string.hpp"
 #include "curlhandle.hpp"
-#include "curlpostdata.hpp"
 #include "currencycode.hpp"
 #include "currencycodeset.hpp"
 #include "currencyexchange.hpp"

src/api/common/src/exchangeprivateapi.cpp

@@ -1,7 +1,6 @@
 #include "exchangeprivateapi.hpp"
 
 #include <algorithm>
-#include <chrono>
 #include <cstdint>
 #include <iterator>
 #include <limits>
@@ -391,7 +390,7 @@ MarketVector GetPossibleMarketsForDustThresholds(const BalancePortfolio &balance
     }
 
    private:
-    int weight(Market mk) const {
+    [[nodiscard]] int weight(Market mk) const {
       // not present is equivalent to a weight of 0
       const auto it = _penaltyPerMarketMap.find(mk);
       return it == _penaltyPerMarketMap.end() ? 0 : it->second;

src/api/exchanges/src/binanceprivateapi.cpp

@@ -16,6 +16,7 @@
 #include "apiquerytypeenum.hpp"
 #include "balanceoptions.hpp"
 #include "balanceportfolio.hpp"
+#include "binance-common-api.hpp"
 #include "binancepublicapi.hpp"
 #include "cachedresult.hpp"
 #include "cct_exception.hpp"
@@ -357,8 +358,7 @@ void FillOrders(const OrdersConstraints& ordersConstraints, const json& ordersAr
       orderVector.emplace_back(std::move(id), matchedVolume, price, placedTime, matchedTime, side);
     } else {
       // Note: below ugly template lambda can be replaced with 'static_assert(false);' in C++23
-      []<bool flag = false>() { static_assert(flag, "no match"); }
-      ();
+      []<bool flag = false>() { static_assert(flag, "no match"); }();
     }
   }
   std::ranges::sort(orderVector);
@@ -820,8 +820,7 @@ ReceivedWithdrawInfo BinancePrivate::queryWithdrawDelivery(const InitiatedWithdr
   json& depositEl = depositStatus[closestDepositPos];
   const RecentDeposit recentDeposit = recentDepositFromJsonEl(depositEl);
 
-  return ReceivedWithdrawInfo(std::move(depositEl["id"].get_ref<string&>()), recentDeposit.amount(),
-                              recentDeposit.timePoint());
+  return {std::move(depositEl["id"].get_ref<string&>()), recentDeposit.amount(), recentDeposit.timePoint()};
 }
 
 }  // namespace cct::api

src/api/exchanges/src/binancepublicapi.cpp

@@ -35,7 +35,6 @@
 #include "market-vector.hpp"
 #include "market.hpp"
 #include "marketorderbook.hpp"
-#include "monetary-amount-vector.hpp"
 #include "monetaryamount.hpp"
 #include "monetaryamountbycurrencyset.hpp"
 #include "order-book-line.hpp"

src/api/exchanges/src/huobiprivateapi.cpp

@@ -14,6 +14,7 @@
 #include "balanceportfolio.hpp"
 #include "base64.hpp"
 #include "cachedresult.hpp"
+#include "cct_cctype.hpp"
 #include "cct_exception.hpp"
 #include "cct_json.hpp"
 #include "cct_log.hpp"
@@ -38,6 +39,7 @@
 #include "orderid.hpp"
 #include "ordersconstraints.hpp"
 #include "permanentcurloptions.hpp"
+#include "query-retry-policy.hpp"
 #include "request-retry.hpp"
 #include "ssl_sha.hpp"
 #include "stringhelpers.hpp"

src/api/exchanges/src/krakenprivateapi.cpp

@@ -42,6 +42,7 @@
 #include "orderid.hpp"
 #include "ordersconstraints.hpp"
 #include "permanentcurloptions.hpp"
+#include "query-retry-policy.hpp"
 #include "request-retry.hpp"
 #include "ssl_sha.hpp"
 #include "stringhelpers.hpp"

src/api/exchanges/src/krakenpublicapi.cpp

@@ -29,7 +29,6 @@
 #include "market.hpp"
 #include "marketorderbook.hpp"
 #include "monetaryamount.hpp"
-#include "monetaryamountbycurrencyset.hpp"
 #include "order-book-line.hpp"
 #include "permanentcurloptions.hpp"
 #include "public-trade-vector.hpp"
@@ -319,9 +318,9 @@ Market GetKrakenMarketOrDefault(const CurrencyExchangeFlatSet& currencies, Marke
   const auto krakenBaseIt = currencies.find(mk.base());
   const auto krakenQuoteIt = currencies.find(mk.quote());
   if (krakenBaseIt != currencies.end() && krakenQuoteIt != currencies.end()) {
-    return Market(krakenBaseIt->altCode(), krakenQuoteIt->altCode());
+    return {krakenBaseIt->altCode(), krakenQuoteIt->altCode()};
   }
-  return Market{};
+  return {};
 }
 }  // namespace
 

src/objects/src/exchangeconfig.cpp

@@ -13,6 +13,7 @@
 #include "cct_string.hpp"
 #include "currencycodevector.hpp"
 #include "durationstring.hpp"
+#include "http-config.hpp"
 #include "monetaryamount.hpp"
 #include "monetaryamountbycurrencyset.hpp"
 #include "permanentcurloptions.hpp"

src/objects/src/marketorderbook.cpp

@@ -4,6 +4,7 @@
 #include <cassert>
 #include <cmath>
 #include <cstdint>
+#include <iterator>
 #include <limits>
 #include <numeric>
 #include <optional>

src/tech/test/stringhelpers_test.cpp

@@ -4,6 +4,7 @@
 
 #include <cstdint>
 #include <limits>
+#include <string_view>
 
 #include "cct_string.hpp"