style: clang-tidy auto fixes (#1730)

Fixes #1729. Please review and commit clang-tidy fixes.

Co-authored-by: kuznetsss <15742918+kuznetsss@users.noreply.github.com>

src/data/CassandraBackend.hpp

@@ -575,7 +575,7 @@ public:
 
         auto mptObjects = doFetchLedgerObjects(mptKeys, ledgerSequence, yield);
 
-        auto it = std::remove_if(mptObjects.begin(), mptObjects.end(), [](Blob const& mpt) { return mpt.size() == 0; });
+        auto it = std::remove_if(mptObjects.begin(), mptObjects.end(), [](Blob const& mpt) { return mpt.empty(); });
 
         mptObjects.erase(it, mptObjects.end());
 
@@ -948,7 +948,8 @@ public:
     writeMPTHolders(std::vector<MPTHolderData> const& data) override
     {
         std::vector<Statement> statements;
-        for (auto [mptId, holder] : data)
+        statements.reserve(data.size());
+for (auto [mptId, holder] : data)
             statements.push_back(schema_->insertMPTHolder.bind(std::move(mptId), std::move(holder)));
 
         executor_.write(std::move(statements));

src/etl/MPTHelpers.cpp

@@ -17,16 +17,21 @@
 */
 //==============================================================================
 
-#include "data/BackendInterface.hpp"
 #include "data/DBHelpers.hpp"
-#include "data/Types.hpp"
 
-#include <fmt/core.h>
 #include <ripple/protocol/STBase.h>
 #include <ripple/protocol/STTx.h>
 #include <ripple/protocol/TxMeta.h>
+#include <xrpl/basics/base_uint.h>
+#include <xrpl/protocol/LedgerFormats.h>
+#include <xrpl/protocol/SField.h>
+#include <xrpl/protocol/STLedgerEntry.h>
+#include <xrpl/protocol/STObject.h>
+#include <xrpl/protocol/TER.h>
+#include <xrpl/protocol/TxFormats.h>
 
-#include <vector>
+#include <optional>
+#include <string>
 
 namespace etl {
 

src/rpc/RPCHelpers.cpp

@@ -81,6 +81,7 @@
 
 #include <algorithm>
 #include <array>
+#include <cassert>
 #include <chrono>
 #include <cstddef>
 #include <cstdint>

src/rpc/handlers/MPTHolders.cpp

@@ -24,17 +24,18 @@
 #include "rpc/RPCHelpers.hpp"
 #include "rpc/common/Types.hpp"
 
+#include <boost/json/array.hpp>
 #include <boost/json/conversion.hpp>
 #include <boost/json/object.hpp>
 #include <boost/json/value.hpp>
 #include <ripple/basics/base_uint.h>
 #include <ripple/basics/strHex.h>
 #include <ripple/protocol/AccountID.h>
-#include <ripple/protocol/ErrorCodes.h>
 #include <ripple/protocol/Indexes.h>
 #include <ripple/protocol/LedgerHeader.h>
 #include <ripple/protocol/jss.h>
-#include <ripple/protocol/nft.h>
+#include <xrpl/protocol/SField.h>
+#include <xrpl/protocol/STLedgerEntry.h>
 
 #include <optional>
 #include <string>
@@ -73,7 +74,7 @@ MPTHoldersHandler::process(MPTHoldersHandler::Input input, Context const& ctx) c
     output.limit = limit;
     output.ledgerIndex = lgrInfo.seq;
 
-    boost::json::array mpts;
+    boost::json::array const mpts;
     for (auto const& mpt : dbResponse.mptokens) {
         ripple::STLedgerEntry const sle{ripple::SerialIter{mpt.data(), mpt.size()}, keylet::mptIssuance(mptID).key};
         boost::json::object mptJson;

tests/common/util/TestObject.cpp

@@ -1135,7 +1135,7 @@ CreateMPTokenObject(std::string_view accountId, ripple::uint192 issuanceID, std:
     mptoken.setFieldH256(ripple::sfPreviousTxnID, ripple::uint256{});
     mptoken.setFieldU32(ripple::sfPreviousTxnLgrSeq, 0);
 
-    if (mptAmount)
+    if (mptAmount != 0u)
         mptoken.setFieldU64(ripple::sfMPTAmount, mptAmount);
 
     return mptoken;

tests/unit/rpc/handlers/LedgerDataTests.cpp

@@ -33,6 +33,7 @@
 #include <gtest/gtest.h>
 #include <xrpl/basics/base_uint.h>
 #include <xrpl/protocol/AccountID.h>
+#include <xrpl/protocol/Indexes.h>
 
 #include <cstdint>
 #include <optional>

tests/unit/rpc/handlers/MPTHoldersTests.cpp

@@ -33,6 +33,7 @@
 #include <ripple/basics/base_uint.h>
 #include <ripple/protocol/Indexes.h>
 #include <ripple/protocol/LedgerHeader.h>
+#include <xrpl/basics/strHex.h>
 
 #include <functional>
 #include <optional>