f take the alternative approach

Author: TheBlueMatt

lightning/src/chain/channelmonitor.rs

@@ -968,10 +968,6 @@ impl<Signer: Sign> ChannelMonitor<Signer> {
 	/// For lockorder enforcement purposes, we need to have a single site which constructs the
 	/// `inner` mutex, otherwise cases where we lock two monitors at the same time (eg in our
 	/// PartialEq implementation) we may decide a lockorder violation has occurred.
-	///
-	/// Note that this approach may be somewhat brittle - if this method gets inlined inner mutexes
-	/// may be considered separate again. In that case we may need to move from considering locks
-	/// by the instruction pointer to considering locks by the symbol module and line number.
 	fn construct_monitor(imp: ChannelMonitorImpl<Signer>) -> Self {
 		ChannelMonitor { inner: Mutex::new(imp) }
 	}