From cb1dd7ed53f9d8b07b9ae3becb5ae279b96f6ec7 Mon Sep 17 00:00:00 2001 From: h3xds1nz Date: Mon, 7 Apr 2025 19:54:05 +0200 Subject: [PATCH] [StyleCleanUp] Add readonly modifiers in WindowsBase --- .../src/Shared/MS/Internal/CopyOnWriteList.cs | 4 +-- .../CulturePreservingExecutionContext.cs | 6 ++--- .../IO/Packaging/PackagingUtilities.cs | 4 +-- .../MS/Internal/ReaderWriterLockWrapper.cs | 12 ++++----- .../Shared/MS/Internal/SafeSecurityHelper.cs | 6 ++--- .../Shared/MS/Internal/TextServicesLoader.cs | 4 +-- .../MS/Internal/WeakReferenceEnumerator.cs | 2 +- .../src/Shared/MS/Utility/FrugalList.cs | 12 ++++----- .../src/Shared/MS/Utility/FrugalMap.cs | 2 +- .../src/Shared/MS/Win32/HandleCollector.cs | 6 ++--- .../src/Shared/MS/Win32/HwndSubclass.cs | 6 ++--- .../src/Shared/MS/Win32/HwndWrapper.cs | 8 +++--- .../Shared/MS/Win32/ManagedWndProcTracker.cs | 4 +-- .../src/Shared/MS/Win32/NativeMethodsOther.cs | 2 +- .../System/Windows/Markup/ReflectionHelper.cs | 4 +-- .../Windows/Markup/TypeConverterHelper.cs | 2 +- .../Windows/Markup/XmlCompatibilityReader.cs | 26 +++++++++---------- .../src/WindowsBase/MS/Internal/AvTrace.cs | 4 +-- .../WindowsBase/MS/Internal/AvTraceDetails.cs | 6 ++--- .../WindowsBase/MS/Internal/AvTraceFormat.cs | 4 +-- .../WindowsBase/MS/Internal/CleanupHelper.cs | 14 +++++----- .../ComponentModel/APCustomTypeDescriptor.cs | 8 +++--- .../ComponentModel/DPCustomTypeDescriptor.cs | 12 ++++----- .../DependencyObjectPropertyDescriptor.cs | 20 +++++++------- .../DependencyObjectProvider.cs | 8 +++--- .../DependencyPropertyAttribute.cs | 6 ++--- .../MS/Internal/ComponentModel/PropertyKey.cs | 2 +- .../WindowsBase/MS/Internal/ContentType.cs | 4 +-- .../MS/Internal/IO/Packaging/Certificate.cs | 4 +-- .../IO/Packaging/CompoundFile/CFStream.cs | 6 ++--- .../CompoundFileDeflateTransform.cs | 2 +- .../CompoundFile/CompressionTransform.cs | 6 ++--- .../RightsManagementEncryptedStream.cs | 6 ++--- .../RightsManagementEncryptionTransform.cs | 6 ++--- .../CompoundFile/StreamWithDictionary.cs | 6 ++--- .../UserUseLicenseDictionaryLoader.cs | 6 ++--- .../Packaging/CompoundFile/VersionedStream.cs | 4 +-- .../CompoundFile/VersionedStreamOwner.cs | 4 +-- .../IO/Packaging/CompressEmulationStream.cs | 4 +-- .../IO/Packaging/OlePropertyStructs.cs | 2 +- .../MS/Internal/IO/Packaging/PackUriHelper.cs | 6 ++--- .../IO/Packaging/SparseMemoryStream.cs | 10 +++---- .../StorageBasedPackageProperties.cs | 4 +-- .../Packaging/XmlDigitalSignatureProcessor.cs | 4 +-- ...ObservableCollectionDefaultValueFactory.cs | 4 +-- .../WindowsBase/MS/Internal/PriorityItem.cs | 6 ++--- .../WindowsBase/MS/Internal/PriorityQueue.cs | 6 ++--- .../RightsManagement/CallbackHandler.cs | 4 +-- .../RightsManagement/ClientSession.cs | 8 +++--- .../RightsManagement/IssuanceLicense.cs | 2 +- .../NativeRightsManagementAPIsStructures.cs | 16 ++++++------ .../RightNameExpirationInfoPair.cs | 8 +++--- .../Threading/ExceptionFilterHelper.cs | 8 +++--- .../MS/Internal/TraceLevelStore.cs | 4 +-- .../WindowsBase/MS/Internal/WeakEventTable.cs | 14 +++++----- .../MS/Internal/WeakReferenceKey.cs | 6 ++--- .../ObjectModel/WeakReadOnlyCollection.cs | 6 ++--- .../DependencyPropertyDescriptor.cs | 12 ++++----- .../System/ComponentModel/GroupDescription.cs | 4 +-- .../System/ComponentModel/IItemProperties.cs | 8 +++--- .../PropertyChangedEventManager.cs | 4 +-- .../ComponentModel/PropertyFilterAttribute.cs | 4 +-- .../CompoundFile/DataSpaceManager.cs | 20 +++++++------- .../IO/Packaging/CompoundFile/StorageRoot.cs | 4 +-- .../IO/Packaging/CompoundFile/StreamInfo.cs | 4 +-- .../IO/Packaging/CompoundFile/VersionPair.cs | 4 +-- .../IO/Packaging/PackageDigitalSignature.cs | 6 ++--- .../PackageDigitalSignatureManager.cs | 13 +++++----- .../Packaging/RightsManagementInformation.cs | 4 +-- .../RightsManagement/CryptoProvider.cs | 6 ++--- .../Security/RightsManagement/Exceptions.cs | 4 +-- .../System/Security/RightsManagement/Grant.cs | 10 +++---- .../LocalizedNameDescriptionPair.cs | 4 +-- .../RightsManagement/PublishLicense.cs | 10 +++---- .../RightsManagement/SecureEnvironment.cs | 4 +-- .../UnsignedPublishLicense.cs | 5 ++-- .../Security/RightsManagement/UseLicense.cs | 8 +++--- .../System/Security/RightsManagement/User.cs | 7 ++--- ...tachedPropertyBrowsableForTypeAttribute.cs | 4 +-- ...yBrowsableWhenAttributePresentAttribute.cs | 4 +-- .../System/Windows/DependencyObjectType.cs | 2 +- .../System/Windows/DependencyProperty.cs | 10 +++---- .../DependencyPropertyChangedEventArgs.cs | 10 +++---- .../System/Windows/DependencySource.cs | 6 ++--- .../DesignerSerializationOptionsAttribute.cs | 4 +-- .../WindowsBase/System/Windows/EntryIndex.cs | 4 +-- .../System/Windows/Input/TraversalRequest.cs | 4 +-- .../System/Windows/LocalValueEnumerator.cs | 4 +-- .../System/Windows/Markup/ServiceProviders.cs | 2 +- .../WindowsBase/System/Windows/NameScope.cs | 4 +-- .../System/Windows/PropertyMetadata.cs | 8 +++--- .../System/Windows/Threading/Dispatcher.cs | 14 +++++----- .../Windows/Threading/DispatcherEventArgs.cs | 4 +-- .../Windows/Threading/DispatcherFrame.cs | 4 +-- .../Threading/DispatcherHookEventArgs.cs | 4 +-- .../Windows/Threading/DispatcherOperation.cs | 12 ++++----- .../DispatcherSynchronizationContext.cs | 4 +-- .../System/Windows/UncommonField.cs | 2 +- .../System/Windows/WeakEventManager.cs | 16 ++++++------ .../System/Windows/WeakEventManagerT.cs | 8 +++--- 100 files changed, 326 insertions(+), 323 deletions(-) diff --git a/src/Microsoft.DotNet.Wpf/src/Shared/MS/Internal/CopyOnWriteList.cs b/src/Microsoft.DotNet.Wpf/src/Shared/MS/Internal/CopyOnWriteList.cs index 80e6db8f9b3..7f0d4849f67 100644 --- a/src/Microsoft.DotNet.Wpf/src/Shared/MS/Internal/CopyOnWriteList.cs +++ b/src/Microsoft.DotNet.Wpf/src/Shared/MS/Internal/CopyOnWriteList.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. using System.Collections; @@ -184,7 +184,7 @@ protected void DoCopyOnWriteCheck() } } - private object _syncRoot; + private readonly object _syncRoot; private ArrayList _LiveList = new ArrayList(); private ArrayList _readonlyWrapper; } diff --git a/src/Microsoft.DotNet.Wpf/src/Shared/MS/Internal/CulturePreservingExecutionContext.cs b/src/Microsoft.DotNet.Wpf/src/Shared/MS/Internal/CulturePreservingExecutionContext.cs index 5ae517d416d..cdcb84b008c 100644 --- a/src/Microsoft.DotNet.Wpf/src/Shared/MS/Internal/CulturePreservingExecutionContext.cs +++ b/src/Microsoft.DotNet.Wpf/src/Shared/MS/Internal/CulturePreservingExecutionContext.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. // @@ -257,11 +257,11 @@ public void Dispose() #region Private Fields - private ExecutionContext _context; + private readonly ExecutionContext _context; private CultureAndContextManager _cultureAndContext; // static delegate to prevent repeated implicit allocations during Run - private static ContextCallback CallbackWrapperDelegate; + private static readonly ContextCallback CallbackWrapperDelegate; #endregion diff --git a/src/Microsoft.DotNet.Wpf/src/Shared/MS/Internal/IO/Packaging/PackagingUtilities.cs b/src/Microsoft.DotNet.Wpf/src/Shared/MS/Internal/IO/Packaging/PackagingUtilities.cs index 54ea931e4f7..4eefb078daf 100644 --- a/src/Microsoft.DotNet.Wpf/src/Shared/MS/Internal/IO/Packaging/PackagingUtilities.cs +++ b/src/Microsoft.DotNet.Wpf/src/Shared/MS/Internal/IO/Packaging/PackagingUtilities.cs @@ -814,8 +814,8 @@ private void CheckDisposed() /// Synchronize access to IsolatedStorage methods that can step on each-other /// /// See PS 1468964 for details. - private static Object _isoStoreSyncObject = new Object(); - private static Object _isolatedStorageFileLock = new Object(); + private static readonly Object _isoStoreSyncObject = new Object(); + private static readonly Object _isolatedStorageFileLock = new Object(); private static ReliableIsolatedStorageFileFolder _defaultFile; private const string XmlNamespace = "xmlns"; private const string _encodingAttribute = "encoding"; diff --git a/src/Microsoft.DotNet.Wpf/src/Shared/MS/Internal/ReaderWriterLockWrapper.cs b/src/Microsoft.DotNet.Wpf/src/Shared/MS/Internal/ReaderWriterLockWrapper.cs index febb1904c96..9a01af5ed11 100644 --- a/src/Microsoft.DotNet.Wpf/src/Shared/MS/Internal/ReaderWriterLockWrapper.cs +++ b/src/Microsoft.DotNet.Wpf/src/Shared/MS/Internal/ReaderWriterLockWrapper.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. // Description: @@ -184,7 +184,7 @@ private void CallWithNonPumpingWait(Action callback) #region Private Fields - private ReaderWriterLockSlim _rwLock; + private readonly ReaderWriterLockSlim _rwLock; private AutoReaderRelease _arr; private AutoWriterRelease _awr; private AutoReaderReleaseClass _arrc; @@ -217,7 +217,7 @@ public void Dispose() _wrapper.ReleaseWriterLock(); } - private ReaderWriterLockWrapper _wrapper; + private readonly ReaderWriterLockWrapper _wrapper; } private struct AutoReaderRelease : IDisposable @@ -232,7 +232,7 @@ public void Dispose() _wrapper.ReleaseReaderLock(); } - private ReaderWriterLockWrapper _wrapper; + private readonly ReaderWriterLockWrapper _wrapper; } private class AutoWriterReleaseClass : IDisposable @@ -247,7 +247,7 @@ public void Dispose() _wrapper.ReleaseWriterLock2(); } - private ReaderWriterLockWrapper _wrapper; + private readonly ReaderWriterLockWrapper _wrapper; } private class AutoReaderReleaseClass : IDisposable @@ -262,7 +262,7 @@ public void Dispose() _wrapper.ReleaseReaderLock2(); } - private ReaderWriterLockWrapper _wrapper; + private readonly ReaderWriterLockWrapper _wrapper; } // This SynchronizationContext waits without pumping messages, like diff --git a/src/Microsoft.DotNet.Wpf/src/Shared/MS/Internal/SafeSecurityHelper.cs b/src/Microsoft.DotNet.Wpf/src/Shared/MS/Internal/SafeSecurityHelper.cs index 37606b3b048..e4dc066c6d0 100644 --- a/src/Microsoft.DotNet.Wpf/src/Shared/MS/Internal/SafeSecurityHelper.cs +++ b/src/Microsoft.DotNet.Wpf/src/Shared/MS/Internal/SafeSecurityHelper.cs @@ -87,7 +87,7 @@ internal static Point ClientToScreen(UIElement relativeTo, Point point) // We use a callback on GC to purge out collected assemblies, so we don't grow indefinitely. // private static Dictionary _assemblies; // get key via GetKeyForAssembly - private static object syncObject = new object(); + private static readonly object syncObject = new object(); private static bool _isGCCallbackPending; // PERF: Cache delegate for CleanupCollectedAssemblies to avoid allocating it each time. @@ -312,8 +312,8 @@ public override bool Equals(object o) // will kick off a thread-pool job that you can use to purge a weakref cache. internal class GCNotificationToken { - private WaitCallback callback; - private object state; + private readonly WaitCallback callback; + private readonly object state; private GCNotificationToken(WaitCallback callback, object state) { diff --git a/src/Microsoft.DotNet.Wpf/src/Shared/MS/Internal/TextServicesLoader.cs b/src/Microsoft.DotNet.Wpf/src/Shared/MS/Internal/TextServicesLoader.cs index c02ff33eae6..b9b529dae7d 100644 --- a/src/Microsoft.DotNet.Wpf/src/Shared/MS/Internal/TextServicesLoader.cs +++ b/src/Microsoft.DotNet.Wpf/src/Shared/MS/Internal/TextServicesLoader.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. // @@ -359,7 +359,7 @@ private enum InstallState // Writes are not thread safe, but we don't mind the neglible perf hit // of potentially writing it twice. private static InstallState s_servicesInstalled = InstallState.Unknown; - private static object s_servicesInstalledLock = new object(); + private static readonly object s_servicesInstalledLock = new object(); #endregion Private Fields } diff --git a/src/Microsoft.DotNet.Wpf/src/Shared/MS/Internal/WeakReferenceEnumerator.cs b/src/Microsoft.DotNet.Wpf/src/Shared/MS/Internal/WeakReferenceEnumerator.cs index a08c90fd421..99101fd7cb6 100644 --- a/src/Microsoft.DotNet.Wpf/src/Shared/MS/Internal/WeakReferenceEnumerator.cs +++ b/src/Microsoft.DotNet.Wpf/src/Shared/MS/Internal/WeakReferenceEnumerator.cs @@ -61,7 +61,7 @@ public void Reset() } private int _i; - private ArrayList _List; + private readonly ArrayList _List; private object _StrongReference; } } diff --git a/src/Microsoft.DotNet.Wpf/src/Shared/MS/Utility/FrugalList.cs b/src/Microsoft.DotNet.Wpf/src/Shared/MS/Utility/FrugalList.cs index 6425cf6f625..810a7f8d8c4 100644 --- a/src/Microsoft.DotNet.Wpf/src/Shared/MS/Utility/FrugalList.cs +++ b/src/Microsoft.DotNet.Wpf/src/Shared/MS/Utility/FrugalList.cs @@ -213,7 +213,7 @@ public virtual FrugalListBase Finish() protected FrugalListBase _store; protected int _validItemCount; protected int _previousEnd; - private int _newCount; + private readonly int _newCount; } #endregion Compacter @@ -1612,9 +1612,9 @@ public override FrugalListBase Finish() return _targetStore; } - private ArrayItemList _targetStore; - private T[] _sourceArray; - private T[] _targetArray; + private readonly ArrayItemList _targetStore; + private readonly T[] _sourceArray; + private readonly T[] _targetArray; } #endregion Compacter @@ -1951,8 +1951,8 @@ public void Finish() } } - private FrugalObjectList _list; - private FrugalListBase.Compacter _storeCompacter; + private readonly FrugalObjectList _list; + private readonly FrugalListBase.Compacter _storeCompacter; } #endregion Compacter } diff --git a/src/Microsoft.DotNet.Wpf/src/Shared/MS/Utility/FrugalMap.cs b/src/Microsoft.DotNet.Wpf/src/Shared/MS/Utility/FrugalMap.cs index 20428baa064..5c1faf7620b 100644 --- a/src/Microsoft.DotNet.Wpf/src/Shared/MS/Utility/FrugalMap.cs +++ b/src/Microsoft.DotNet.Wpf/src/Shared/MS/Utility/FrugalMap.cs @@ -1655,7 +1655,7 @@ public override int Count // Hashtable will return null from its indexer if the key is not // found OR if the value is null. To distinguish between these // two cases we insert NullValue instead of null. - private static object NullValue = new object(); + private static readonly object NullValue = new object(); internal Dictionary _entries; } diff --git a/src/Microsoft.DotNet.Wpf/src/Shared/MS/Win32/HandleCollector.cs b/src/Microsoft.DotNet.Wpf/src/Shared/MS/Win32/HandleCollector.cs index ffad4dd3869..5c95dd1025e 100644 --- a/src/Microsoft.DotNet.Wpf/src/Shared/MS/Win32/HandleCollector.cs +++ b/src/Microsoft.DotNet.Wpf/src/Shared/MS/Win32/HandleCollector.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. using System; @@ -11,7 +11,7 @@ internal static class HandleCollector private static HandleType[] handleTypes; private static int handleTypeCount = 0; - private static Object handleMutex = new Object(); + private static readonly Object handleMutex = new Object(); /// /// Adds the given handle to the handle collector. This keeps the @@ -78,7 +78,7 @@ private class HandleType { internal readonly string name; - private int initialThreshHold; + private readonly int initialThreshHold; private int threshHold; private int handleCount; private readonly int deltaPercent; diff --git a/src/Microsoft.DotNet.Wpf/src/Shared/MS/Win32/HwndSubclass.cs b/src/Microsoft.DotNet.Wpf/src/Shared/MS/Win32/HwndSubclass.cs index 78be90d9195..2d46d632dbc 100644 --- a/src/Microsoft.DotNet.Wpf/src/Shared/MS/Win32/HwndSubclass.cs +++ b/src/Microsoft.DotNet.Wpf/src/Shared/MS/Win32/HwndSubclass.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. using System.Runtime.InteropServices; @@ -590,9 +590,9 @@ private enum Bond /// By instantiating this delegate as a static variable we ensure that /// it will remain alive long enough to process messages. /// - private static NativeMethods.WndProc DefWndProcStub = new NativeMethods.WndProc(DefWndProcWrapper); + private static readonly NativeMethods.WndProc DefWndProcStub = new NativeMethods.WndProc(DefWndProcWrapper); - private static IntPtr DefWndProc; + private static readonly IntPtr DefWndProc; private IntPtr _hwndAttached; private HandleRef _hwndHandleRef; diff --git a/src/Microsoft.DotNet.Wpf/src/Shared/MS/Win32/HwndWrapper.cs b/src/Microsoft.DotNet.Wpf/src/Shared/MS/Win32/HwndWrapper.cs index cc7cc630e1f..ced06545154 100644 --- a/src/Microsoft.DotNet.Wpf/src/Shared/MS/Win32/HwndWrapper.cs +++ b/src/Microsoft.DotNet.Wpf/src/Shared/MS/Win32/HwndWrapper.cs @@ -358,15 +358,15 @@ public DestroyWindowArgs(IntPtr handle, ushort classAtom) private IntPtr _handle; private UInt16 _classAtom; private WeakReferenceList _hooks; - private int _ownerThreadID; + private readonly int _ownerThreadID; - private HwndWrapperHook _wndProc; + private readonly HwndWrapperHook _wndProc; private bool _isDisposed; - private bool _isInCreateWindow = false; // debugging variable (temporary) + private readonly bool _isInCreateWindow = false; // debugging variable (temporary) // Message to cause a dispose. We need this to ensure we destroy the window on the right thread. - private static WindowMessage s_msgGCMemory; + private static readonly WindowMessage s_msgGCMemory; } // class RawWindow } diff --git a/src/Microsoft.DotNet.Wpf/src/Shared/MS/Win32/ManagedWndProcTracker.cs b/src/Microsoft.DotNet.Wpf/src/Shared/MS/Win32/ManagedWndProcTracker.cs index c99a15bdd60..93f97d757e7 100644 --- a/src/Microsoft.DotNet.Wpf/src/Shared/MS/Win32/ManagedWndProcTracker.cs +++ b/src/Microsoft.DotNet.Wpf/src/Shared/MS/Win32/ManagedWndProcTracker.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. //#define LOGGING @@ -238,7 +238,7 @@ private static void Log(string msg) private static IntPtr _cachedDefWindowProcA = IntPtr.Zero; private static IntPtr _cachedDefWindowProcW = IntPtr.Zero; - private static Hashtable _hwndList = new Hashtable(10); + private static readonly Hashtable _hwndList = new Hashtable(10); private static bool _exiting = false; } } diff --git a/src/Microsoft.DotNet.Wpf/src/Shared/MS/Win32/NativeMethodsOther.cs b/src/Microsoft.DotNet.Wpf/src/Shared/MS/Win32/NativeMethodsOther.cs index ce166c5a724..c8554a13578 100644 --- a/src/Microsoft.DotNet.Wpf/src/Shared/MS/Win32/NativeMethodsOther.cs +++ b/src/Microsoft.DotNet.Wpf/src/Shared/MS/Win32/NativeMethodsOther.cs @@ -206,7 +206,7 @@ internal class ICONINFO internal abstract class WpfSafeHandle : SafeHandleZeroOrMinusOneIsInvalid { - private int _collectorId; + private readonly int _collectorId; protected WpfSafeHandle(bool ownsHandle, int collectorId) : base(ownsHandle) { diff --git a/src/Microsoft.DotNet.Wpf/src/Shared/System/Windows/Markup/ReflectionHelper.cs b/src/Microsoft.DotNet.Wpf/src/Shared/System/Windows/Markup/ReflectionHelper.cs index f710356a081..9c0665bc670 100644 --- a/src/Microsoft.DotNet.Wpf/src/Shared/System/Windows/Markup/ReflectionHelper.cs +++ b/src/Microsoft.DotNet.Wpf/src/Shared/System/Windows/Markup/ReflectionHelper.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. #nullable disable @@ -476,7 +476,7 @@ private static Assembly LoadAssemblyHelper(string assemblyGivenName, string asse return retassem; } - private static Hashtable _loadedAssembliesHash = new Hashtable(8); + private static readonly Hashtable _loadedAssembliesHash = new Hashtable(8); #else // returns true is sourceAssembly declares LocalAssemblyName as a friend internal static bool IsFriendAssembly(Assembly sourceAssembly) diff --git a/src/Microsoft.DotNet.Wpf/src/Shared/System/Windows/Markup/TypeConverterHelper.cs b/src/Microsoft.DotNet.Wpf/src/Shared/System/Windows/Markup/TypeConverterHelper.cs index a106b9e726b..b2221bb1fd4 100644 --- a/src/Microsoft.DotNet.Wpf/src/Shared/System/Windows/Markup/TypeConverterHelper.cs +++ b/src/Microsoft.DotNet.Wpf/src/Shared/System/Windows/Markup/TypeConverterHelper.cs @@ -29,7 +29,7 @@ namespace System.Windows.Markup /// internal static class TypeConverterHelper { - private static CultureInfo invariantEnglishUS = CultureInfo.InvariantCulture; + private static readonly CultureInfo invariantEnglishUS = CultureInfo.InvariantCulture; internal static CultureInfo InvariantEnglishUS { diff --git a/src/Microsoft.DotNet.Wpf/src/Shared/System/Windows/Markup/XmlCompatibilityReader.cs b/src/Microsoft.DotNet.Wpf/src/Shared/System/Windows/Markup/XmlCompatibilityReader.cs index c25bbf5f729..23362803d45 100644 --- a/src/Microsoft.DotNet.Wpf/src/Shared/System/Windows/Markup/XmlCompatibilityReader.cs +++ b/src/Microsoft.DotNet.Wpf/src/Shared/System/Windows/Markup/XmlCompatibilityReader.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. #nullable disable @@ -1642,14 +1642,14 @@ public NamespaceElementPair(string namespaceName, string itemName) /// private class CompatibilityScope { - private CompatibilityScope _previous; - private int _depth; + private readonly CompatibilityScope _previous; + private readonly int _depth; private bool _fallbackSeen; private bool _inAlternateContent; private bool _inProcessContent; private bool _choiceTaken; private bool _choiceSeen; - private XmlCompatibilityReader _reader; + private readonly XmlCompatibilityReader _reader; private Dictionary _ignorables; private Dictionary _processContents; private Dictionary _preserveElements; @@ -1948,8 +1948,8 @@ public void Verify() private class ProcessContentSet { private bool _all; - private string _namespaceName; - private XmlCompatibilityReader _reader; + private readonly string _namespaceName; + private readonly XmlCompatibilityReader _reader; private HashSet _names; public ProcessContentSet(string namespaceName, XmlCompatibilityReader reader) @@ -2003,8 +2003,8 @@ public void Add(string elementName) private class PreserveItemSet { private bool _all; - private string _namespaceName; - private XmlCompatibilityReader _reader; + private readonly string _namespaceName; + private readonly XmlCompatibilityReader _reader; private Dictionary _names; public PreserveItemSet(string namespaceName, XmlCompatibilityReader reader) @@ -2059,12 +2059,12 @@ public void Add(string itemName) #region Private Fields private bool _inAttribute; // for Save/Restore ReaderPosition private string _currentName; // for Save/Restore ReaderPosition - private IsXmlNamespaceSupportedCallback _namespaceCallback; + private readonly IsXmlNamespaceSupportedCallback _namespaceCallback; private Dictionary _knownNamespaces; - private Dictionary _namespaceMap = new Dictionary(); private Dictionary _subsumingNamespaces; - private Dictionary _elementHandler = new Dictionary(); - private Dictionary _attributeHandler = new Dictionary(); + private readonly Dictionary _namespaceMap = new Dictionary(); + private readonly Dictionary _elementHandler = new Dictionary(); + private readonly Dictionary _attributeHandler = new Dictionary(); private int _depthOffset; // offset for Depth method, to account for elements that should be ignored by client private int _ignoredAttributeCount; private int _attributePosition; // used for ScanForCompatibility / HandleIgnorable @@ -2086,7 +2086,7 @@ public void Add(string itemName) private const string XmlnsDeclaration = "xmlns"; private const string MarkupCompatibilityURI = "http://schemas.openxmlformats.org/markup-compatibility/2006"; - private static string[] _predefinedNamespaces = new string[4] + private static readonly string[] _predefinedNamespaces = new string[4] { "http://www.w3.org/2000/xmlns/", "http://www.w3.org/XML/1998/namespace", diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/AvTrace.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/AvTrace.cs index ccc859647d0..5aca1af5c1f 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/AvTrace.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/AvTrace.cs @@ -479,8 +479,8 @@ public static Type GetTypeHelper(object value) private static bool _hasBeenRefreshed = false; // Delegates to create and remove the TraceSource instance - private GetTraceSourceDelegate _getTraceSourceDelegate; - private ClearTraceSourceDelegate _clearTraceSourceDelegate; + private readonly GetTraceSourceDelegate _getTraceSourceDelegate; + private readonly ClearTraceSourceDelegate _clearTraceSourceDelegate; // Cache of TraceSource instance; real value resides in PresentationTraceSources. private TraceSource _traceSource; diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/AvTraceDetails.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/AvTraceDetails.cs index e40fe5db360..fab33d723dd 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/AvTraceDetails.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/AvTraceDetails.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. namespace MS.Internal @@ -41,8 +41,8 @@ public string[] Labels } } - private int _id; - private string[] _labels; + private readonly int _id; + private readonly string[] _labels; } } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/AvTraceFormat.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/AvTraceFormat.cs index ac7ad6cd6eb..635e9a6ce50 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/AvTraceFormat.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/AvTraceFormat.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. namespace MS.Internal @@ -22,7 +22,7 @@ public override string Message } } - private string _message; + private readonly string _message; } } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/CleanupHelper.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/CleanupHelper.cs index 7a5cc7be16e..e81ad3c627a 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/CleanupHelper.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/CleanupHelper.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. // Description: Helper for classes that need to clean up data structures @@ -116,16 +116,16 @@ private void OnStarvationTick(object sender, EventArgs e) } private DispatcherTimer _cleanupTimer; - private DispatcherTimer _starvationTimer; - private DispatcherTimer _defaultCleanupTimer; + private readonly DispatcherTimer _starvationTimer; + private readonly DispatcherTimer _defaultCleanupTimer; private DispatcherPriority _cleanupTimerPriority; private int _cleanupRequests; private bool _waitingForGC; - private Func _cleanupCallback; - private TimeSpan _basePollingInterval; - private TimeSpan _maxPollingInterval; + private readonly Func _cleanupCallback; + private readonly TimeSpan _basePollingInterval; + private readonly TimeSpan _maxPollingInterval; // When an instance of this class is GC'd and finalized, it // tells the CleanupHelper that a GC has occurred. @@ -141,7 +141,7 @@ internal GCDetector(CleanupHelper parent) _parent._waitingForGC = false; } - private CleanupHelper _parent; + private readonly CleanupHelper _parent; } } } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ComponentModel/APCustomTypeDescriptor.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ComponentModel/APCustomTypeDescriptor.cs index ba3a97a4f16..2e5fb17114e 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ComponentModel/APCustomTypeDescriptor.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ComponentModel/APCustomTypeDescriptor.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. using System.ComponentModel; @@ -284,10 +284,10 @@ private DependencyProperty[] GetRegisteredProperties() #region Private Fields - private ICustomTypeDescriptor _parent; - private DependencyObject _instance; + private readonly ICustomTypeDescriptor _parent; + private readonly DependencyObject _instance; - private static object _syncLock = new object(); + private static readonly object _syncLock = new object(); // Synchronized by "_syncLock" private static int _dpCacheCount = 0; diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ComponentModel/DPCustomTypeDescriptor.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ComponentModel/DPCustomTypeDescriptor.cs index d4850f25e7c..6e689983358 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ComponentModel/DPCustomTypeDescriptor.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ComponentModel/DPCustomTypeDescriptor.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. using System.Collections; @@ -372,16 +372,16 @@ private PropertyDescriptorCollection CreateProperties() #region Private Fields - private ICustomTypeDescriptor _parent; - private Type _objectType; - private object _instance; + private readonly ICustomTypeDescriptor _parent; + private readonly Type _objectType; + private readonly object _instance; // Synchronized by "_propertyMap" - private static Dictionary _propertyMap = + private static readonly Dictionary _propertyMap = new Dictionary(new PropertyDescriptorComparer()); // Synchronized by "_typeProperties" - private static Hashtable _typeProperties = new Hashtable(); + private static readonly Hashtable _typeProperties = new Hashtable(); private const PropertyFilterOptions _anySet = PropertyFilterOptions.SetValues | PropertyFilterOptions.UnsetValues; private const PropertyFilterOptions _anyValid = PropertyFilterOptions.Valid; diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ComponentModel/DependencyObjectPropertyDescriptor.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ComponentModel/DependencyObjectPropertyDescriptor.cs index 8ab555d548e..815fc7af095 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ComponentModel/DependencyObjectPropertyDescriptor.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ComponentModel/DependencyObjectPropertyDescriptor.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. using System.Collections; @@ -810,29 +810,29 @@ private static Type[] DpType { #region Private Fields - private static Binder _dpBinder = new AttachedPropertyMethodSelector(); + private static readonly Binder _dpBinder = new AttachedPropertyMethodSelector(); - private static object _nullMethodSentinel = new object(); + private static readonly object _nullMethodSentinel = new object(); // Synchronized by "_getMethodCache". Note these are "reflection" // member infos and should only be passed types returned from // GetReflectionType. - private static Hashtable _getMethodCache = new Hashtable(); + private static readonly Hashtable _getMethodCache = new Hashtable(); // Synchronized by "_setMethodCache". Note these are "reflection" // member infos and should only be passed types returned from // GetReflectionType. - private static Hashtable _setMethodCache = new Hashtable(); + private static readonly Hashtable _setMethodCache = new Hashtable(); // Synchronization object for Attributes property. This would be better to be a // member than a static value, but the need for a lock on Attributes is very // rare and isn't worth the additional space this would take up as a member - private static object _attributeSyncLock = new object(); + private static readonly object _attributeSyncLock = new object(); - private PropertyDescriptor _property; - private DependencyProperty _dp; - private Type _componentType; - private PropertyMetadata _metadata; + private readonly PropertyDescriptor _property; + private readonly DependencyProperty _dp; + private readonly Type _componentType; + private readonly PropertyMetadata _metadata; private bool _queriedShouldSerializeMethod; private bool _queriedResetMethod; private Dictionary _trackers; diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ComponentModel/DependencyObjectProvider.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ComponentModel/DependencyObjectProvider.cs index bcca0e8adc3..77fd8527aac 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ComponentModel/DependencyObjectProvider.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ComponentModel/DependencyObjectProvider.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. using System.Collections; @@ -256,13 +256,13 @@ internal static DependencyPropertyKind GetDependencyPropertyKind(DependencyPrope private static readonly UncommonField _cacheSlot = new UncommonField(null); // Synchronized by "_propertyMap" - private static Dictionary _propertyMap = new Dictionary(); + private static readonly Dictionary _propertyMap = new Dictionary(); // Synchronized by "_propertyKindMap" - private static Dictionary _propertyKindMap = new Dictionary(); + private static readonly Dictionary _propertyKindMap = new Dictionary(); // Synchronized by "_attachInfoMap" - private static Hashtable _attachInfoMap = new Hashtable(); + private static readonly Hashtable _attachInfoMap = new Hashtable(); #endregion Private Fields } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ComponentModel/DependencyPropertyAttribute.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ComponentModel/DependencyPropertyAttribute.cs index 44051cd0ae7..ace9127a0fd 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ComponentModel/DependencyPropertyAttribute.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ComponentModel/DependencyPropertyAttribute.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. using System.Windows; @@ -135,8 +135,8 @@ internal DependencyProperty DependencyProperty #region Private Fields - private DependencyProperty _dp; - private bool _isAttached; + private readonly DependencyProperty _dp; + private readonly bool _isAttached; #endregion Private Fields } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ComponentModel/PropertyKey.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ComponentModel/PropertyKey.cs index 188b45d240a..4d906e3275a 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ComponentModel/PropertyKey.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ComponentModel/PropertyKey.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. using System.Windows; diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ContentType.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ContentType.cs index f71cbdb27cb..a1c5e5de5b0 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ContentType.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ContentType.cs @@ -644,9 +644,9 @@ private void EnsureParameterDictionary() private string _contentType = null; private string _type = String.Empty; private string _subType = String.Empty; - private string _originalString; + private readonly string _originalString; private Dictionary _parameterDictionary = null; - private bool _isInitialized = false; + private readonly bool _isInitialized; private const string _quote = "\""; private const char _semicolonSeparator = ';'; diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/Certificate.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/Certificate.cs index 4f48c249348..c4861d4105a 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/Certificate.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/Certificate.cs @@ -156,7 +156,7 @@ internal CertificatePart(System.IO.Packaging.Package container, Uri partName) // Private Fields // //------------------------------------------------------ - private PackagePart _part; // part that houses the certificate + private readonly PackagePart _part; // part that houses the certificate private X509Certificate2 _certificate; // certificate itself // certificate part constants @@ -165,7 +165,7 @@ internal CertificatePart(System.IO.Packaging.Package container, Uri partName) private static readonly string _certificatePartNamePrefix = "/package/services/digital-signature/certificate/"; private static readonly string _certificatePartNameExtension = ".cer"; private static readonly string _certificatePartRelationshipType = "http://schemas.openxmlformats.org/package/2006/relationships/digital-signature/certificate"; - private static long _maximumCertificateStreamLength = 0x40000; // 4MB + private static readonly long _maximumCertificateStreamLength = 0x40000; // 4MB #endregion Private Members } } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/CFStream.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/CFStream.cs index 06bc2d4b12f..43f83d97b22 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/CFStream.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/CFStream.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. // Description: @@ -341,7 +341,7 @@ protected override void Dispose(bool disposing) // //------------------------------------------------------ private IStream _safeIStream; - private FileAccess access; + private readonly FileAccess access; /// /// If only this stream object is held open, and the rest of the container @@ -349,6 +349,6 @@ protected override void Dispose(bool disposing) /// tree open because the CLR GC doesn't realize that our IStream has /// a dependency on the rest of the container object tree. /// - private StreamInfo backReference; + private readonly StreamInfo backReference; } } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/CompoundFileDeflateTransform.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/CompoundFileDeflateTransform.cs index 807c2a59b6d..91c025fbeb3 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/CompoundFileDeflateTransform.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/CompoundFileDeflateTransform.cs @@ -424,7 +424,7 @@ private static void ThrowIfZLibError(ZLibNative.ErrorCode retVal) //------------------------------------------------------ // for reading each block header - private byte[] _headerBuf = new byte[_blockHeaderSize]; // 3 ulongs + private readonly byte[] _headerBuf = new byte[_blockHeaderSize]; // 3 ulongs // static private const int _defaultBlockSize = 0x1000; // 4k default diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/CompressionTransform.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/CompressionTransform.cs index 3e9082cb158..9e770eda652 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/CompressionTransform.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/CompressionTransform.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. using System.Collections; @@ -111,8 +111,8 @@ public CompressionTransform(TransformEnvironment myEnvironment) // Private Data // //------------------------------------------------------ - private TransformEnvironment _transformEnvironment; - private VersionedStreamOwner _versionedStreamOwner; // our instance data stream wrapped + private readonly TransformEnvironment _transformEnvironment; + private readonly VersionedStreamOwner _versionedStreamOwner; // our instance data stream wrapped private static readonly string _featureName = "Microsoft.Metadata.CompressionTransform"; private static readonly VersionPair _currentFeatureVersion = new VersionPair(1, 0); diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/RightsManagementEncryptedStream.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/RightsManagementEncryptedStream.cs index e7d22e148d4..2ece95f77a9 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/RightsManagementEncryptedStream.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/RightsManagementEncryptedStream.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. using System.IO; @@ -881,7 +881,7 @@ private void RandomFillUp(Byte[] buffer, int offset, int count) private long _streamPosition; // always start at 0 - private CryptoProvider _cryptoProvider; + private readonly CryptoProvider _cryptoProvider; private const int _prefixLengthSize = 8; // 8 byte original stream size prefix @@ -901,7 +901,7 @@ private void RandomFillUp(Byte[] buffer, int offset, int count) // MaxValues below are used in order to ensure that we do not trigger any form of Isolated Storage Backup // This is not a goal here. We are definitely would like to keep SparseMemoryStream in the cached(in - memory) mode. - // In the context of our auto flush logic set at 16K we are pretty safe with those values… + // In the context of our auto flush logic set at 16K we are pretty safe with those values… private SparseMemoryStream _readCache = new SparseMemoryStream(Int32.MaxValue, Int64.MaxValue, false); private SparseMemoryStream _writeCache = new SparseMemoryStream(Int32.MaxValue, Int64.MaxValue, false); diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/RightsManagementEncryptionTransform.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/RightsManagementEncryptionTransform.cs index 204c4832877..24c521d17ee 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/RightsManagementEncryptionTransform.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/RightsManagementEncryptionTransform.cs @@ -942,7 +942,7 @@ internal UseLicense UseLicense set { _useLicense = value; } } - private ContentUser _user; + private readonly ContentUser _user; private UseLicense _useLicense; } @@ -1291,7 +1291,7 @@ Encoding encoding // // The stream in which the FormatVersion and publish license is stored. // - private VersionedStreamOwner _publishLicenseStream; + private readonly VersionedStreamOwner _publishLicenseStream; // // Uninterpreted bytes from the publish license stream header. @@ -1301,7 +1301,7 @@ Encoding encoding // // The storage under which use licenses are stored. // - private StorageInfo _useLicenseStorage; + private readonly StorageInfo _useLicenseStorage; // // All use licenses reside in streams whose names begin with this prefix: diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/StreamWithDictionary.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/StreamWithDictionary.cs index 0fd7b24dd69..d3bbbd457a9 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/StreamWithDictionary.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/StreamWithDictionary.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. // Description: @@ -12,8 +12,8 @@ namespace MS.Internal.IO.Packaging.CompoundFile { internal class StreamWithDictionary : Stream, IDictionary { - private Stream baseStream; - private IDictionary baseDictionary; + private readonly Stream baseStream; + private readonly IDictionary baseDictionary; private bool _disposed; // keep track of if we are disposed internal StreamWithDictionary( Stream wrappedStream, IDictionary wrappedDictionary ) diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/UserUseLicenseDictionaryLoader.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/UserUseLicenseDictionaryLoader.cs index 0cac8042cac..09ff7f67f83 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/UserUseLicenseDictionaryLoader.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/UserUseLicenseDictionaryLoader.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. // Description: @@ -136,12 +136,12 @@ ref bool stop // // The object that provides the implementation of the IDictionary methods. // - private Dictionary _dict; + private readonly Dictionary _dict; // // Text encoding object used to read or write publish licenses and use licenses. // - private UTF8Encoding _utf8Encoding = new UTF8Encoding(); + private readonly UTF8Encoding _utf8Encoding = new UTF8Encoding(); #endregion Private Fields } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/VersionedStream.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/VersionedStream.cs index f9b39b25308..ccecb84d90b 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/VersionedStream.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/VersionedStream.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. // Description: @@ -261,7 +261,7 @@ protected void CheckDisposed() // Private Fields // //------------------------------------------------------ - private VersionedStreamOwner _versionOwner; + private readonly VersionedStreamOwner _versionOwner; private Stream _stream; // null indicates Disposed state } } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/VersionedStreamOwner.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/VersionedStreamOwner.cs index 5e50c72b4b3..e665dd4119f 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/VersionedStreamOwner.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompoundFile/VersionedStreamOwner.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. // Description: @@ -417,7 +417,7 @@ private void EnsureParsed() //------------------------------------------------------ private bool _writeOccurred; // did one of our streams get written to? private bool _readOccurred; // did one of our streams get read from? - private FormatVersion _codeVersion; // code version + private readonly FormatVersion _codeVersion; // code version private FormatVersion _fileVersion; // current file version (null if not read or created yet) private long _dataOffset = 0; // where FormatVersion ends and data begins } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompressEmulationStream.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompressEmulationStream.cs index fc3f6e8c096..e0e5b052d2d 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompressEmulationStream.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/CompressEmulationStream.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. // Description: @@ -331,7 +331,7 @@ protected void CheckDisposed() private bool _dirty; // do we need to recompress? protected Stream _baseStream; // stream we ultimately decompress from and to in the container protected Stream _tempStream; // temporary storage for the uncompressed stream - private IDeflateTransform _transformer; // does the actual compress/decompress for us + private readonly IDeflateTransform _transformer; // does the actual compress/decompress for us #endregion } } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/OlePropertyStructs.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/OlePropertyStructs.cs index 02368ed2caf..014301d238a 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/OlePropertyStructs.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/OlePropertyStructs.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. using System.Runtime.InteropServices; diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/PackUriHelper.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/PackUriHelper.cs index 88a4c4207b8..15950d957f1 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/PackUriHelper.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/PackUriHelper.cs @@ -557,10 +557,10 @@ private int Compare(ValidatedPartUri otherPartUri) //------------------------------------------------------ private ValidatedPartUri _normalizedPartUri; - private string _partUriString; + private readonly string _partUriString; private string _normalizedPartUriString; - private bool _isNormalized; - private bool _isRelationshipPartUri; + private readonly bool _isNormalized; + private readonly bool _isRelationshipPartUri; //String Uppercase variants diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/SparseMemoryStream.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/SparseMemoryStream.cs index 967903536c9..2f0ecda0aee 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/SparseMemoryStream.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/SparseMemoryStream.cs @@ -802,7 +802,7 @@ private void EnsureIsolatedStoreStream() // //------------------------------------------------------ //we use this class to track total memory consumed by the Memory streams that we are using - private TrackingMemoryStreamFactory _trackingMemoryStreamFactory = new TrackingMemoryStreamFactory(); + private readonly TrackingMemoryStreamFactory _trackingMemoryStreamFactory = new TrackingMemoryStreamFactory(); private string _isolatedStorageStreamFileName; private Stream _isolatedStorageStream; @@ -824,10 +824,10 @@ private void EnsureIsolatedStoreStream() // This list is also maintained in offset order private MemoryStreamBlock _searchBlock; - private long _lowWaterMark; - private long _highWaterMark; + private readonly long _lowWaterMark; + private readonly long _highWaterMark; - private bool _autoCloseSmallBlockGaps; + private readonly bool _autoCloseSmallBlockGaps; } internal class MemoryStreamBlock : IComparable @@ -896,7 +896,7 @@ int IComparable.CompareTo(MemoryStreamBlock other) } } - private MemoryStream _stream; + private readonly MemoryStream _stream; private long _offset; } } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/StorageBasedPackageProperties.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/StorageBasedPackageProperties.cs index aac717aec4e..8c4d6a67cf6 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/StorageBasedPackageProperties.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/StorageBasedPackageProperties.cs @@ -874,13 +874,13 @@ private void private bool _disposed; - private int _grfMode; // Mode in which the compound file was opened. + private readonly int _grfMode; // Mode in which the compound file was opened. // // Interface to the OLE property sets in the compound file representing // the RM-protected OPC package. // - private IPropertySetStorage _pss; + private readonly IPropertySetStorage _pss; private IPropertyStorage _psSummInfo; private IPropertyStorage _psDocSummInfo; diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/XmlDigitalSignatureProcessor.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/XmlDigitalSignatureProcessor.cs index 7f82eb74ff8..4f07efa8bf6 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/XmlDigitalSignatureProcessor.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/IO/Packaging/XmlDigitalSignatureProcessor.cs @@ -1209,10 +1209,10 @@ private void ValidateReferences(IEnumerable references, bool allowPackageSpecifi // Private Members // //------------------------------------------------------ - private PackagePart _signaturePart; + private readonly PackagePart _signaturePart; private X509Certificate2 _certificate; // non-null if it's embedded private bool _lookForEmbeddedCert; - private PackageDigitalSignatureManager _manager; + private readonly PackageDigitalSignatureManager _manager; private PackageDigitalSignature _signature; // parsed from part or newly created private SignedXml _signedXml; // our format friend private String _hashAlgorithmName; // first hash algorithm obtained - considered to be the setting for the entire signature diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ObservableCollectionDefaultValueFactory.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ObservableCollectionDefaultValueFactory.cs index de060c62028..bbbc727b189 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ObservableCollectionDefaultValueFactory.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/ObservableCollectionDefaultValueFactory.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. using System.Windows; @@ -106,6 +106,6 @@ internal void OnDefaultValueChanged(object sender, NotifyCollectionChangedEventA } - private ObservableCollection _default; + private readonly ObservableCollection _default; } } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/PriorityItem.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/PriorityItem.cs index 19bc7e2d224..12f111f7019 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/PriorityItem.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/PriorityItem.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. namespace System.Windows.Threading @@ -23,8 +23,8 @@ public PriorityItem(T data) internal PriorityItem PriorityPrev {get{return _priorityPrev;} set{_priorityPrev=value;}} internal PriorityItem PriorityNext {get{return _priorityNext;} set{_priorityNext=value;}} - private T _data; - + private readonly T _data; + private PriorityItem _sequentialPrev; private PriorityItem _sequentialNext; diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/PriorityQueue.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/PriorityQueue.cs index 68a598642e5..df8a55c24d0 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/PriorityQueue.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/PriorityQueue.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. namespace System.Windows.Threading @@ -385,8 +385,8 @@ private void RemoveItemFromSequentialChain(PriorityItem item) } // Priority chains... - private SortedList> _priorityChains; // NOTE: should be Priority - private Stack> _cacheReusableChains; + private readonly SortedList> _priorityChains; // NOTE: should be Priority + private readonly Stack> _cacheReusableChains; // Sequential chain... private PriorityItem _head; diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/Security/RightsManagement/CallbackHandler.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/Security/RightsManagement/CallbackHandler.cs index 76893e4f680..24d25593b00 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/Security/RightsManagement/CallbackHandler.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/Security/RightsManagement/CallbackHandler.cs @@ -110,9 +110,9 @@ private void Dispose(bool disposing) } - private CallbackDelegate _callbackDelegate; + private readonly CallbackDelegate _callbackDelegate; - private AutoResetEvent _resetEvent; + private readonly AutoResetEvent _resetEvent; private string _callbackData; private int _hr; private Exception _exception; diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/Security/RightsManagement/ClientSession.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/Security/RightsManagement/ClientSession.cs index da25936fe83..6326891239d 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/Security/RightsManagement/ClientSession.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/Security/RightsManagement/ClientSession.cs @@ -1963,13 +1963,13 @@ private void CheckDisposed() private const string _passportActivationRegistryFullKeyName = @"HKEY_LOCAL_MACHINE\Software\Microsoft\MSDRM\ServiceLocation\PassportActivation"; private const string _passportActivationRegistryKeyName = @"Software\Microsoft\MSDRM\ServiceLocation\PassportActivation"; - private ContentUser _user = null; + private readonly ContentUser _user; private CallbackHandler _callbackHandler; private SafeRightsManagementSessionHandle _hSession = null; // if this is zero, we are disposed // we preserve this so ve can remove certificates in case of temp activation - private UserActivationMode _userActivationMode = UserActivationMode.Permanent; + private readonly UserActivationMode _userActivationMode = UserActivationMode.Permanent; private SafeRightsManagementEnvironmentHandle _envHandle = null; // if this is null, we are disposed @@ -1979,7 +1979,7 @@ private void CheckDisposed() // the following 2 arrays are used for parsing and converting between String and Enum; // therefore, the entries in the _rightEnums and the _rightNames must be in the same order. - private static ContentRight[] _rightEnums = { + private static readonly ContentRight[] _rightEnums = { ContentRight.View, ContentRight.Edit, ContentRight.Print, @@ -1995,7 +1995,7 @@ private void CheckDisposed() ContentRight.Export}; // entries in this array must be in UPPERCASE, as we make such assumption during parsing - private static string[] _rightNames = { + private static readonly string[] _rightNames = { "VIEW", "EDIT", "PRINT", diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/Security/RightsManagement/IssuanceLicense.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/Security/RightsManagement/IssuanceLicense.cs index 5831784ec20..fe322e1605a 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/Security/RightsManagement/IssuanceLicense.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/Security/RightsManagement/IssuanceLicense.cs @@ -1037,7 +1037,7 @@ private string ConvertAuthenticationTypeToString(ContentUser user) } - private List _pubHandlesList = new List(50); // initial capacity + private readonly List _pubHandlesList = new List(50); // initial capacity private SafeRightsManagementPubHandle _issuanceLicenseHandle = null; // if this is null, we are disposed private const string DefaultContentType = "MS-GUID"; diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/Security/RightsManagement/NativeRightsManagementAPIsStructures.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/Security/RightsManagement/NativeRightsManagementAPIsStructures.cs index 7a8e9c75d04..99217a264e3 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/Security/RightsManagement/NativeRightsManagementAPIsStructures.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/Security/RightsManagement/NativeRightsManagementAPIsStructures.cs @@ -78,14 +78,14 @@ internal DateTime GetDateTime (DateTime defaultValue) } } - private ushort Year =0; - private ushort Month =0; - private ushort DayOfWeek =0; - private ushort Day =0; - private ushort Hour =0; - private ushort Minute =0; - private ushort Second =0; - private ushort Milliseconds =0; + private readonly ushort Year; + private readonly ushort Month; + private readonly ushort DayOfWeek; + private readonly ushort Day; + private readonly ushort Hour; + private readonly ushort Minute; + private readonly ushort Second; + private readonly ushort Milliseconds; } [StructLayout(LayoutKind.Sequential)] diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/Security/RightsManagement/RightNameExpirationInfoPair.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/Security/RightsManagement/RightNameExpirationInfoPair.cs index 49b3a1aadba..1712fd4cf47 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/Security/RightsManagement/RightNameExpirationInfoPair.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/Security/RightsManagement/RightNameExpirationInfoPair.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. // Description: @@ -53,9 +53,9 @@ internal DateTime ValidUntil } } - private string _rightName; - private DateTime _validFrom; - private DateTime _validUntil; + private readonly string _rightName; + private readonly DateTime _validFrom; + private readonly DateTime _validUntil; } } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/Threading/ExceptionFilterHelper.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/Threading/ExceptionFilterHelper.cs index 31c3cb75850..4be0b19ccaa 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/Threading/ExceptionFilterHelper.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/Threading/ExceptionFilterHelper.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. // Description: Internal class replicating the functionality of the @@ -48,8 +48,8 @@ internal object TryCatchWhen( object source, return result; } - private InternalRealCallDelegate _internalRealCall; - private FilterExceptionDelegate _filterException; - private CatchExceptionDelegate _catchException; + private readonly InternalRealCallDelegate _internalRealCall; + private readonly FilterExceptionDelegate _filterException; + private readonly CatchExceptionDelegate _catchException; } } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/TraceLevelStore.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/TraceLevelStore.cs index c02e1b25c48..de8cc953952 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/TraceLevelStore.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/TraceLevelStore.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. namespace MS.Internal @@ -80,7 +80,7 @@ internal static void SetTraceLevel(object element, PresentationTraceLevel traceL // Private Fields // - private static Dictionary _dictionary = new Dictionary(); + private static readonly Dictionary _dictionary = new(); #endregion Private Fields diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/WeakEventTable.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/WeakEventTable.cs index 4bae231ea8b..81b902357ae 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/WeakEventTable.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/WeakEventTable.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. using System.Collections; @@ -436,16 +436,16 @@ private void OnShutDown() // Private Fields // - private Hashtable _managerTable = new Hashtable(); // maps manager type -> instance - private Hashtable _dataTable = new Hashtable(); // maps EventKey -> data - private Hashtable _eventNameTable = new Hashtable(); // maps -> manager + private readonly Hashtable _managerTable = new Hashtable(); // maps manager type -> instance + private readonly Hashtable _dataTable = new Hashtable(); // maps EventKey -> data + private readonly Hashtable _eventNameTable = new Hashtable(); // maps -> manager - private ReaderWriterLockWrapper _lock = new ReaderWriterLockWrapper(); + private readonly ReaderWriterLockWrapper _lock = new ReaderWriterLockWrapper(); private int _cleanupRequests; private bool _cleanupEnabled = true; - private CleanupHelper _cleanupHelper; + private readonly CleanupHelper _cleanupHelper; private bool _inPurge; - private List _toRemove = new List(); + private readonly List _toRemove = new List(); #if WeakEventTelemetry const int LOH_Threshold = 85000; // per LOH docs diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/WeakReferenceKey.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/WeakReferenceKey.cs index da3daaa77c0..da2111811db 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/WeakReferenceKey.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/MS/Internal/WeakReferenceKey.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. namespace MS.Internal.Utility @@ -45,7 +45,7 @@ public override int GetHashCode() return _hashCode; } - private WeakReference _item; - private int _hashCode; + private readonly WeakReference _item; + private readonly int _hashCode; } } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Collections/ObjectModel/WeakReadOnlyCollection.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Collections/ObjectModel/WeakReadOnlyCollection.cs index 519c0f47469..102838aeb9b 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Collections/ObjectModel/WeakReadOnlyCollection.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Collections/ObjectModel/WeakReadOnlyCollection.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. @@ -23,7 +23,7 @@ namespace System.Collections.ObjectModel internal class WeakReadOnlyCollection: IList, IList { //IList list; - private IList list; + private readonly IList list; [NonSerialized] private Object _syncRoot; @@ -307,7 +307,7 @@ void IEnumerator.Reset() { ie.Reset(); } - private IEnumerator ie; + private readonly IEnumerator ie; } } } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/ComponentModel/DependencyPropertyDescriptor.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/ComponentModel/DependencyPropertyDescriptor.cs index 51089cef176..5627e693ce4 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/ComponentModel/DependencyPropertyDescriptor.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/ComponentModel/DependencyPropertyDescriptor.cs @@ -595,17 +595,17 @@ private PropertyDescriptor Property #region Private Fields private PropertyDescriptor _property; - private Type _componentType; - private DependencyProperty _dp; - private bool _isAttached; - private PropertyMetadata _metadata; + private readonly Type _componentType; + private readonly DependencyProperty _dp; + private readonly bool _isAttached; + private readonly PropertyMetadata _metadata; // Synchronized by "_cache" - private static Dictionary _cache = + private static readonly Dictionary _cache = new Dictionary( ReferenceEqualityComparer.Instance ); - private static Dictionary _ignorePropertyTypeCache = + private static readonly Dictionary _ignorePropertyTypeCache = new Dictionary( ReferenceEqualityComparer.Instance ); diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/ComponentModel/GroupDescription.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/ComponentModel/GroupDescription.cs index e15253f01ab..cf9e14fc249 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/ComponentModel/GroupDescription.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/ComponentModel/GroupDescription.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. using System.Collections; @@ -245,7 +245,7 @@ private void SortDescriptionsChanged(object sender, NotifyCollectionChangedEvent // //------------------------------------------------------ - private ObservableCollection _explicitGroupNames; + private readonly ObservableCollection _explicitGroupNames; private SortDescriptionCollection _sort; private IComparer _customSort; diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/ComponentModel/IItemProperties.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/ComponentModel/IItemProperties.cs index 7756d59280f..3469c1345f9 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/ComponentModel/IItemProperties.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/ComponentModel/IItemProperties.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. // Description: Editing functionality for collection views. @@ -51,8 +51,8 @@ public ItemPropertyInfo(string name, Type type, object descriptor) /// public object Descriptor { get { return _descriptor; } } - private string _name; - private Type _type; - private object _descriptor; + private readonly string _name; + private readonly Type _type; + private readonly object _descriptor; } } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/ComponentModel/PropertyChangedEventManager.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/ComponentModel/PropertyChangedEventManager.cs index b90e2ca0f8f..ec7d5c459d4 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/ComponentModel/PropertyChangedEventManager.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/ComponentModel/PropertyChangedEventManager.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. // Description: Manager for the PropertyChanged event in the "weak event listener" @@ -612,7 +612,7 @@ private void OnPropertyChanged(object sender, PropertyChangedEventArgs args) #endregion Private Methods private ListenerList _proposedAllListenersList; - private List _toRemove = new List(); + private readonly List _toRemove = new List(); private static readonly string AllListenersKey = ""; // not a legal property name } } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/ComponentModel/PropertyFilterAttribute.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/ComponentModel/PropertyFilterAttribute.cs index e88a7997263..de459d6dd4e 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/ComponentModel/PropertyFilterAttribute.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/ComponentModel/PropertyFilterAttribute.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. namespace System.ComponentModel @@ -136,7 +136,7 @@ public PropertyFilterOptions Filter #region Private Fields - private PropertyFilterOptions _filter; + private readonly PropertyFilterOptions _filter; #endregion Private Fields } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/IO/Packaging/CompoundFile/DataSpaceManager.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/IO/Packaging/CompoundFile/DataSpaceManager.cs index 1ad6f499140..9cc807ebc24 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/IO/Packaging/CompoundFile/DataSpaceManager.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/IO/Packaging/CompoundFile/DataSpaceManager.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. using System.Collections; @@ -54,7 +54,7 @@ internal class DataSpaceManager /// There is only one data space manager per container instance. This /// points back to "our" reference. /// - private StorageRoot _associatedStorage; + private readonly StorageRoot _associatedStorage; /// /// Maps container references to data spaces @@ -107,7 +107,7 @@ internal class DataSpaceManager /// data in a state where it could not be written out because parts of the /// transform stack has already been disposed. /// - private ArrayList _transformedStreams; + private readonly ArrayList _transformedStreams; /// /// Table of "well-known" -- that is, "built-in" -- transforms. The keys are @@ -208,7 +208,7 @@ internal int ClassType /// internal StorageInfo transformStorage; - private int _classType; + private readonly int _classType; private byte[] _extraData; } @@ -1705,10 +1705,10 @@ internal interface IDataTransform /// internal class TransformInitializationEventArgs : EventArgs { - private IDataTransform dataInstance; - private string dataSpaceLabel; - private string streamPath; - private string transformLabel; + private readonly IDataTransform dataInstance; + private readonly string dataSpaceLabel; + private readonly string streamPath; + private readonly string transformLabel; internal TransformInitializationEventArgs( IDataTransform instance, @@ -1776,8 +1776,8 @@ internal string TransformInstanceLabel /// internal class TransformEnvironment { - private DataSpaceManager transformHost; - private string transformLabel; + private readonly DataSpaceManager transformHost; + private readonly string transformLabel; /// /// This object is only created internally by the data space manager. diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/IO/Packaging/CompoundFile/StorageRoot.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/IO/Packaging/CompoundFile/StorageRoot.cs index e259b785fe8..2657813dc84 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/IO/Packaging/CompoundFile/StorageRoot.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/IO/Packaging/CompoundFile/StorageRoot.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. using System.Runtime.InteropServices; @@ -38,7 +38,7 @@ internal class StorageRoot : StorageInfo /// /// If we know we are in a read-only mode, we know not to do certain things. /// - private bool containerIsReadOnly; + private readonly bool containerIsReadOnly; /// /// When data space manager is being initialized, sometimes it trips diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/IO/Packaging/CompoundFile/StreamInfo.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/IO/Packaging/CompoundFile/StreamInfo.cs index 9689607a1a8..41714f48fb4 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/IO/Packaging/CompoundFile/StreamInfo.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/IO/Packaging/CompoundFile/StreamInfo.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. using System.Runtime.InteropServices; @@ -80,7 +80,7 @@ public class StreamInfo /// /// CompoundFileStreamReference for this StreamInfo object /// - private CompoundFileStreamReference _streamReference; + private readonly CompoundFileStreamReference _streamReference; /// /// We need to rememeber the FileAccess that was used for openning diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/IO/Packaging/CompoundFile/VersionPair.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/IO/Packaging/CompoundFile/VersionPair.cs index 77da47a0aa6..6db894f0110 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/IO/Packaging/CompoundFile/VersionPair.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/IO/Packaging/CompoundFile/VersionPair.cs @@ -349,8 +349,8 @@ public int CompareTo(Object obj) #region Member Variables - private Int16 _major; // Major number - private Int16 _minor; // Minor number + private readonly Int16 _major; // Major number + private readonly Int16 _minor; // Minor number #endregion Member Variables } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/IO/Packaging/PackageDigitalSignature.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/IO/Packaging/PackageDigitalSignature.cs index ac278afd942..8e55ca9b851 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/IO/Packaging/PackageDigitalSignature.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/IO/Packaging/PackageDigitalSignature.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. using System.Security.Cryptography.Xml; @@ -421,8 +421,8 @@ private void ThrowIfInvalidated() // Private Fields // //------------------------------------------------------ - private PackageDigitalSignatureManager _manager; - private XmlDigitalSignatureProcessor _processor; + private readonly PackageDigitalSignatureManager _manager; + private readonly XmlDigitalSignatureProcessor _processor; private CertificatePart _certificatePart; private ReadOnlyCollection _signedParts; private ReadOnlyCollection _signedRelationshipSelectors; diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/IO/Packaging/PackageDigitalSignatureManager.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/IO/Packaging/PackageDigitalSignatureManager.cs index 0012c4d802d..8bbf9bbf160 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/IO/Packaging/PackageDigitalSignatureManager.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/IO/Packaging/PackageDigitalSignatureManager.cs @@ -95,8 +95,8 @@ internal SignatureVerificationEventArgs(PackageDigitalSignature signature, // Private Members // //------------------------------------------------------ - private PackageDigitalSignature _signature; - private VerifyResult _result; + private readonly PackageDigitalSignature _signature; + private readonly VerifyResult _result; } /// @@ -897,7 +897,7 @@ public bool Match(String id) return (string.Equals(_id, id, StringComparison.Ordinal)); } - private string _id; + private readonly string _id; } /// @@ -1342,19 +1342,20 @@ private PackagePart OriginPart // //------------------------------------------------------ private CertificateEmbeddingOption _certificateEmbeddingOption; - private Package _container; + private readonly Package _container; private IntPtr _parentWindow; - private static Uri _defaultOriginPartName = PackUriHelper.CreatePartUri(new Uri("/package/services/digital-signature/origin.psdsor", UriKind.Relative)); private Uri _originPartName = _defaultOriginPartName; private PackagePart _originPart; private String _hashAlgorithmString = DefaultHashAlgorithm; private String _signatureTimeFormat = XmlSignatureProperties.DefaultDateTimeFormat; private List _signatures; - private Dictionary _transformDictionary; + private readonly Dictionary _transformDictionary; private bool _originSearchConducted; // don't look more than once for Origin part private bool _originPartExists; // was the part found? private ReadOnlyCollection _signatureList; // lazy-init cached return value for Signatures property + private static readonly Uri _defaultOriginPartName = PackUriHelper.CreatePartUri(new Uri("/package/services/digital-signature/origin.psdsor", UriKind.Relative)); + private static readonly ContentType _originPartContentType = new ContentType("application/vnd.openxmlformats-package.digital-signature-origin"); private static readonly String _guidStorageFormatString = @"N"; // N - simple format without adornments diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/IO/Packaging/RightsManagementInformation.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/IO/Packaging/RightsManagementInformation.cs index 6d5ab25135c..2c32bdcdbe8 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/IO/Packaging/RightsManagementInformation.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/IO/Packaging/RightsManagementInformation.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. using System.Security.RightsManagement; @@ -166,6 +166,6 @@ public IDictionary GetEmbeddedUseLicenses() return _rmet.GetEmbeddedUseLicenses(); } - private RightsManagementEncryptionTransform _rmet; + private readonly RightsManagementEncryptionTransform _rmet; } } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/CryptoProvider.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/CryptoProvider.cs index dba51815840..4fb1ba6ce56 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/CryptoProvider.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/CryptoProvider.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. using System.Collections.ObjectModel; @@ -521,10 +521,10 @@ private SafeRightsManagementHandle BoundLicenseOwnerViewRightsHandle // if this is Invalid, we are disposed private List _boundLicenseHandleList; - private List _boundRightsInfoList; + private readonly List _boundRightsInfoList; private ReadOnlyCollection _boundGrantReadOnlyCollection; - private ContentUser _owner; + private readonly ContentUser _owner; private bool _disposed; } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/Exceptions.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/Exceptions.cs index 5b2194b29a2..1507b917726 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/Exceptions.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/Exceptions.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. using System.Runtime.Serialization; @@ -153,7 +153,7 @@ public RightsManagementFailureCode FailureCode } } - private RightsManagementFailureCode _failureCode; + private readonly RightsManagementFailureCode _failureCode; private const string _serializationFailureCodeAttributeName = "FailureCode"; } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/Grant.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/Grant.cs index aef83d7ad12..bb82478421e 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/Grant.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/Grant.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. namespace System.Security.RightsManagement @@ -101,9 +101,9 @@ public DateTime ValidUntil } } - private ContentUser _user; - private ContentRight _right; - private DateTime _validFrom; - private DateTime _validUntil; + private readonly ContentUser _user; + private readonly ContentRight _right; + private readonly DateTime _validFrom; + private readonly DateTime _validUntil; } } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/LocalizedNameDescriptionPair.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/LocalizedNameDescriptionPair.cs index fd89456dc70..c0682e924f5 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/LocalizedNameDescriptionPair.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/LocalizedNameDescriptionPair.cs @@ -73,7 +73,7 @@ public override int GetHashCode() return Name.GetHashCode() ^ Description.GetHashCode(); } - private string _name; - private string _description; + private readonly string _name; + private readonly string _description; } } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/PublishLicense.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/PublishLicense.cs index e57298e8082..1b33a9682ea 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/PublishLicense.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/PublishLicense.cs @@ -162,10 +162,10 @@ public UseLicense AcquireUseLicenseNoUI(SecureEnvironment secureEnvironment) return secureEnvironment.ClientSession.AcquireUseLicense(_serializedPublishLicense, true); } - private string _serializedPublishLicense; - private string _referralInfoName; - private Uri _referralInfoUri; - private Guid _contentId; - private Uri _useLicenseAcquisitionUriFromPublishLicense = null; + private readonly string _serializedPublishLicense; + private readonly string _referralInfoName; + private readonly Uri _referralInfoUri; + private readonly Guid _contentId; + private readonly Uri _useLicenseAcquisitionUriFromPublishLicense; } } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/SecureEnvironment.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/SecureEnvironment.cs index 4ad78561bc2..bd612e92898 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/SecureEnvironment.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/SecureEnvironment.cs @@ -353,8 +353,8 @@ private void CheckDisposed() ObjectDisposedException.ThrowIf(_clientSession == null, typeof(SecureEnvironment)); } - private ContentUser _user; - private string _applicationManifest; + private readonly ContentUser _user; + private readonly string _applicationManifest; private ClientSession _clientSession; // if null we are disposed } } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/UnsignedPublishLicense.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/UnsignedPublishLicense.cs index 8ee00b3b6ad..a4b6c7e08c7 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/UnsignedPublishLicense.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/UnsignedPublishLicense.cs @@ -314,10 +314,11 @@ internal RevocationPoint RevocationPoint _revocationPoint = value; } } - + + private readonly ICollection _grantCollection; + private Guid _contentId; private ContentUser _owner; - private ICollection _grantCollection; private string _referralInfoName; private Uri _referralInfoUri; private IDictionary _localizedNameDescriptionDictionary = null; diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/UseLicense.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/UseLicense.cs index 8161c0c6761..f1c6875e685 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/UseLicense.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/UseLicense.cs @@ -140,9 +140,9 @@ public override int GetHashCode() } - private string _serializedUseLicense; - private Guid _contentId; - private ContentUser _owner = null; - private IDictionary _applicationSpecificDataDictionary = null; + private readonly string _serializedUseLicense; + private readonly Guid _contentId; + private readonly ContentUser _owner; + private readonly IDictionary _applicationSpecificDataDictionary; } } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/User.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/User.cs index e94fbd0b86f..ef971759361 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/User.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Security/RightsManagement/User.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. using MS.Internal; @@ -257,8 +257,9 @@ internal static bool CompareToOwner(string name) private const string AnyoneUserName = "Anyone"; private static ContentUser _anyoneUser; - private string _name; - private AuthenticationType _authenticationType; + private readonly string _name; + private readonly AuthenticationType _authenticationType; + private int hashValue; private bool hashCalcIsDone; // flag that indicates the value in hasValue is already calculated and usable } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/AttachedPropertyBrowsableForTypeAttribute.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/AttachedPropertyBrowsableForTypeAttribute.cs index fd8a612955b..077fb2955bd 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/AttachedPropertyBrowsableForTypeAttribute.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/AttachedPropertyBrowsableForTypeAttribute.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. namespace System.Windows @@ -159,7 +159,7 @@ internal override bool UnionResults // //------------------------------------------------------ - private Type _targetType; + private readonly Type _targetType; private DependencyObjectType _dTargetType; private bool _dTargetTypeChecked; } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/AttachedPropertyBrowsableWhenAttributePresentAttribute.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/AttachedPropertyBrowsableWhenAttributePresentAttribute.cs index 4ccaf459bdf..0e2a7fb0f7f 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/AttachedPropertyBrowsableWhenAttributePresentAttribute.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/AttachedPropertyBrowsableWhenAttributePresentAttribute.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. using System.ComponentModel; @@ -104,7 +104,7 @@ internal override bool IsBrowsable(DependencyObject d, DependencyProperty dp) // //------------------------------------------------------ - private Type _attributeType; + private readonly Type _attributeType; } } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/DependencyObjectType.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/DependencyObjectType.cs index 6b2623a5b07..8f19cbc5c18 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/DependencyObjectType.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/DependencyObjectType.cs @@ -212,7 +212,7 @@ private DependencyObjectType() private DependencyObjectType _baseDType; // Synchronized: Covered by DispatcherLock - private static Dictionary DTypeFromCLRType = new Dictionary(); + private static readonly Dictionary DTypeFromCLRType = new(); // Synchronized: Covered by DispatcherLock private static int DTypeCount = 0; diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/DependencyProperty.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/DependencyProperty.cs index 66c1789ef85..0e07ea2e7f3 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/DependencyProperty.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/DependencyProperty.cs @@ -1132,11 +1132,11 @@ internal CoerceValueCallback DesignerCoerceValueCallback /// Standard unset value public static readonly object UnsetValue = new NamedObject("DependencyProperty.UnsetValue"); - private string _name; - private Type _propertyType; - private Type _ownerType; - private PropertyMetadata _defaultMetadata; - private ValidateValueCallback _validateValueCallback; + private readonly string _name; + private readonly Type _propertyType; + private readonly Type _ownerType; + private readonly PropertyMetadata _defaultMetadata; + private readonly ValidateValueCallback _validateValueCallback; private DependencyPropertyKey _readOnlyKey; diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/DependencyPropertyChangedEventArgs.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/DependencyPropertyChangedEventArgs.cs index fd1bf62767f..c043302d278 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/DependencyPropertyChangedEventArgs.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/DependencyPropertyChangedEventArgs.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. namespace System.Windows @@ -330,15 +330,15 @@ private enum PrivateFlags : byte #region Data - private DependencyProperty _property; - private PropertyMetadata _metadata; + private readonly DependencyProperty _property; + private readonly PropertyMetadata _metadata; private PrivateFlags _flags; private EffectiveValueEntry _oldEntry; private EffectiveValueEntry _newEntry; - - private OperationType _operationType; + + private readonly OperationType _operationType; #endregion Data } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/DependencySource.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/DependencySource.cs index eb19370e2ab..7e22693bc44 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/DependencySource.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/DependencySource.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. namespace System.Windows @@ -40,7 +40,7 @@ public DependencyProperty DependencyProperty get { return _dp; } } - private DependencyObject _d; - private DependencyProperty _dp; + private readonly DependencyObject _d; + private readonly DependencyProperty _dp; } } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/DesignerSerializationOptionsAttribute.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/DesignerSerializationOptionsAttribute.cs index a43b272ba09..f9106daca11 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/DesignerSerializationOptionsAttribute.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/DesignerSerializationOptionsAttribute.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. using System.ComponentModel; @@ -44,7 +44,7 @@ public DesignerSerializationOptions DesignerSerializationOptions #region Data - private DesignerSerializationOptions _designerSerializationOptions; + private readonly DesignerSerializationOptions _designerSerializationOptions; #endregion Data } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/EntryIndex.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/EntryIndex.cs index 6358ab564bc..520f16cefc9 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/EntryIndex.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/EntryIndex.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. /***************************************************************************\ @@ -41,7 +41,7 @@ public uint Index get { return _store & 0x7FFFFFFF; } } - private uint _store; + private readonly uint _store; } } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Input/TraversalRequest.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Input/TraversalRequest.cs index 54ac52ea7e1..53e9e539346 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Input/TraversalRequest.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Input/TraversalRequest.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. namespace System.Windows.Input @@ -45,7 +45,7 @@ public bool Wrapped public FocusNavigationDirection FocusNavigationDirection { get { return _focusNavigationDirection; } } private bool _wrapped; - private FocusNavigationDirection _focusNavigationDirection; + private readonly FocusNavigationDirection _focusNavigationDirection; } /// diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/LocalValueEnumerator.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/LocalValueEnumerator.cs index 58485b9c177..ad305f96576 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/LocalValueEnumerator.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/LocalValueEnumerator.cs @@ -123,8 +123,8 @@ internal LocalValueEnumerator(LocalValueEntry[] snapshot, int count) } private int _index; - private LocalValueEntry[] _snapshot; - private int _count; + private readonly LocalValueEntry[] _snapshot; + private readonly int _count; } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Markup/ServiceProviders.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Markup/ServiceProviders.cs index b6528325b33..ed6101dddff 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Markup/ServiceProviders.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Markup/ServiceProviders.cs @@ -48,6 +48,6 @@ public void AddService(Type serviceType, Object service) } } - private Dictionary _objDict = new Dictionary(); + private readonly Dictionary _objDict = new(); } } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/NameScope.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/NameScope.cs index 20468bbc5f1..38d1ab1382c 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/NameScope.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/NameScope.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. /***************************************************************************\ @@ -371,7 +371,7 @@ public ICollection Values #region class Enumerator private class Enumerator : IEnumerator> { - private IDictionaryEnumerator _enumerator; + private readonly IDictionaryEnumerator _enumerator; public Enumerator(HybridDictionary nameMap) { diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/PropertyMetadata.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/PropertyMetadata.cs index 22790c536b3..9ce796736aa 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/PropertyMetadata.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/PropertyMetadata.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. using MS.Internal; @@ -513,7 +513,7 @@ private static bool DefaultFreezeValueCallback( return true; } - private static FreezeValueCallback _defaultFreezeValueCallback = DefaultFreezeValueCallback; + private static readonly FreezeValueCallback _defaultFreezeValueCallback = DefaultFreezeValueCallback; /// /// Creates a new instance of this property metadata. This method is used @@ -744,8 +744,8 @@ internal bool Sealed // We use this uncommon field to stash values created by our default value factory // in the owner's _localStore. private static readonly UncommonField _defaultValueFactoryCache = new UncommonField(); - private static FrugalMapIterationCallback _removalCallback = new FrugalMapIterationCallback(DefaultValueCacheRemovalCallback); - private static FrugalMapIterationCallback _promotionCallback = new FrugalMapIterationCallback(DefaultValueCachePromotionCallback); + private static readonly FrugalMapIterationCallback _removalCallback = new FrugalMapIterationCallback(DefaultValueCacheRemovalCallback); + private static readonly FrugalMapIterationCallback _promotionCallback = new FrugalMapIterationCallback(DefaultValueCachePromotionCallback); } /// diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Threading/Dispatcher.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Threading/Dispatcher.cs index 9b6b1dd8d90..df360841614 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Threading/Dispatcher.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Threading/Dispatcher.cs @@ -2799,14 +2799,14 @@ private object[] CombineParameters(object arg, object[] args) private const int TIMERID_TIMERS = 2; private const int DELTA_BACKGROUND = 1; - private static List _dispatchers; + private static readonly List _dispatchers; private static WeakReference _possibleDispatcher; private static readonly object _globalLock; [ThreadStatic] private static Dispatcher _tlsDispatcher; // use TLS for ownership only - private Thread _dispatcherThread; + private readonly Thread _dispatcherThread; private int _frameDepth; internal bool _exitAllFrames; // used from DispatcherFrame @@ -2822,21 +2822,21 @@ private object[] CombineParameters(object arg, object[] args) private MessageOnlyHwndWrapper _window; - private HwndWrapperHook _hook; + private readonly HwndWrapperHook _hook; private int _postedProcessingType; - private static WindowMessage _msgProcessQueue; + private static readonly WindowMessage _msgProcessQueue; - private static ExceptionWrapper _exceptionWrapper; + private static readonly ExceptionWrapper _exceptionWrapper; private static readonly object ExceptionDataKey = new object(); // Preallocated arguments for exception handling. // This helps avoid allocations in the handler code, a potential // source of secondary exceptions (i.e. in Out-Of-Memory cases). - private DispatcherUnhandledExceptionEventArgs _unhandledExceptionEventArgs; + private readonly DispatcherUnhandledExceptionEventArgs _unhandledExceptionEventArgs; private DispatcherUnhandledExceptionFilterEventHandler _unhandledExceptionFilter; - private DispatcherUnhandledExceptionFilterEventArgs _exceptionFilterEventArgs; + private readonly DispatcherUnhandledExceptionFilterEventArgs _exceptionFilterEventArgs; private object _reserved0; private object _reserved1; diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Threading/DispatcherEventArgs.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Threading/DispatcherEventArgs.cs index 7b8987d0f96..2642ecfd9ba 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Threading/DispatcherEventArgs.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Threading/DispatcherEventArgs.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. namespace System.Windows.Threading @@ -26,7 +26,7 @@ internal DispatcherEventArgs(Dispatcher dispatcher) _dispatcher = dispatcher; } - private Dispatcher _dispatcher; + private readonly Dispatcher _dispatcher; } } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Threading/DispatcherFrame.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Threading/DispatcherFrame.cs index dc8903b48c1..ac7dd9fa0d6 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Threading/DispatcherFrame.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Threading/DispatcherFrame.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. namespace System.Windows.Threading @@ -84,7 +84,7 @@ public bool Continue } } - private bool _exitWhenRequested; + private readonly bool _exitWhenRequested; private bool _continue; } } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Threading/DispatcherHookEventArgs.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Threading/DispatcherHookEventArgs.cs index dacd44893f7..a96a9a37a70 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Threading/DispatcherHookEventArgs.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Threading/DispatcherHookEventArgs.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. namespace System.Windows.Threading @@ -41,7 +41,7 @@ public DispatcherOperation Operation } } - private DispatcherOperation _operation; + private readonly DispatcherOperation _operation; } /// diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Threading/DispatcherOperation.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Threading/DispatcherOperation.cs index 826b9b6aaa6..bc20dfefd5e 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Threading/DispatcherOperation.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Threading/DispatcherOperation.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. using System.Threading; @@ -602,8 +602,8 @@ private void Exit() _operation.Completed -= new EventHandler(OnCompletedOrAborted); } - private DispatcherOperation _operation; - private Timer _waitTimer; + private readonly DispatcherOperation _operation; + private readonly Timer _waitTimer; } private class DispatcherOperationEvent @@ -670,9 +670,9 @@ private object DispatcherLock get { return _operation.DispatcherLock; } } - private DispatcherOperation _operation; - private TimeSpan _timeout; - private ManualResetEvent _event; + private readonly DispatcherOperation _operation; + private readonly TimeSpan _timeout; + private readonly ManualResetEvent _event; private bool _eventClosed; } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Threading/DispatcherSynchronizationContext.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Threading/DispatcherSynchronizationContext.cs index 3dcfdec7ffb..f2d4a5211b4 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Threading/DispatcherSynchronizationContext.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/Threading/DispatcherSynchronizationContext.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. using System.Threading; @@ -124,7 +124,7 @@ public override SynchronizationContext CreateCopy() internal Dispatcher _dispatcher; - private DispatcherPriority _priority; + private readonly DispatcherPriority _priority; } } diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/UncommonField.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/UncommonField.cs index ec0e56fb42e..97c869f18d9 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/UncommonField.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/UncommonField.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. using System.Runtime.CompilerServices; diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/WeakEventManager.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/WeakEventManager.cs index 12764610e41..087b119e8b7 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/WeakEventManager.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/WeakEventManager.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. // @@ -488,9 +488,9 @@ private WeakEventTable Table // Private Fields // - private WeakEventTable _table; + private readonly WeakEventTable _table; private static readonly object StaticSource = new NamedObject("StaticSource"); - private static MethodInfo s_DeliverEventMethodInfo; + private static readonly MethodInfo s_DeliverEventMethodInfo; #endregion Private Fields @@ -522,8 +522,8 @@ public bool Matches(object target, Delegate handler) public Delegate Handler { get { return (_handler != null) ? (Delegate)_handler.Target : null; } } public bool HasHandler { get { return _handler != null; } } - private WeakReference _target; - private WeakReference _handler; + private readonly WeakReference _target; + private readonly WeakReference _handler; } /// @@ -871,12 +871,12 @@ public void EndUse() Interlocked.Decrement(ref _users); } - private FrugalObjectList _list; // list of listeners + private readonly FrugalObjectList _list; // list of listeners private int _users; // number of active users - private System.Runtime.CompilerServices.ConditionalWeakTable + private readonly System.Runtime.CompilerServices.ConditionalWeakTable _cwt = new System.Runtime.CompilerServices.ConditionalWeakTable(); - private static ListenerList s_empty = new ListenerList(); + private static readonly ListenerList s_empty = new ListenerList(); } protected class ListenerList : ListenerList diff --git a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/WeakEventManagerT.cs b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/WeakEventManagerT.cs index 05ae02c7834..f7415ad5eb5 100644 --- a/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/WeakEventManagerT.cs +++ b/src/Microsoft.DotNet.Wpf/src/WindowsBase/System/Windows/WeakEventManagerT.cs @@ -1,4 +1,4 @@ -// Licensed to the .NET Foundation under one or more agreements. +// Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. // @@ -118,9 +118,9 @@ private static WeakEventManager CurrentManager(string #region Private Data - private Delegate _handler; - private string _eventName; - private EventInfo _eventInfo; + private readonly Delegate _handler; + private readonly string _eventName; + private readonly EventInfo _eventInfo; #endregion Private Data }