From 9bb545efcd181b5b47f332fd9c178eb3bff4c905 Mon Sep 17 00:00:00 2001
From: Daniel Weber <dw@exram.de>
Date: Wed, 16 May 2018 16:42:43 +0200
Subject: [PATCH] SynchronizationContextScheduler.Schedule: Avoid unnecessary
 allocation on one code path.

---
 .../SynchronizationContextScheduler.cs        | 20 +++++++++----------
 1 file changed, 9 insertions(+), 11 deletions(-)

diff --git a/Rx.NET/Source/src/System.Reactive/Concurrency/SynchronizationContextScheduler.cs b/Rx.NET/Source/src/System.Reactive/Concurrency/SynchronizationContextScheduler.cs
index 5a3a04128b..0ec534f1a9 100644
--- a/Rx.NET/Source/src/System.Reactive/Concurrency/SynchronizationContextScheduler.cs
+++ b/Rx.NET/Source/src/System.Reactive/Concurrency/SynchronizationContextScheduler.cs
@@ -57,22 +57,20 @@ public override IDisposable Schedule<TState>(TState state, Func<IScheduler, TSta
             if (action == null)
                 throw new ArgumentNullException(nameof(action));
 
-            var d = new SingleAssignmentDisposable();
-
             if (!_alwaysPost && _context == SynchronizationContext.Current)
             {
-                d.Disposable = action(this, state);
+                return action(this, state);
             }
-            else
+
+            var d = new SingleAssignmentDisposable();
+
+            _context.PostWithStartComplete(() =>
             {
-                _context.PostWithStartComplete(() =>
+                if (!d.IsDisposed)
                 {
-                    if (!d.IsDisposed)
-                    {
-                        d.Disposable = action(this, state);
-                    }
-                });
-            }
+                    d.Disposable = action(this, state);
+                }
+            });
 
             return d;
         }