docs: fix spelling in comments (#6002)

src/libxrpl/basics/BasicConfig.cpp

@@ -49,7 +49,7 @@ Section::append(std::vector<std::string> const& lines)
     // <key> '=' <value>
     static boost::regex const re1(
         "^"                        // start of line
-        "(?:\\s*)"                 // whitespace (optonal)
+        "(?:\\s*)"                 // whitespace (optional)
         "([a-zA-Z][_a-zA-Z0-9]*)"  // <key>
         "(?:\\s*)"                 // whitespace (optional)
         "(?:=)"                    // '='

src/test/app/NFToken_test.cpp

@@ -7273,7 +7273,7 @@ class NFTokenBaseUtil_test : public beast::unit_test::suite
                         mintAndCreateSellOffer(env, alice, XRP(0));
 
                     // Bob can accept the offer because the new NFT is stored in
-                    // an existing NFTokenPage so no new reserve is requried
+                    // an existing NFTokenPage so no new reserve is required
                     env(token::acceptSellOffer(bob, sellOfferIndex));
                     env.close();
                 }

src/test/app/PayChan_test.cpp

@@ -2150,7 +2150,7 @@ struct PayChan_test : public beast::unit_test::suite
                 // Owner closes, will close after settleDelay
                 env(claim(alice, chan), txflags(tfClose));
                 env.close();
-                // settle delay hasn't ellapsed. Channels should exist.
+                // settle delay hasn't elapsed. Channels should exist.
                 BEAST_EXPECT(channelExists(*env.current(), chan));
                 auto const closeTime = env.current()->info().parentCloseTime;
                 auto const minExpiration = closeTime + settleDelay;

src/test/csf/Peer.h

@@ -423,7 +423,7 @@ struct Peer
         {
             minDuration = std::min(minDuration, link.data.delay);
 
-            // Send a messsage to neighbors to find the ledger
+            // Send a message to neighbors to find the ledger
             net.send(
                 this, link.target, [to = link.target, from = this, ledgerID]() {
                     if (auto it = to->ledgers.find(ledgerID);

src/test/csf/collectors.h

@@ -47,7 +47,7 @@ namespace csf {
 /** Group of collectors.
 
     Presents a group of collectors as a single collector which process an event
-    by calling each collector sequentially. This is analagous to CollectorRefs
+    by calling each collector sequentially. This is analogous to CollectorRefs
     in CollectorRef.h, but does *not* erase the type information of the combined
     collectors.
  */

src/test/csf/events.h

@@ -32,7 +32,7 @@ namespace test {
 namespace csf {
 
 // Events are emitted by peers at a variety of points during the simulation.
-// Each event is emitted by a particlar peer at a particular time. Collectors
+// Each event is emitted by a particular peer at a particular time. Collectors
 // process these events, perhaps calculating statistics or storing events to
 // a log for post-processing.
 //

src/test/csf/ledgers.h

@@ -54,7 +54,7 @@ namespace csf {
 
     Ledgers are immutable value types. All ledgers with the same sequence
     number, transactions, close time, etc. will have the same ledger ID. The
-    LedgerOracle class below manges ID assignments for a simulation and is the
+    LedgerOracle class below manages ID assignments for a simulation and is the
     only way to close and create a new ledger. Since the parent ledger ID is
     part of type, this also means ledgers with distinct histories will have
     distinct ids, even if they have the same set of transactions, sequence

src/test/nodestore/TestBase.h

@@ -101,7 +101,7 @@ public:
                     case 3:
                         return hotUNKNOWN;
                 }
-                // will never happen, but make static analysys tool happy.
+                // will never happen, but make static analysis tool happy.
                 return hotUNKNOWN;
             }();
 

src/test/nodestore/import_test.cpp

@@ -255,7 +255,7 @@ parse_args(std::string const& s)
     // <key> '=' <value>
     static boost::regex const re1(
         "^"                        // start of line
-        "(?:\\s*)"                 // whitespace (optonal)
+        "(?:\\s*)"                 // whitespace (optional)
         "([a-zA-Z][_a-zA-Z0-9]*)"  // <key>
         "(?:\\s*)"                 // whitespace (optional)
         "(?:=)"                    // '='

src/test/overlay/tx_reduce_relay_test.cpp

@@ -264,7 +264,7 @@ private:
         // (20+0.25*(60-20)-5=25), queue the rest, skip counts towards relayed
         // (60-25-5=30)
         testRelay("skip", true, 60, 0, 20, 25, 25, 30, skip);
-        // relay to minPeers + disabled + 25% of (nPeers - minPeers - disalbed)
+        // relay to minPeers + disabled + 25% of (nPeers - minPeers - disabled)
         // (20+10+0.25*(70-20-10)=40), queue the rest (30)
         testRelay("disabled", true, 70, 10, 20, 25, 40, 30);
         // relay to minPeers + disabled-not-in-skip + 25% of (nPeers - minPeers

src/test/rpc/NoRippleCheck_test.cpp

@@ -271,7 +271,7 @@ class NoRippleCheckLimits_test : public beast::unit_test::suite
         auto checkBalance = [&env]() {
             // this is endpoint drop prevention. Non admin ports will drop
             // requests if they are coming too fast, so we manipulate the
-            // resource manager here to reset the enpoint balance (for
+            // resource manager here to reset the endpoint balance (for
             // localhost) if we get too close to the drop limit. It would
             // be better if we could add this functionality to Env somehow
             // or otherwise disable endpoint charging for certain test