Do not panic on attempting to force-close a subchannel

bitcoin-test-utils/src/lib.rs

@@ -42,8 +42,7 @@ pub fn from_hex(hex: &str, target: &mut [u8]) -> Result<usize, ()> {
 /// Transforms an hex string to a Vec<u8>.
 /// Panics if the string is not valid hex.
 pub fn str_to_hex(hex_str: &str) -> Vec<u8> {
-    let mut hex = Vec::<u8>::new();
-    hex.resize(hex_str.len() / 2, 0);
+    let mut hex = vec![0; hex_str.len() / 2];
     from_hex(hex_str, &mut hex).unwrap();
     hex
 }

dlc-manager/src/manager.rs

@@ -129,7 +129,13 @@ macro_rules! check_for_timed_out_channels {
                 let is_timed_out = timeout < $manager.time.unix_time_now();
                 if is_timed_out {
                     let sub_channel = if channel.is_sub_channel() {
-                        unimplemented!();
+                        log::info!(
+                            "Skipping force-closure of subchannel {}: not supported",
+                            bitcoin::hashes::hex::ToHex::to_hex(&channel.channel_id[..])
+                        );
+                        continue;
+
+                        // TODO: Implement subchannel force closing
                         // let s = get_sub_channel_in_state!(
                         //     $manager,
                         //     channel.channel_id,

dlc-messages/src/message_handler.rs

@@ -188,9 +188,7 @@ impl CustomMessageHandler for MessageHandler {
         org: &PublicKey,
     ) -> Result<(), LightningError> {
         let mut segment_readers = self.segment_readers.lock().unwrap();
-        let segment_reader = segment_readers
-            .entry(*org)
-            .or_insert_with(SegmentReader::new);
+        let segment_reader = segment_readers.entry(*org).or_default();
 
         if segment_reader.expecting_chunk() {
             match msg {