🐛 Fix build error for legacy and nullable subjects

When copying the implementation from System.Reactive, we improperly brought HasObservers as an override, but it should be just virtual, since we don't have a SubjectBase class.
devlooped · Sep 22, 2021 · d9e738b · d9e738b
1 parent 8cd6f92
commit d9e738b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/RxFree/build/Legacy/Subject.cs b/src/RxFree/build/Legacy/Subject.cs
@@ -35,7 +35,7 @@ internal class Subject<T> : IObserver<T>, IObservable<T>, IDisposable
         /// <summary>
         /// Indicates whether the subject has observers subscribed to it.
         /// </summary>
-        public override bool HasObservers => Volatile.Read(ref observers).Length != 0;
+        public virtual bool HasObservers => Volatile.Read(ref observers).Length != 0;
 
         /// <summary>
         /// Indicates whether the subject has been disposed.

diff --git a/src/RxFree/build/Nullable/Subject.cs b/src/RxFree/build/Nullable/Subject.cs
@@ -35,7 +35,7 @@ internal class Subject<T> : IObserver<T>, IObservable<T>, IDisposable
         /// <summary>
         /// Indicates whether the subject has observers subscribed to it.
         /// </summary>
-        public override bool HasObservers => Volatile.Read(ref observers).Length != 0;
+        public virtual bool HasObservers => Volatile.Read(ref observers).Length != 0;
 
         /// <summary>
         /// Indicates whether the subject has been disposed.