Further document the initial routing table send.
authorMatt Corallo <git@bluematt.me>
Wed, 12 Feb 2020 18:40:29 +0000 (13:40 -0500)
committerMatt Corallo <git@bluematt.me>
Fri, 6 Mar 2020 01:59:43 +0000 (20:59 -0500)
As requested by Arik at https://github.com/rust-bitcoin/rust-lightning/pull/435#discussion_r378093821

lightning/src/ln/msgs.rs

index e5903f7d603e947ebc5eb9440a1fb9adf3531cba..1ad798fd4bfaf2417d20fdeb4030dd876d7dd77c 100644 (file)
@@ -600,10 +600,11 @@ pub trait RoutingMessageHandler : Send + Sync {
        fn handle_htlc_fail_channel_update(&self, update: &HTLCFailChannelUpdate);
        /// Gets a subset of the channel announcements and updates required to dump our routing table
        /// to a remote node, starting at the short_channel_id indicated by starting_point and
-       /// including batch_amount entries.
+       /// including the batch_amount entries immediately higher in numerical value than starting_point.
        fn get_next_channel_announcements(&self, starting_point: u64, batch_amount: u8) -> Vec<(ChannelAnnouncement, ChannelUpdate, ChannelUpdate)>;
        /// Gets a subset of the node announcements required to dump our routing table to a remote node,
-       /// starting at the node *after* the provided publickey and including batch_amount entries.
+       /// starting at the node *after* the provided publickey and including batch_amount entries
+       /// immediately higher (as defined by <PublicKey as Ord>::cmp) than starting_point.
        /// If None is provided for starting_point, we start at the first node.
        fn get_next_node_announcements(&self, starting_point: Option<&PublicKey>, batch_amount: u8) -> Vec<NodeAnnouncement>;
        /// Returns whether a full sync should be requested from a peer.