Fix typos and cleanup in various files #12593

 105 | @@ -106,7 +106,7 @@ std::string ScriptToAsmStr(const CScript& script, const bool fAttemptSighashDeco
 106 |                          const unsigned char chSigHashType = vch.back();
 107 |                          if (mapSigHashTypes.count(chSigHashType)) {
 108 |                              strSigHashDecode = "[" + mapSigHashTypes.find(chSigHashType)->second + "]";
 109 | -                            vch.pop_back(); // remove the sighash type byte. it will be replaced by the decode.
 110 | +                            vch.pop_back(); // Remove the sighash type byte. It will be replaced by the decode.

---

  51 | @@ -52,7 +52,7 @@ public static void testSecKeyVerifyPos() throws AssertFailException{
  52 |      }
  53 |  
  54 |      /**
  55 | -      * This tests secret key verify() for a invalid secretkey
  56 | +      * This tests secret key verify() for an invalid secretkey

---

  34 | @@ -35,7 +35,7 @@ bool ParseInt32(const std::string& str, int32_t *out)
  35 |      errno = 0; // strtol will not set errno if valid
  36 |      long int n = strtol(str.c_str(), &endp, 10);
  37 |      if(out) *out = (int32_t)n;
  38 | -    // Note that strtol returns a *long int*, so even if strtol doesn't report a over/underflow
  39 | +    // Note that strtol returns a *long int*, so even if strtol doesn't report an over/underflow

---

 354 | @@ -355,7 +355,7 @@ def run_test(self):
 355 |          for mode in [True, False]:
 356 |              self.nodes[0].ensure_ascii = mode
 357 |              # unicode check: Basic Multilingual Plane, Supplementary Plane respectively
 358 | -            for s in [u'рыба', u'𝅘𝅥𝅯']:
 359 | +            for s in [u'рыба', u'𝅘𝅥𝅯']:

---

 338 | @@ -339,7 +339,7 @@ bool MarkBlockAsInFlight(NodeId nodeid, const uint256& hash, const CBlockIndex*
 339 |      CNodeState *state = State(nodeid);
 340 |      assert(state != nullptr);
 341 |  
 342 | -    // Short-circuit most stuff in case its from the same node
 343 | +    // Short-circuit most stuff in case it's from the same node

---

 948 | @@ -949,7 +949,7 @@ static bool AcceptToMemoryPoolWorker(const CChainParams& chainparams, CTxMemPool
 949 |  
 950 |          // This transaction should only count for fee estimation if:
 951 |          // - it isn't a BIP 125 replacement transaction (may not be widely supported)
 952 | -        // - it's not being readded during a reorg which bypasses typical mempool fee limits
 953 | +        // - it's not being read during a reorg which bypasses typical mempool fee limits

---

1986 | @@ -1987,7 +1987,7 @@ UniValue listsinceblock(const JSONRPCRequest& request)
1987 |              "  ],\n"
1988 |              "  \"removed\": [\n"
1989 |              "    <structure is the same as \"transactions\" above, only present if include_removed=true>\n"
1990 | -            "    Note: transactions that were readded in the active chain will appear as-is in this array, and may thus have a positive confirmation count.\n"
1991 | +            "    Note: transactions that were read in the active chain will appear as-is in this array, and may thus have a positive confirmation count.\n"

---

 753 | @@ -754,7 +754,7 @@ BOOST_AUTO_TEST_CASE(test_LockDirectory)
 754 |      thr.join();
 755 |      BOOST_CHECK_EQUAL(threadresult, true);
 756 |  #ifndef WIN32
 757 | -    // Try to aquire lock in child process while we're holding it, this should fail.
 758 | +    // Try to acquire lock in child process while we're holding it, this should fail.

---

 764 | @@ -765,7 +765,7 @@ BOOST_AUTO_TEST_CASE(test_LockDirectory)
 765 |      // Probing lock from our side now should succeed, but not hold on to the lock.
 766 |      BOOST_CHECK_EQUAL(LockDirectory(dirname, lockname, true), true);
 767 |  
 768 | -    // Try to acquire the lock in the child process, this should be succesful.
 769 | +    // Try to acquire the lock in the child process, this should be successful.

---

  48 | @@ -49,7 +49,7 @@ def read_dump(file_name, addrs, script_addrs, hd_master_addr_old):
  49 |                      # count key types
  50 |                      for addrObj in addrs:
  51 |                          if addrObj['address'] == addr.split(",")[0] and addrObj['hdkeypath'] == keypath and keytype == "label=":
  52 | -                            # a labled entry in the wallet should contain both a native address
  53 | +                            # a labeled entry in the wallet should contain both a native address

---

 162 | @@ -163,7 +163,7 @@ void test_cache_erase(size_t megabytes)
 163 |      for (uint32_t i = (n_insert / 2); i < n_insert; ++i)
 164 |          set.insert(hashes_insert_copy[i]);
 165 |  
 166 | -    /** elements that we marked erased but that are still there */
 167 | +    /** elements that we marked erased but are still there */

---

4183 | @@ -4184,8 +4184,8 @@ bool CWalletTx::AcceptToMemoryPool(const CAmount& nAbsurdFee, CValidationState&
4184 |      // We must set fInMempool here - while it will be re-set to true by the
4185 |      // entered-mempool callback, if we did not there would be a race where a
4186 |      // user could call sendmoney in a loop and hit spurious out of funds errors
4187 | -    // because we think that the transaction they just generated's change is
4188 | -    // unavailable as we're not yet aware its in mempool.
4189 | +    // because we think that the just generated transaction's change is
4190 | +    // unavailable as we're not yet aware it's in mempool.

---

 344 | @@ -345,7 +345,7 @@ Status Version::Get(const ReadOptions& options,
 345 |  
 346 |    // We can search level-by-level since entries never hop across
 347 |    // levels.  Therefore we are guaranteed that if we find data
 348 | -  // in an smaller level, later levels are irrelevant.
 349 | +  // in a smaller level, later levels are irrelevant.

---