Use readonly record struct

Resolves peer review feedback, and also seems to make it faster.
App-vNext · Oct 30, 2023 · 68dbc79 · 68dbc79
1 parent 5e3c076
commit 68dbc79
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 14 deletions.
diff --git a/...acts/results/Polly.Core.Benchmarks.CompositeComponentBenchmark-report-github.md b/...acts/results/Polly.Core.Benchmarks.CompositeComponentBenchmark-report-github.md
@@ -11,4 +11,4 @@ LaunchCount=2  WarmupCount=10
 ```
 | Method                         | Mean     | Error    | StdDev   | Allocated |
 |------------------------------- |---------:|---------:|---------:|----------:|
-| CompositeComponent_ExecuteCore | 41.37 ns | 1.722 ns | 2.413 ns |         - |
+| CompositeComponent_ExecuteCore | 37.44 ns | 0.713 ns | 0.952 ns |         - |
diff --git a/src/Polly.Core/Utils/Pipeline/DelegatingComponent.cs b/src/Polly.Core/Utils/Pipeline/DelegatingComponent.cs
@@ -77,18 +77,6 @@ internal sealed class DelegatingComponent : PipelineComponent
             new StateWrapper(Next!, callback, state!));
     }
 
-    private struct StateWrapper
-    {
-        public StateWrapper(PipelineComponent next, object callback, object state)
-        {
-            Next = next;
-            Callback = callback;
-            State = state;
-        }
-
-        public PipelineComponent Next;
-        public object Callback;
-        public object State;
-    }
+    private readonly record struct StateWrapper(PipelineComponent Next, object Callback, object State);
 #endif
 }