Generalize feature methods to work in any context

Refactoring the features module allowed for making code specific to
certain contexts generalizable. Specifically, KNOWN_FEATURE_MASK and
UNKNOWN_FEATURE_MASK are defined on Context instead of hardcoded in each
method specialization. Thus, such methods are no longer required.

Author: jkczyz

lightning/src/ln/features.rs

@@ -274,33 +274,6 @@ impl InitFeatures {
 	}
 }
 
-impl ChannelFeatures {
-	/// Takes the flags that we know how to interpret in an init-context features that are also
-	/// relevant in a channel-context features and creates a channel-context features from them.
-	pub(crate) fn with_known_relevant_init_flags(_init_ctx: &InitFeatures) -> Self {
-		// There are currently no channel flags defined that we understand.
-		Self { flags: Vec::new(), mark: PhantomData, }
-	}
-}
-
-impl NodeFeatures {
-	/// Takes the flags that we know how to interpret in an init-context features that are also
-	/// relevant in a node-context features and creates a node-context features from them.
-	/// Be sure to blank out features that are unknown to us.
-	pub(crate) fn with_known_relevant_init_flags(init_ctx: &InitFeatures) -> Self {
-		use ln::features::sealed::Context;
-		let byte_count = sealed::NodeContext::KNOWN_FEATURE_MASK.len();
-
-		let mut flags = Vec::new();
-		for (i, feature_byte) in init_ctx.flags.iter().enumerate() {
-			if i < byte_count {
-				flags.push(feature_byte & sealed::NodeContext::KNOWN_FEATURE_MASK[i]);
-			}
-		}
-		Self { flags, mark: PhantomData, }
-	}
-}
-
 impl<T: sealed::Context> Features<T> {
 	/// Create a blank Features with no features set
 	pub fn empty() -> Features<T> {
@@ -318,6 +291,20 @@ impl<T: sealed::Context> Features<T> {
 		}
 	}
 
+	/// Takes the flags that we know how to interpret in an init-context features that are also
+	/// relevant in a node-context features and creates a node-context features from them.
+	/// Be sure to blank out features that are unknown to us.
+	pub(crate) fn with_known_relevant_init_flags(init_ctx: &InitFeatures) -> Self {
+		let byte_count = T::KNOWN_FEATURE_MASK.len();
+		let mut flags = Vec::new();
+		for (i, feature_byte) in init_ctx.flags.iter().enumerate() {
+			if i < byte_count {
+				flags.push(feature_byte & T::KNOWN_FEATURE_MASK[i]);
+			}
+		}
+		Self { flags, mark: PhantomData, }
+	}
+
 	#[cfg(test)]
 	/// Create a Features given a set of flags, in LE.
 	pub fn from_le_bytes(flags: Vec<u8>) -> Features<T> {
@@ -334,15 +321,13 @@ impl<T: sealed::Context> Features<T> {
 	}
 
 	pub(crate) fn requires_unknown_bits(&self) -> bool {
-		use ln::features::sealed::Context;
-		let byte_count = sealed::InitContext::UNKNOWN_FEATURE_MASK.len();
-
 		// Bitwise AND-ing with all even bits set except for known features will select unknown
 		// required features.
+		let byte_count = T::UNKNOWN_FEATURE_MASK.len();
 		self.flags.iter().enumerate().any(|(i, &byte)| {
 			let required_features = 0b01_01_01_01;
 			let unknown_features = if i < byte_count {
-				sealed::InitContext::UNKNOWN_FEATURE_MASK[i]
+				T::UNKNOWN_FEATURE_MASK[i]
 			} else {
 				0b11_11_11_11
 			};
@@ -351,14 +336,12 @@ impl<T: sealed::Context> Features<T> {
 	}
 
 	pub(crate) fn supports_unknown_bits(&self) -> bool {
-		use ln::features::sealed::Context;
-		let byte_count = sealed::InitContext::UNKNOWN_FEATURE_MASK.len();
-
 		// Bitwise AND-ing with all even and odd bits set except for known features will select
 		// unknown features.
+		let byte_count = T::UNKNOWN_FEATURE_MASK.len();
 		self.flags.iter().enumerate().any(|(i, &byte)| {
 			let unknown_features = if i < byte_count {
-				sealed::InitContext::UNKNOWN_FEATURE_MASK[i]
+				T::UNKNOWN_FEATURE_MASK[i]
 			} else {
 				0b11_11_11_11
 			};