From b3caa615cd54c1865cf5c3fb27addaba9644ec59 Mon Sep 17 00:00:00 2001
From: Martin Tomka <martintomka@microsoft.com>
Date: Thu, 27 Jul 2023 15:18:29 +0200
Subject: [PATCH] Simplify OutcomeResilienceStrategy

---
 src/Polly.Core/Utils/OutcomeResilienceStrategy.cs | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/src/Polly.Core/Utils/OutcomeResilienceStrategy.cs b/src/Polly.Core/Utils/OutcomeResilienceStrategy.cs
index 60b163c7f4f..0d4368fcf2d 100644
--- a/src/Polly.Core/Utils/OutcomeResilienceStrategy.cs
+++ b/src/Polly.Core/Utils/OutcomeResilienceStrategy.cs
@@ -16,13 +16,11 @@ protected internal sealed override ValueTask<Outcome<TResult>> ExecuteCore<TResu
         TState state)
     {
         // Check if we can cast directly, thus saving some cycles and improving the performance
-        if (context.ResultType == typeof(T))
+        if (callback is Func<ResilienceContext, TState, ValueTask<Outcome<T>>> casted)
         {
-            // cast is safe here, because TResult and T are the same type
-            var callbackCasted = (Func<ResilienceContext, TState, ValueTask<Outcome<T>>>)(object)callback;
-            var valueTask = ExecuteCore(callbackCasted, context, state);
-
-            return TaskHelper.ConvertValueTask<T, TResult>(valueTask, context);
+            return TaskHelper.ConvertValueTask<T, TResult>(
+                ExecuteCore(casted, context, state),
+                context);
         }
         else
         {