f some feedback

Author: TheBlueMatt

CHANGELOG.md

@@ -4,23 +4,25 @@
  * Support for relaying onion messages has been added via a new
    `OnionMessenger` struct when passed as the `OnionMessageHandler` to a
    `PeerManager`. Pre-encoded onion messages can also be sent and received
-   (#1503, #1640, #1650, #1652, #1688).
+   (#1503, #1650, #1652, #1688).
  * Rate-limiting of outbound gossip syncs has been rewritten to utilize less
    buffering inside LDK. The new rate-limiting is also used for onion messages
    to avoid delaying other messages (#1604. #1660, #1683).
  * Rather than spawning a full OS thread, `lightning-background-processor` has
    a new `process_events_async` method which takes the place of a
-   `BackgroundProcessor` for those using Rust async (#1657).
+   `BackgroundProcessor` for those using Rust's async (#1657).
  * The `Filter::register_output` return value has been removed, as it was
-   very difficult to correctly implemented (i.e. without blocking) (#1663).
+   very difficult to correctly implement (i.e., without blocking). Users
+   previously using it should instead pass dependent transactions in via new
+   `chain::Confirm::transactions_confirmed` calls (#1663).
  * `ChannelHandshakeConfig::their_channel_reserve_proportional_millionths` has
    been added to allow configuring counterparty reserve values (#1619).
  * The `rust-bitcoin` dependency has been updated 0.29 (#1658).
  * The `bitcoin_hashes` dependency has been updated 0.11 (#1677).
  * `ChannelManager::broadcast_node_announcement` has been moved to
    `PeerManager` (#1699).
  * `{Init,Node}Features` sent to peers/broadcasted are now fetched via the
-   various `*MessageHandler` traits, rather than hard-coded (#1701, XXX OMs).
+   various `*MessageHandler` traits, rather than hard-coded (#1701, #1688).
  * `Event::PaymentPathFailed::rejected_by_dest` has been renamed
    `payment_failed_permanently` (#1702).
  * `Invoice` now derives the std `Hash` trait (#1575).
@@ -38,10 +40,10 @@
    output via a new `Balance::CounterpartyRevokedOutputClaimable` (#1495).
  * Inbound HTLCs for which we do not (yet) have a preimage are now included in
    `get_claimable_balance` via a `Balance::MaybePreimageClaimableHTLC` (#1673).
- * Probes which fail prior to being sent over their first hop are correctly
+ * Probes that fail prior to being sent over their first hop are correctly
    failed with a `Event::ProbeFailed` rather than a `PaymentPathFailed` (#1704).
  * Pending `Event::HTLCHandlingFailed`s are no longer lost on restart (#1700).
- * HTLCs which fail prior to being sent over their first hop are now marked as
+ * HTLCs that fail prior to being sent over their first hop are now marked as
    retryable via `!PaymentPathFailed::payment_failed_permanently` (#1702).
  * Dust HTLCs are now considered failed in the payment tracking logic after the
    commitment transaction confirms, allowing retry on restart (#1691).