f suggest not fc'ing

Author: TheBlueMatt

lightning/src/chain/mod.rs

@@ -179,13 +179,13 @@ pub trait Confirm {
 ///
 /// Note that there is no error variant - any failure to persist a [`ChannelMonitor`] should be
 /// retried indefinitely, the node shut down (as if we cannot update stored data we can't do much
-/// of anything useful), or the channel force-close using
-/// [`ChannelManager::force_close_broadcasting_latest_txn`].
+/// of anything useful).
 ///
-/// Note that in the last case, force-closing with the latest [`ChannelMonitorUpdate`] applied may
-/// result in a transaction which can only be spent by the latest [`ChannelMonitor`]. Thus, if the
-/// latest [`ChannelMonitor`] is not durably persisted anywhere, naively calling
-/// [`ChannelManager::force_close_broadcasting_latest_txn`] *may result in loss of funds*!
+/// Note that channels should generally not be force-closed after a persistence failure.
+/// Force-closing with the latest [`ChannelMonitorUpdate`] applied may result in a transaction
+/// being broadcast which can only be spent by the latest [`ChannelMonitor`]! Thus, if the
+/// latest [`ChannelMonitor`] is not durably persisted anywhere and exists only in memory, naively
+/// calling [`ChannelManager::force_close_broadcasting_latest_txn`] *may result in loss of funds*!
 #[derive(Clone, Copy, Debug, PartialEq, Eq)]
 pub enum ChannelMonitorUpdateStatus {
 	/// The update has been durably persisted and all copies of the relevant [`ChannelMonitor`]