f minor tweaks

Author: TheBlueMatt

lightning/src/routing/network_graph.rs

@@ -46,6 +46,8 @@ use bitcoin::hashes::hex::ToHex;
 #[cfg(feature = "std")]
 use std::time::{SystemTime, UNIX_EPOCH};
 
+/// We remove stale channel directional info two weeks after the last update, per BOLT 7's
+/// suggestion.
 const STALE_CHANNEL_UPDATE_AGE_LIMIT_SECS: u64 = 60 * 60 * 24 * 14;
 
 /// The maximum number of extra bytes which we do not understand in a gossip message before we will
@@ -1065,7 +1067,7 @@ impl NetworkGraph {
 	///
 	/// While there is no formal requirement that nodes regularly re-broadcast their channel
 	/// updates every two weeks, the non-normative section of BOLT 7 currently suggests that
-	/// pruning occurs for updates which are at least two weeks old, which we implement here.
+	/// pruning occur for updates which are at least two weeks old, which we implement here.
 	///
 	///
 	/// This method is only available with the `std` feature. See
@@ -1081,13 +1083,14 @@ impl NetworkGraph {
 	///
 	/// While there is no formal requirement that nodes regularly re-broadcast their channel
 	/// updates every two weeks, the non-normative section of BOLT 7 currently suggests that
-	/// pruning occurs for updates which are at least two weeks old, which we implement here.
+	/// pruning occur for updates which are at least two weeks old, which we implement here.
 	///
 	/// This function takes the current unix time as an argument. For users with the `std` feature
 	/// enabled, [`NetworkGraph::remove_stale_channels`] may be preferable.
 	pub fn remove_stale_channels_with_time(&self, current_time_unix: u64) {
 		let mut channels = self.channels.write().unwrap();
 		// Time out if we haven't received an update in at least 14 days.
+		if current_time_unix > u32::max_value() { return; } // Remove by 2106
 		let min_time_unix: u32 = (current_time_unix - STALE_CHANNEL_UPDATE_AGE_LIMIT_SECS) as u32;
 		// Sadly BTreeMap::retain was only stabilized in 1.53 so we can't switch to it for some
 		// time.