Use Default derive and allow dead_code on currently unused field to m…

…ake clippy lint check happy. (#23) * Use Default derive and allow dead_code on currently unused fields to make clippy lint check happy. * Remove unused saga_template field.
oxidecomputer · Jan 19, 2022 · bd00d6f · bd00d6f
1 parent c6ea85c
commit bd00d6f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 9 deletions.
diff --git a/src/example_provision.rs b/src/example_provision.rs
@@ -53,13 +53,8 @@ pub struct ExampleParams {
     pub instance_name: String,
 }
 
-#[derive(Debug)]
+#[derive(Debug, Default)]
 pub struct ExampleContext;
-impl Default for ExampleContext {
-    fn default() -> ExampleContext {
-        ExampleContext {}
-    }
-}
 
 type SagaExampleContext = ActionContext<ExampleSagaType>;
 

diff --git a/src/saga_exec.rs b/src/saga_exec.rs
@@ -343,10 +343,9 @@ struct TaskParams<UserType: SagaType> {
  */
 #[derive(Debug)]
 pub struct SagaExecutor<UserType: SagaType> {
+    #[allow(dead_code)]
     log: slog::Logger,
 
-    // TODO This could probably be a reference instead.
-    saga_template: Arc<SagaTemplate<UserType>>,
     saga_metadata: Arc<SagaTemplateMetadata>,
 
     /** Channel for monitoring execution completion */
@@ -635,7 +634,6 @@ impl<UserType: SagaType> SagaExecutor<UserType> {
 
         Ok(SagaExecutor {
             log,
-            saga_template,
             saga_metadata,
             finish_tx,
             saga_id,