Housekeeping - Update Spelling and minor code fixes (#3649)

<!-- Please be sure to read the
[Contribute](https://github.com/reactiveui/reactiveui#contribute)
section of the README -->

**What kind of change does this PR introduce?**
<!-- Bug fix, feature, docs update, ... -->

Update

**What is the current behavior?**
<!-- You can also link to an open issue here. -->

Code has spelling errors

**What is the new behavior?**
<!-- If this is a feature change -->

Fixed ReactiveUI main project spelling

**What might this PR break?**

none expected

**Please check if the PR fulfills these requirements**
- [ ] Tests for the changes have been added (for bug fixes / features)
- [ ] Docs have been added / updated (for bug fixes / features)

**Other information**:

Author: ChrisPulman

src/ReactiveUI.Fody.Helpers/ReactiveDependencyAttribute.cs

@@ -11,19 +11,17 @@ namespace ReactiveUI.Fody.Helpers;
 /// Attribute that marks a property as a Reactive Dependency.
 /// </summary>
 /// <seealso cref="System.Attribute" />
+/// <remarks>
+/// Initializes a new instance of the <see cref="ReactiveDependencyAttribute"/> class.
+/// </remarks>
+/// <param name="targetName">Name of the target.</param>
 [AttributeUsage(AttributeTargets.Property)]
-public sealed class ReactiveDependencyAttribute : Attribute
+public sealed class ReactiveDependencyAttribute(string targetName) : Attribute
 {
-    /// <summary>
-    /// Initializes a new instance of the <see cref="ReactiveDependencyAttribute"/> class.
-    /// </summary>
-    /// <param name="targetName">Name of the target.</param>
-    public ReactiveDependencyAttribute(string targetName) => Target = targetName;
-
     /// <summary>
     /// Gets the name of the backing property.
     /// </summary>
-    public string Target { get; }
+    public string Target { get; } = targetName;
 
     /// <summary>
     /// Gets or sets the target property on the backing property.

src/ReactiveUI/Activation/ViewForMixins.cs

@@ -227,14 +227,14 @@ public static IDisposable WhenActivated(this IActivatableView item, Action<Compo
                            },
                            view);
 
-    private static IDisposable HandleViewActivation(Func<IEnumerable<IDisposable>> block, IObservable<bool> activation)
+    private static CompositeDisposable HandleViewActivation(Func<IEnumerable<IDisposable>> block, IObservable<bool> activation)
     {
         var viewDisposable = new SerialDisposable();
 
         return new CompositeDisposable(
                                        activation.Subscribe(activated =>
                                        {
-                                           // NB: We need to make sure to respect ordering so that the cleanup
+                                           // NB: We need to make sure to respect ordering so that the clean up
                                            // happens before we invoke block again
                                            viewDisposable.Disposable = Disposable.Empty;
                                            if (activated)
@@ -245,7 +245,7 @@ private static IDisposable HandleViewActivation(Func<IEnumerable<IDisposable>> b
                                        viewDisposable);
     }
 
-    private static IDisposable HandleViewModelActivation(IViewFor view, IObservable<bool> activation)
+    private static CompositeDisposable HandleViewModelActivation(IViewFor view, IObservable<bool> activation)
     {
         var vmDisposable = new SerialDisposable();
         var viewVmDisposable = new SerialDisposable();
@@ -259,7 +259,7 @@ private static IDisposable HandleViewModelActivation(IViewFor view, IObservable<
                                                    .Select(x => x as IActivatableViewModel)
                                                    .Subscribe(x =>
                                                    {
-                                                       // NB: We need to make sure to respect ordering so that the cleanup
+                                                       // NB: We need to make sure to respect ordering so that the clean up
                                                        // happens before we activate again
                                                        vmDisposable.Disposable = Disposable.Empty;
                                                        if (x is not null)

src/ReactiveUI/Activation/ViewModelActivator.cs

@@ -45,9 +45,9 @@ public sealed class ViewModelActivator : IDisposable
     /// </summary>
     public ViewModelActivator()
     {
-        _blocks = new List<Func<IEnumerable<IDisposable>>>();
-        _activated = new Subject<Unit>();
-        _deactivated = new Subject<Unit>();
+        _blocks = new();
+        _activated = new();
+        _deactivated = new();
     }
 
     /// <summary>
@@ -111,4 +111,4 @@ public void Dispose()
     /// </summary>
     /// <param name="block">The block to add.</param>
     internal void AddActivationBlock(Func<IEnumerable<IDisposable>> block) => _blocks.Add(block);
-}
+}

src/ReactiveUI/Bindings/Property/IPropertyBinderImplementation.cs

@@ -226,4 +226,4 @@ IDisposable BindTo<TValue, TTarget, TTValue>(
         object? conversionHint,
         IBindingTypeConverter? vmToViewConverterOverride = null)
         where TTarget : class;
-}
+}

src/ReactiveUI/Bindings/Property/PropertyBinderImplementation.cs

@@ -3,7 +3,6 @@
 // The .NET Foundation licenses this file to you under the MIT license.
 // See the LICENSE file in the project root for full license information.
 
-using System.Diagnostics.CodeAnalysis;
 using System.Globalization;
 
 namespace ReactiveUI;
@@ -257,10 +256,7 @@ public IDisposable BindTo<TValue, TTarget, TTValue>(
         }
 
         var setter = _setMethodCache.Get((fromType, targetType));
-
-#pragma warning disable IDE0031 // Use null propagation
         return setter is null ? null : setter.PerformSet;
-#pragma warning restore IDE0031 // Use null propagation
     }
 
     private (IDisposable disposable, IObservable<TValue> value) BindToDirect<TTarget, TValue, TObs>(
@@ -332,7 +328,7 @@ private bool EvalBindingHooks<TViewModel, TView>(TViewModel? viewModel, TView vi
                                                                        })
                                                                  : (() => new IObservedChange<object, object?>[]
                                                                        {
-                                                                           new ObservedChange<object, object?>(null!, null!, viewModel)
+                                                                           new ObservedChange<object, object?>(null!, null, viewModel)
                                                                        });
 
         var vFetcher = new Func<IObservedChange<object, object?>[]>(() =>
@@ -354,7 +350,6 @@ private bool EvalBindingHooks<TViewModel, TView>(TViewModel? viewModel, TView vi
         return shouldBind;
     }
 
-    [SuppressMessage("Roslynator", "RCS1176: Use var instead of explicit", Justification = "Required for generics")]
     private IReactiveBinding<TView, (object? view, bool isViewModel)> BindImpl<TViewModel, TView, TVMProp, TVProp, TDontCare>(
         TViewModel? viewModel,
         TView view,
@@ -466,4 +461,4 @@ signalViewUpdate is null ?
                                                                             BindingDirection.TwoWay,
                                                                             disposable);
     }
-}
+}

src/ReactiveUI/Bindings/Property/PropertyBindingMixins.cs

@@ -75,7 +75,7 @@ static PropertyBindingMixins()
 
     /// <summary>
     /// Binds the specified view model property to the given view property, and
-    /// provide a custom view update signaler to signal when the view property has been updated.
+    /// provide a custom view update signaller to signal when the view property has been updated.
     /// </summary>
     /// <typeparam name="TViewModel">The type of the view model being bound.</typeparam>
     /// <typeparam name="TView">The type of the view being bound.</typeparam>
@@ -327,4 +327,4 @@ public static IDisposable BindTo<TValue, TTarget, TTValue>(
         IBindingTypeConverter? vmToViewConverterOverride = null)
         where TTarget : class =>
         _binderImplementation.BindTo(@this, target, property, conversionHint, vmToViewConverterOverride);
-}
+}

