diff --git a/contrib/macdeploy/gen-sdk b/contrib/macdeploy/gen-sdk index 86a6262b5ce..f0bbabf81f9 100755 --- a/contrib/macdeploy/gen-sdk +++ b/contrib/macdeploy/gen-sdk @@ -54,7 +54,7 @@ def run(): out_sdktgz_path = pathlib.Path("./{}.tar.gz".format(out_name)) def tarfp_add_with_base_change(tarfp, dir_to_add, alt_base_dir): - """Add all files in dir_to_add to tarfp, but prepent alt_base_dir to the files' + """Add all files in dir_to_add to tarfp, but prepend alt_base_dir to the files' names e.g. if the only file under /root/bazdir is /root/bazdir/qux, invoking: diff --git a/contrib/signet/getcoins.py b/contrib/signet/getcoins.py index 19751ae2695..75430e5b1aa 100755 --- a/contrib/signet/getcoins.py +++ b/contrib/signet/getcoins.py @@ -152,7 +152,7 @@ if res: elif res.status_code == 404: print('The specified faucet URL does not exist. Please check for any server issues/typo.') elif res.status_code == 429: - print('The script does not allow for repeated transactions as the global faucet is rate-limitied to 1 request/IP/day. You can access the faucet website to get more coins manually') + print('The script does not allow for repeated transactions as the global faucet is rate-limited to 1 request/IP/day. You can access the faucet website to get more coins manually') else: print(f'Returned Error Code {res.status_code}\n{res.text}\n') print('Please check the provided arguments for their validity and/or any possible typo.') diff --git a/doc/productivity.md b/doc/productivity.md index 181c697d730..022f89259ba 100644 --- a/doc/productivity.md +++ b/doc/productivity.md @@ -194,7 +194,7 @@ You can do: git range-diff master previously-reviewed-head new-head ``` -Note that `git range-diff` also work for rebases: +Note that `git range-diff` also works for rebases: ``` P1--P2--P3--P4--P5 <-- previously-reviewed-head diff --git a/doc/release-notes-27826.md b/doc/release-notes-27826.md index 7f8a9ee4ded..ef1dbe33f7e 100644 --- a/doc/release-notes-27826.md +++ b/doc/release-notes-27826.md @@ -1,4 +1,4 @@ -- Logs now include which peer sent us a header. Additionaly there are fewer +- Logs now include which peer sent us a header. Additionally there are fewer redundant header log messages. A side-effect of this change is that for some untypical cases new headers aren't logged anymore, e.g. a direct `BLOCK` message with a previously unknown header and `submitheader` RPC. (#27826) diff --git a/doc/zmq.md b/doc/zmq.md index 9fea2b41706..cb15444a486 100644 --- a/doc/zmq.md +++ b/doc/zmq.md @@ -127,7 +127,7 @@ serialized transaction. Notifies about all transactions, both when they are added to mempool or when a new block arrives. This means a transaction could be published multiple times: first when it enters -mempool and then again in each block that includes it. The body part of the mesage is the +mempool and then again in each block that includes it. The body part of the message is the 32-byte transaction hash in reversed byte order. #### rawblock diff --git a/src/common/netif.cpp b/src/common/netif.cpp index 51a7522dbf5..ab5a82f3151 100644 --- a/src/common/netif.cpp +++ b/src/common/netif.cpp @@ -309,7 +309,7 @@ std::vector GetLocalAddresses() if (status != NO_ERROR) { // This includes ERROR_NO_DATA if there are no addresses and thus there's not even one PIP_ADAPTER_ADDRESSES // record in the returned structure. - LogPrintLevel(BCLog::NET, BCLog::Level::Error, "Could not get local adapter addreses: %s\n", NetworkErrorString(status)); + LogPrintLevel(BCLog::NET, BCLog::Level::Error, "Could not get local adapter addresses: %s\n", NetworkErrorString(status)); return addresses; } diff --git a/src/test/fuzz/muhash.cpp b/src/test/fuzz/muhash.cpp index 318a203699c..922c6ff8d4b 100644 --- a/src/test/fuzz/muhash.cpp +++ b/src/test/fuzz/muhash.cpp @@ -198,7 +198,7 @@ FUZZ_TARGET(muhash) muhash3.Finalize(out2); }, [&] { - // Test that dividing a MuHash by itself brings it back to it's initial state + // Test that dividing a MuHash by itself brings it back to its initial state // See note about clang + self-assignment in test/uint256_tests.cpp #if defined(__clang__) @@ -216,7 +216,7 @@ FUZZ_TARGET(muhash) out2 = initial_state_hash; }, [&] { - // Test that removing all added elements brings the object back to it's initial state + // Test that removing all added elements brings the object back to its initial state muhash.Remove(data); muhash.Remove(data2); muhash.Finalize(out); diff --git a/src/test/fuzz/p2p_headers_presync.cpp b/src/test/fuzz/p2p_headers_presync.cpp index b31b74ee4fe..a2c3a0f3e62 100644 --- a/src/test/fuzz/p2p_headers_presync.cpp +++ b/src/test/fuzz/p2p_headers_presync.cpp @@ -32,9 +32,9 @@ public: PeerManager::Options peerman_opts; node::ApplyArgsManOptions(*m_node.args, peerman_opts); peerman_opts.max_headers_result = FUZZ_MAX_HEADERS_RESULTS; - // The peerman's rng is a global that is re-used, so it will be re-used + // The peerman's rng is a global that is reused, so it will be reused // and may cause non-determinism between runs. This may even influence - // the global RNG, because seeding may be done from the gloabl one. For + // the global RNG, because seeding may be done from the global one. For // now, avoid it influencing the global RNG, and initialize it with a // constant instead. peerman_opts.deterministic_rng = true; diff --git a/src/test/fuzz/policy_estimator_io.cpp b/src/test/fuzz/policy_estimator_io.cpp index 3e7d0933439..02dc5fec867 100644 --- a/src/test/fuzz/policy_estimator_io.cpp +++ b/src/test/fuzz/policy_estimator_io.cpp @@ -27,7 +27,7 @@ FUZZ_TARGET(policy_estimator_io, .init = initialize_policy_estimator_io) FuzzedDataProvider fuzzed_data_provider(buffer.data(), buffer.size()); FuzzedFileProvider fuzzed_file_provider{fuzzed_data_provider}; AutoFile fuzzed_auto_file{fuzzed_file_provider.open()}; - // Re-using block_policy_estimator across runs to avoid costly creation of CBlockPolicyEstimator object. + // Reusing block_policy_estimator across runs to avoid costly creation of CBlockPolicyEstimator object. static CBlockPolicyEstimator block_policy_estimator{FeeestPath(*g_setup->m_node.args), DEFAULT_ACCEPT_STALE_FEE_ESTIMATES}; if (block_policy_estimator.Read(fuzzed_auto_file)) { block_policy_estimator.Write(fuzzed_auto_file); diff --git a/src/test/fuzz/utxo_snapshot.cpp b/src/test/fuzz/utxo_snapshot.cpp index 13f60881926..16196b0e25f 100644 --- a/src/test/fuzz/utxo_snapshot.cpp +++ b/src/test/fuzz/utxo_snapshot.cpp @@ -104,7 +104,7 @@ void utxo_snapshot_fuzz(FuzzBufferType buffer) FuzzedDataProvider fuzzed_data_provider(buffer.data(), buffer.size()); SetMockTime(ConsumeTime(fuzzed_data_provider, /*min=*/1296688602)); // regtest genesis block timestamp auto& setup{*g_setup}; - bool dirty_chainman{false}; // Re-use the global chainman, but reset it when it is dirty + bool dirty_chainman{false}; // Reuse the global chainman, but reset it when it is dirty auto& chainman{*setup.m_node.chainman}; const auto snapshot_path = gArgs.GetDataDirNet() / "fuzzed_snapshot.dat"; diff --git a/src/test/ipc_test.cpp b/src/test/ipc_test.cpp index fb21b3a71d0..dd46883fb09 100644 --- a/src/test/ipc_test.cpp +++ b/src/test/ipc_test.cpp @@ -53,7 +53,7 @@ static std::string TempPath(std::string_view pattern) //! on the object through FooInterface (defined in ipc_test.capnp). void IpcPipeTest() { - // Setup: create FooImplemention object and listen for FooInterface requests + // Setup: create FooImplementation object and listen for FooInterface requests std::promise>> foo_promise; std::function disconnect_client; std::thread thread([&]() { diff --git a/src/test/rbf_tests.cpp b/src/test/rbf_tests.cpp index 671dab56f71..41bfe28c37d 100644 --- a/src/test/rbf_tests.cpp +++ b/src/test/rbf_tests.cpp @@ -563,7 +563,7 @@ BOOST_FIXTURE_TEST_CASE(calc_feerate_diagram_rbf, TestChain100Setup) BOOST_CHECK(replace_two_chunks_single_cluster->second == expected_new_chunks); } - // You can have more than two direct conflicts if the there are multiple affected clusters, all of size 2 or less + // You can have more than two direct conflicts if there are multiple affected clusters, all of size 2 or less const auto conflict_1 = make_tx(/*inputs=*/ {m_coinbase_txns[2]}, /*output_values=*/ {10 * COIN}); AddToMempool(pool, entry.Fee(low_fee).FromTx(conflict_1)); const auto conflict_1_entry = pool.GetIter(conflict_1->GetHash()).value(); diff --git a/src/test/txreconciliation_tests.cpp b/src/test/txreconciliation_tests.cpp index e258e3353da..75e6d1c729e 100644 --- a/src/test/txreconciliation_tests.cpp +++ b/src/test/txreconciliation_tests.cpp @@ -51,7 +51,7 @@ BOOST_AUTO_TEST_CASE(ForgetPeerTest) TxReconciliationTracker tracker(TXRECONCILIATION_VERSION); NodeId peer_id0 = 0; - // Removing peer after pre-registring works and does not let to register the peer. + // Removing peer after pre-registering works and does not let to register the peer. tracker.PreRegisterPeer(peer_id0); tracker.ForgetPeer(peer_id0); BOOST_CHECK_EQUAL(tracker.RegisterPeer(peer_id0, true, 1, 1), ReconciliationRegisterResult::NOT_FOUND); diff --git a/src/txgraph.cpp b/src/txgraph.cpp index 0a65d269e53..33afca68b0b 100644 --- a/src/txgraph.cpp +++ b/src/txgraph.cpp @@ -324,7 +324,7 @@ private: ChunkIndex m_main_chunkindex; /** Number of index-observing objects in existence (BlockBuilderImpls). */ size_t m_main_chunkindex_observers{0}; - /** Cache of discarded ChunkIndex node handles to re-use, avoiding additional allocation. */ + /** Cache of discarded ChunkIndex node handles to reuse, avoiding additional allocation. */ std::vector m_main_chunkindex_discarded; /** A Locator that describes whether, where, and in which Cluster an Entry appears. diff --git a/src/wallet/dump.cpp b/src/wallet/dump.cpp index fc904de973b..6b193ad72f3 100644 --- a/src/wallet/dump.cpp +++ b/src/wallet/dump.cpp @@ -61,7 +61,7 @@ bool DumpWallet(const ArgsManager& args, WalletDatabase& db, bilingual_str& erro // Write out the file format std::string format = db.Format(); - // BDB files that are opened using BerkeleyRODatabase have it's format as "bdb_ro" + // BDB files that are opened using BerkeleyRODatabase have its format as "bdb_ro" // We want to override that format back to "bdb" if (format == "bdb_ro") { format = "bdb"; diff --git a/src/wallet/scriptpubkeyman.cpp b/src/wallet/scriptpubkeyman.cpp index d2ac171f8d3..6a090c77334 100644 --- a/src/wallet/scriptpubkeyman.cpp +++ b/src/wallet/scriptpubkeyman.cpp @@ -1042,7 +1042,7 @@ bool DescriptorScriptPubKeyMan::TopUpWithDB(WalletBatch& batch, unsigned int siz const CPubKey& pubkey = pk_pair.second; if (m_map_pubkeys.count(pubkey) != 0) { // We don't need to give an error here. - // It doesn't matter which of many valid indexes the pubkey has, we just need an index where we can derive it and it's private key + // It doesn't matter which of many valid indexes the pubkey has, we just need an index where we can derive it and its private key continue; } m_map_pubkeys[pubkey] = i; @@ -1447,7 +1447,7 @@ void DescriptorScriptPubKeyMan::SetCache(const DescriptorCache& cache) const CPubKey& pubkey = pk_pair.second; if (m_map_pubkeys.count(pubkey) != 0) { // We don't need to give an error here. - // It doesn't matter which of many valid indexes the pubkey has, we just need an index where we can derive it and it's private key + // It doesn't matter which of many valid indexes the pubkey has, we just need an index where we can derive it and its private key continue; } m_map_pubkeys[pubkey] = i; diff --git a/src/wallet/spend.cpp b/src/wallet/spend.cpp index a330c31a87a..d96946e7b89 100644 --- a/src/wallet/spend.cpp +++ b/src/wallet/spend.cpp @@ -1090,7 +1090,7 @@ static util::Result CreateTransactionInternal( // Get size of spending the change output int change_spend_size = CalculateMaximumSignedInputSize(change_prototype_txout, &wallet, /*coin_control=*/nullptr); // If the wallet doesn't know how to sign change output, assume p2sh-p2wpkh - // as lower-bound to allow BnB to do it's thing + // as lower-bound to allow BnB to do its thing if (change_spend_size == -1) { coin_selection_params.change_spend_size = DUMMY_NESTED_P2WPKH_INPUT_SIZE; } else { diff --git a/test/functional/mempool_updatefromblock.py b/test/functional/mempool_updatefromblock.py index 49da6835fcc..46b69d67387 100755 --- a/test/functional/mempool_updatefromblock.py +++ b/test/functional/mempool_updatefromblock.py @@ -211,7 +211,7 @@ class MempoolUpdateFromBlockTest(BitcoinTestFramework): # Last tx fits now self.nodes[0].sendrawtransaction(chain[-1]["hex"]) - # Finally, reorg to empty chain kick everything back into mempool + # Finally, reorg to empty chain to kick everything back into mempool # at normal chain limits for block in fork_blocks: self.nodes[0].submitblock(block.serialize().hex()) diff --git a/test/functional/mining_basic.py b/test/functional/mining_basic.py index 9a77ad7a719..89f633cf7f0 100755 --- a/test/functional/mining_basic.py +++ b/test/functional/mining_basic.py @@ -105,7 +105,7 @@ class MiningTest(BitcoinTestFramework): wallet_sigops = MiniWallet(node, mode=MiniWalletMode.RAW_P2PK) self.generate(wallet_sigops, 1, sync_fun=self.no_op) - # Mature with regular coinbases to prevent inteference with other tests + # Mature with regular coinbases to prevent interference with other tests self.generate(self.wallet, 100, sync_fun=self.no_op) # Generate three transactions that must be mined in sequence diff --git a/test/lint/spelling.ignore-words.txt b/test/lint/spelling.ignore-words.txt index ccf2e6964b7..fe08eabefa9 100644 --- a/test/lint/spelling.ignore-words.txt +++ b/test/lint/spelling.ignore-words.txt @@ -1,15 +1,13 @@ afile amountIn -asend +anc blockin bu cachable clen -crypted debbugs deques fo -fpr hashIn hights incomin @@ -18,8 +16,8 @@ lief mor nd nin +ot outIn -re-use requestor ser siz