samples: philosophers: clippy suggestions

Simple suggestions by clippy to make the code more idiomatic Rust.

Signed-off-by: David Brown <[email protected]>

Author: d3zd3z

samples/philosophers/src/channel.rs

@@ -32,21 +32,17 @@ enum Command {
 }
 
 /// This implements a single Fork on the server side for the ChannelSync.
+#[derive(Default)]
 enum ChannelFork {
     /// The fork is free,
+    #[default]
     Free,
     /// The work is in use, nobody is waiting.
     InUse,
     /// The fork is in use, and someone is waiting on it.
     InUseWait(Sender<()>),
 }
 
-impl Default for ChannelFork {
-    fn default() -> Self {
-        ChannelFork::Free
-    }
-}
-
 impl ChannelFork {
     /// Attempt to aquire the work.  If it is free, reply to the sender, otherwise, track them to
     /// reply to them when the fork is freed up.

samples/philosophers/src/dynsemsync.rs

@@ -37,15 +37,13 @@ pub fn dyn_semaphore_sync() -> Vec<Arc<dyn ForkSync>> {
         .each_ref()
         .map(|()| Arc::new(Semaphore::new(1, 1).unwrap()));
 
-    let syncers = (0..NUM_PHIL)
+    (0..NUM_PHIL)
         .map(|_| {
             let syncer = SemSync {
                 forks: forks.clone(),
             };
             let item = Box::new(syncer) as Box<dyn ForkSync>;
             Arc::from(item)
         })
-        .collect();
-
-    syncers
+        .collect()
 }

samples/philosophers/src/semsync.rs

@@ -38,17 +38,15 @@ pub fn semaphore_sync() -> Vec<Arc<dyn ForkSync>> {
         Arc::new(m.init_once((1, 1)).unwrap())
     });
 
-    let syncers = (0..NUM_PHIL)
+    (0..NUM_PHIL)
         .map(|_| {
             let syncer = SemSync {
                 forks: forks.clone(),
             };
             let item = Box::new(syncer) as Box<dyn ForkSync>;
             Arc::from(item)
         })
-        .collect();
-
-    syncers
+        .collect()
 }
 
 kobj_define! {