src/ReactiveUI/Bindings/Reactive/IReactiveBinding.cs

@@ -17,7 +17,7 @@ public interface IReactiveBinding<out TView, out TValue> : IDisposable
     where TView : IViewFor
 {
     /// <summary>
-    /// Gets an expression representing the property on the viewmodel bound to the view.
+    /// Gets an expression representing the property on the ViewModel bound to the view.
     /// This can be a child property, for example x.Foo.Bar.Baz in which case
     /// that will be the expression.
     /// </summary>
@@ -29,7 +29,7 @@ public interface IReactiveBinding<out TView, out TValue> : IDisposable
     TView View { get; }
 
     /// <summary>
-    /// Gets an expression representing the property on the view bound to the viewmodel.
+    /// Gets an expression representing the property on the view bound to the ViewModel.
     /// This can be a child property, for example x.Foo.Bar.Baz in which case
     /// that will be the expression.
     /// </summary>
@@ -44,4 +44,4 @@ public interface IReactiveBinding<out TView, out TValue> : IDisposable
     /// Gets the direction of the binding.
     /// </summary>
     BindingDirection Direction { get; }
-}
+}

src/ReactiveUI/Bindings/Reactive/ReactiveBinding.cs

@@ -5,42 +5,29 @@
 
 namespace ReactiveUI;
 
-internal class ReactiveBinding<TView, TValue> : IReactiveBinding<TView, TValue>
+internal class ReactiveBinding<TView, TValue>(
+    TView view,
+    Expression viewExpression,
+    Expression viewModelExpression,
+    IObservable<TValue?> changed,
+    BindingDirection direction,
+    IDisposable bindingDisposable) : IReactiveBinding<TView, TValue>
     where TView : IViewFor
 {
-    private readonly IDisposable _bindingDisposable;
-
-    public ReactiveBinding(
-        TView view,
-        Expression viewExpression,
-        Expression viewModelExpression,
-        IObservable<TValue?> changed,
-        BindingDirection direction,
-        IDisposable bindingDisposable)
-    {
-        View = view;
-        ViewExpression = viewExpression;
-        ViewModelExpression = viewModelExpression;
-        Direction = direction;
-        Changed = changed;
-
-        _bindingDisposable = bindingDisposable;
-    }
-
     /// <inheritdoc />
-    public Expression ViewModelExpression { get; }
+    public Expression ViewModelExpression { get; } = viewModelExpression;
 
     /// <inheritdoc />
-    public TView View { get; }
+    public TView View { get; } = view;
 
     /// <inheritdoc />
-    public Expression ViewExpression { get; }
+    public Expression ViewExpression { get; } = viewExpression;
 
     /// <inheritdoc />
-    public IObservable<TValue?> Changed { get; }
+    public IObservable<TValue?> Changed { get; } = changed;
 
     /// <inheritdoc />
-    public BindingDirection Direction { get; }
+    public BindingDirection Direction { get; } = direction;
 
     /// <inheritdoc />
     public void Dispose()
@@ -57,7 +44,7 @@ protected virtual void Dispose(bool isDisposing)
     {
         if (isDisposing)
         {
-            _bindingDisposable.Dispose();
+            bindingDisposable.Dispose();
         }
     }
-}
+}

src/ReactiveUI/Comparers/ChainedComparer.cs

@@ -5,27 +5,19 @@
 
 namespace ReactiveUI;
 
-internal sealed class ChainedComparer<T> : IComparer<T>
+internal sealed class ChainedComparer<T>(IComparer<T>? parent, Comparison<T> comparison) : IComparer<T>
 {
-    private readonly IComparer<T>? _parent;
-    private readonly Comparison<T> _inner;
-
-    public ChainedComparer(IComparer<T>? parent, Comparison<T> comparison)
-    {
-        _parent = parent;
-        _inner = comparison;
-    }
 
     /// <inheritdoc />
     public int Compare(T? x, T? y)
     {
-        var parentResult = _parent?.Compare(x!, y!) ?? 0;
+        var parentResult = parent?.Compare(x!, y!) ?? 0;
 
         if (x is null && y is null)
         {
             return 0;
         }
 
-        return parentResult != 0 ? parentResult : _inner(x!, y!);
+        return parentResult != 0 ? parentResult : comparison(x!, y!);
     }
-}
+}

src/ReactiveUI/Expression/ExpressionRewriter.cs

@@ -101,7 +101,7 @@ protected override Expression VisitUnary(UnaryExpression node)
     protected override Expression VisitMethodCall(MethodCallExpression node)
     {
         // Rewrite a method call to an indexer as an index expression
-        if (node.Arguments.Any(e => !(e is ConstantExpression)) || !node.Method.IsSpecialName)
+        if (node.Arguments.Any(e => e is not ConstantExpression) || !node.Method.IsSpecialName)
         {
             throw new NotSupportedException("Index expressions are only supported with constants.");
         }
@@ -120,11 +120,11 @@ protected override Expression VisitMethodCall(MethodCallExpression node)
 
     protected override Expression VisitIndex(IndexExpression node)
     {
-        if (node.Arguments.Any(e => !(e is ConstantExpression)))
+        if (node.Arguments.Any(e => e is not ConstantExpression))
         {
             throw new NotSupportedException("Index expressions are only supported with constants.");
         }
 
         return base.VisitIndex(node);
     }
-}
+}