Make classes sealed (#15675)

xtqqczze · web-flow · commit 4c2d6dcfca29 · 2021-07-04T16:49:52.000+05:00
Fix [RCS1225](https://github.com/JosefPihrt/Roslynator/blob/master/docs/analyzers/RCS1225.md)
diff --git a/src/Microsoft.PowerShell.Commands.Management/cimSupport/cmdletization/cim/TerminatingErrorTracker.cs b/src/Microsoft.PowerShell.Commands.Management/cimSupport/cmdletization/cim/TerminatingErrorTracker.cs
@@ -19,7 +19,7 @@ namespace Microsoft.PowerShell.Cmdletization.Cim
     /// <summary>
     /// Tracks (per-session) terminating errors in a given cmdlet invocation.
     /// </summary>
-    internal class TerminatingErrorTracker
+    internal sealed class TerminatingErrorTracker
     {
         #region Getting tracker for a given cmdlet invocation
 
diff --git a/src/Microsoft.PowerShell.Commands.Management/cimSupport/cmdletization/cim/cimOperationOptionsHelper.cs b/src/Microsoft.PowerShell.Commands.Management/cimSupport/cmdletization/cim/cimOperationOptionsHelper.cs
@@ -14,7 +14,7 @@
 
 namespace Microsoft.PowerShell.Cmdletization.Cim
 {
-    internal class CimCustomOptionsDictionary
+    internal sealed class CimCustomOptionsDictionary
     {
         private readonly IDictionary<string, object> _dict;
         private readonly object _dictModificationLock = new();
diff --git a/src/System.Management.Automation/CoreCLR/CorePsAssemblyLoadContext.cs b/src/System.Management.Automation/CoreCLR/CorePsAssemblyLoadContext.cs
@@ -15,7 +15,7 @@ namespace System.Management.Automation
     /// <summary>
     /// The powershell custom AssemblyLoadContext implementation.
     /// </summary>
-    internal partial class PowerShellAssemblyLoadContext
+    internal sealed partial class PowerShellAssemblyLoadContext
     {
         #region Resource_Strings
 
diff --git a/src/System.Management.Automation/FormatAndOutput/common/FormatXMLWriter.cs b/src/System.Management.Automation/FormatAndOutput/common/FormatXMLWriter.cs
@@ -13,7 +13,7 @@ namespace Microsoft.PowerShell.Commands
     /// <summary>
     /// Helper class for writing formatting directives to XML.
     /// </summary>
-    internal class FormatXmlWriter
+    internal sealed class FormatXmlWriter
     {
         private XmlWriter _writer;
         private bool _exportScriptBlock;
diff --git a/src/System.Management.Automation/engine/CommandInfo.cs b/src/System.Management.Automation/engine/CommandInfo.cs
@@ -928,7 +928,7 @@ public PSMemberNameAndType(string name, PSTypeName typeName, object value = null
     /// but can be used where a real type might not be available, in which case the name of the type can be used.
     /// The type encodes the members of dynamic objects in the type name.
     /// </summary>
-    internal class PSSyntheticTypeName : PSTypeName
+    internal sealed class PSSyntheticTypeName : PSTypeName
     {
         internal static PSSyntheticTypeName Create(string typename, IList<PSMemberNameAndType> membersTypes) => Create(new PSTypeName(typename), membersTypes);
 
diff --git a/src/System.Management.Automation/engine/Modules/AnalysisCache.cs b/src/System.Management.Automation/engine/Modules/AnalysisCache.cs
@@ -642,7 +642,7 @@ private static bool GetModuleEntryFromCache(string modulePath, out DateTime last
         }
     }
 
-    internal class AnalysisCacheData
+    internal sealed class AnalysisCacheData
     {
         private static byte[] GetHeader()
         {
diff --git a/src/System.Management.Automation/engine/MshSnapinQualifiedName.cs b/src/System.Management.Automation/engine/MshSnapinQualifiedName.cs
@@ -10,7 +10,7 @@ namespace System.Management.Automation
     /// <summary>
     /// A class representing a name that is qualified by the PSSnapin name.
     /// </summary>
-    internal class PSSnapinQualifiedName
+    internal sealed class PSSnapinQualifiedName
     {
         private PSSnapinQualifiedName(string[] splitName)
         {
diff --git a/src/System.Management.Automation/engine/interpreter/PowerShellInstructions.cs b/src/System.Management.Automation/engine/interpreter/PowerShellInstructions.cs
@@ -15,7 +15,7 @@
 
 namespace System.Management.Automation.Interpreter
 {
-    internal class UpdatePositionInstruction : Instruction
+    internal sealed class UpdatePositionInstruction : Instruction
     {
         private readonly int _sequencePoint;
         private readonly bool _checkBreakpoints;
diff --git a/src/System.Management.Automation/engine/parser/SafeValues.cs b/src/System.Management.Automation/engine/parser/SafeValues.cs
@@ -356,7 +356,7 @@ public object VisitParenExpression(ParenExpressionAst parenExpressionAst)
      * except in the case of handling the unary operator
      * ExecutionContext is provided to ensure we can resolve variables
      */
-    internal class GetSafeValueVisitor : ICustomAstVisitor2
+    internal sealed class GetSafeValueVisitor : ICustomAstVisitor2
     {
         internal enum SafeValueContext
         {
diff --git a/src/System.Management.Automation/engine/parser/SemanticChecks.cs b/src/System.Management.Automation/engine/parser/SemanticChecks.cs
@@ -17,7 +17,7 @@
 
 namespace System.Management.Automation.Language
 {
-    internal class SemanticChecks : AstVisitor2, IAstPostVisitHandler
+    internal sealed class SemanticChecks : AstVisitor2, IAstPostVisitHandler
     {
         private readonly Parser _parser;
 
diff --git a/src/System.Management.Automation/engine/parser/SymbolResolver.cs b/src/System.Management.Automation/engine/parser/SymbolResolver.cs
@@ -275,7 +275,7 @@ public bool IsInMethodScope()
         }
     }
 
-    internal class SymbolResolver : AstVisitor2, IAstPostVisitHandler
+    internal sealed class SymbolResolver : AstVisitor2, IAstPostVisitHandler
     {
         private readonly SymbolResolvePostActionVisitor _symbolResolvePostActionVisitor;
         internal readonly SymbolTable _symbolTable;
diff --git a/src/System.Management.Automation/engine/parser/VariableAnalysis.cs b/src/System.Management.Automation/engine/parser/VariableAnalysis.cs
@@ -41,7 +41,7 @@ internal VariableAnalysisDetails()
         public List<Ast> AssociatedAsts { get; }
     }
 
-    internal class FindAllVariablesVisitor : AstVisitor
+    internal sealed class FindAllVariablesVisitor : AstVisitor
     {
         private static readonly HashSet<string> s_hashOfPessimizingCmdlets = new HashSet<string>(StringComparer.OrdinalIgnoreCase);
 
diff --git a/src/System.Management.Automation/engine/remoting/client/ClientMethodExecutor.cs b/src/System.Management.Automation/engine/remoting/client/ClientMethodExecutor.cs
@@ -13,7 +13,7 @@ namespace System.Management.Automation.Remoting
     /// <summary>
     /// Executes methods on the client.
     /// </summary>
-    internal class ClientMethodExecutor
+    internal sealed class ClientMethodExecutor
     {
         /// <summary>
         /// Transport manager.
diff --git a/src/System.Management.Automation/engine/remoting/common/WireDataFormat/RemoteDebuggingCapability.cs b/src/System.Management.Automation/engine/remoting/common/WireDataFormat/RemoteDebuggingCapability.cs
@@ -13,7 +13,7 @@ namespace System.Management.Automation.Remoting
     /// version on the server. These capabilities will be used in remote debugging sessions to
     /// determine what is supported by the server.
     /// </summary>
-    internal class RemoteDebuggingCapability
+    internal sealed class RemoteDebuggingCapability
     {
         private readonly HashSet<string> _supportedCommands = new HashSet<string>();
 
diff --git a/src/System.Management.Automation/engine/remoting/common/WireDataFormat/RemoteSessionCapability.cs b/src/System.Management.Automation/engine/remoting/common/WireDataFormat/RemoteSessionCapability.cs
@@ -171,7 +171,7 @@ internal enum HostDefaultDataId
     /// <summary>
     /// The HostDefaultData class.
     /// </summary>
-    internal class HostDefaultData
+    internal sealed class HostDefaultData
     {
         /// <summary>
         /// Data.
diff --git a/src/System.Management.Automation/engine/remoting/common/WireDataFormat/RemotingDataObject.cs b/src/System.Management.Automation/engine/remoting/common/WireDataFormat/RemotingDataObject.cs
@@ -267,7 +267,7 @@ private static Guid DeserializeGuid(Stream serializedDataStream)
         #endregion
     }
 
-    internal class RemoteDataObject : RemoteDataObject<object>
+    internal sealed class RemoteDataObject : RemoteDataObject<object>
     {
         #region Constructors / Factory
 
diff --git a/src/System.Management.Automation/engine/remoting/server/ServerSteppablePipelineDriver.cs b/src/System.Management.Automation/engine/remoting/server/ServerSteppablePipelineDriver.cs
@@ -14,7 +14,7 @@ namespace System.Management.Automation
     /// <summary>
     /// Execution context used for stepping.
     /// </summary>
-    internal class ExecutionContextForStepping : IDisposable
+    internal sealed class ExecutionContextForStepping : IDisposable
     {
         private readonly ExecutionContext _executionContext;
         private PSInformationalBuffers _originalInformationalBuffers;
diff --git a/src/System.Management.Automation/engine/runtime/Binding/Binders.cs b/src/System.Management.Automation/engine/runtime/Binding/Binders.cs
@@ -513,7 +513,7 @@ internal static BindingRestrictions GetOptionalVersionAndLanguageCheckForType(Dy
     /// The standard interop ConvertBinder is used to allow third party dynamic objects to get the first chance
     /// at the conversion in case they do support enumeration, but do not implement IEnumerable directly.
     /// </summary>
-    internal class PSEnumerableBinder : ConvertBinder
+    internal sealed class PSEnumerableBinder : ConvertBinder
     {
         private static readonly PSEnumerableBinder s_binder = new PSEnumerableBinder();
 
@@ -780,7 +780,7 @@ private static IEnumerator PSObjectStringRule(CallSite site, object obj)
     /// <summary>
     /// This binder is used for the @() operator.
     /// </summary>
-    internal class PSToObjectArrayBinder : DynamicMetaObjectBinder
+    internal sealed class PSToObjectArrayBinder : DynamicMetaObjectBinder
     {
         private static readonly PSToObjectArrayBinder s_binder = new PSToObjectArrayBinder();
 
@@ -836,7 +836,7 @@ public override DynamicMetaObject Bind(DynamicMetaObject target, DynamicMetaObje
         }
     }
 
-    internal class PSPipeWriterBinder : DynamicMetaObjectBinder
+    internal sealed class PSPipeWriterBinder : DynamicMetaObjectBinder
     {
         private static readonly PSPipeWriterBinder s_binder = new PSPipeWriterBinder();
 
@@ -930,7 +930,7 @@ private static void AutomationNullRule(CallSite site, object obj, Pipe pipe, Exe
     /// The target in this binder is the RHS, the result expression is an IList where the Count matches the
     /// number of values assigned (_elements) on the left hand side of the assign.
     /// </summary>
-    internal class PSArrayAssignmentRHSBinder : DynamicMetaObjectBinder
+    internal sealed class PSArrayAssignmentRHSBinder : DynamicMetaObjectBinder
     {
         private static readonly List<PSArrayAssignmentRHSBinder> s_binders = new List<PSArrayAssignmentRHSBinder>();
         private readonly int _elements;
@@ -1049,7 +1049,7 @@ public override DynamicMetaObject Bind(DynamicMetaObject target, DynamicMetaObje
     ///       case, the format string is generated by the parser, so we know that there is no custom
     ///       formatting to consider.
     /// </summary>
-    internal class PSToStringBinder : DynamicMetaObjectBinder
+    internal sealed class PSToStringBinder : DynamicMetaObjectBinder
     {
         private static readonly PSToStringBinder s_binder = new PSToStringBinder();
 
@@ -1114,7 +1114,7 @@ internal static Expression InvokeToString(Expression context, Expression target)
     /// <summary>
     /// This binder is used to optimize the conversion of the result.
     /// </summary>
-    internal class PSPipelineResultToBoolBinder : DynamicMetaObjectBinder
+    internal sealed class PSPipelineResultToBoolBinder : DynamicMetaObjectBinder
     {
         private static readonly PSPipelineResultToBoolBinder s_binder = new PSPipelineResultToBoolBinder();
 
@@ -1175,7 +1175,7 @@ public override DynamicMetaObject Bind(DynamicMetaObject target, DynamicMetaObje
         }
     }
 
-    internal class PSInvokeDynamicMemberBinder : DynamicMetaObjectBinder
+    internal sealed class PSInvokeDynamicMemberBinder : DynamicMetaObjectBinder
     {
         private class KeyComparer : IEqualityComparer<PSInvokeDynamicMemberBinderKeyType>
         {
@@ -1286,7 +1286,7 @@ public int GetHashCode(PSGetOrSetDynamicMemberBinderKeyType obj)
         }
     }
 
-    internal class PSGetDynamicMemberBinder : DynamicMetaObjectBinder
+    internal sealed class PSGetDynamicMemberBinder : DynamicMetaObjectBinder
     {
         private static readonly Dictionary<PSGetOrSetDynamicMemberBinderKeyType, PSGetDynamicMemberBinder> s_binderCache =
             new Dictionary<PSGetOrSetDynamicMemberBinderKeyType, PSGetDynamicMemberBinder>(new PSDynamicGetOrSetBinderKeyComparer());
@@ -1396,7 +1396,7 @@ internal static object GetIDictionaryMember(IDictionary hash, object key)
         }
     }
 
-    internal class PSSetDynamicMemberBinder : DynamicMetaObjectBinder
+    internal sealed class PSSetDynamicMemberBinder : DynamicMetaObjectBinder
     {
         private static readonly Dictionary<PSGetOrSetDynamicMemberBinderKeyType, PSSetDynamicMemberBinder> s_binderCache =
             new Dictionary<PSGetOrSetDynamicMemberBinderKeyType, PSSetDynamicMemberBinder>(new PSDynamicGetOrSetBinderKeyComparer());
@@ -1488,7 +1488,7 @@ public override DynamicMetaObject Bind(DynamicMetaObject target, DynamicMetaObje
         }
     }
 
-    internal class PSSwitchClauseEvalBinder : DynamicMetaObjectBinder
+    internal sealed class PSSwitchClauseEvalBinder : DynamicMetaObjectBinder
     {
         // Increase this cache size if we add a new flag to the switch statement that:
         //    - Influences evaluation of switch elements
@@ -1608,7 +1608,7 @@ public override DynamicMetaObject Bind(DynamicMetaObject target, DynamicMetaObje
 
     // This class implements the standard binder CreateInstanceBinder, but this binder handles the CallInfo a little differently.
     // The ArgumentNames are not used to invoke a constructor, instead they are used to set properties/fields in the attribute.
-    internal class PSAttributeGenerator : CreateInstanceBinder
+    internal sealed class PSAttributeGenerator : CreateInstanceBinder
     {
         private static readonly Dictionary<CallInfo, PSAttributeGenerator> s_binderCache =
             new Dictionary<CallInfo, PSAttributeGenerator>();
@@ -1789,7 +1789,7 @@ public override DynamicMetaObject FallbackCreateInstance(DynamicMetaObject targe
         }
     }
 
-    internal class PSCustomObjectConverter : DynamicMetaObjectBinder
+    internal sealed class PSCustomObjectConverter : DynamicMetaObjectBinder
     {
         private static readonly PSCustomObjectConverter s_binder = new PSCustomObjectConverter();
 
@@ -1825,7 +1825,7 @@ public override DynamicMetaObject Bind(DynamicMetaObject target, DynamicMetaObje
         }
     }
 
-    internal class PSDynamicConvertBinder : DynamicMetaObjectBinder
+    internal sealed class PSDynamicConvertBinder : DynamicMetaObjectBinder
     {
         private static readonly PSDynamicConvertBinder s_binder = new PSDynamicConvertBinder();
 
@@ -1863,7 +1863,7 @@ public override DynamicMetaObject Bind(DynamicMetaObject target, DynamicMetaObje
     /// <summary>
     /// This binder is used to copy mutable value types when assigning to variables, otherwise just assigning the target object directly.
     /// </summary>
-    internal class PSVariableAssignmentBinder : DynamicMetaObjectBinder
+    internal sealed class PSVariableAssignmentBinder : DynamicMetaObjectBinder
     {
         private static readonly PSVariableAssignmentBinder s_binder = new PSVariableAssignmentBinder();
         internal static int s_mutableValueWithInstanceMemberVersion;
@@ -2116,7 +2116,7 @@ internal static BindingRestrictions GetVersionCheck(int expectedVersionNumber)
     /// <summary>
     /// The binder for common binary operators.  PowerShell specific binary operators are handled elsewhere.
     /// </summary>
-    internal class PSBinaryOperationBinder : BinaryOperationBinder
+    internal sealed class PSBinaryOperationBinder : BinaryOperationBinder
     {
         #region Constructors and factory methods
 
@@ -3411,7 +3411,7 @@ private DynamicMetaObject BinaryComparisonCommon(DynamicMetaObject targetAsEnume
     /// <summary>
     /// The binder for unary operators like !, -, or +.
     /// </summary>
-    internal class PSUnaryOperationBinder : UnaryOperationBinder
+    internal sealed class PSUnaryOperationBinder : UnaryOperationBinder
     {
         private static PSUnaryOperationBinder s_notBinder;
         private static PSUnaryOperationBinder s_bnotBinder;
@@ -3727,7 +3727,7 @@ private DynamicMetaObject IncrDecr(DynamicMetaObject target, int valueToAdd, Dyn
     /// <summary>
     /// The binder for converting a value, e.g. [int]"42"
     /// </summary>
-    internal class PSConvertBinder : ConvertBinder
+    internal sealed class PSConvertBinder : ConvertBinder
     {
         private static readonly Dictionary<Type, PSConvertBinder> s_binderCache = new Dictionary<Type, PSConvertBinder>();
         internal int _version;
@@ -3926,7 +3926,7 @@ private static string StringToStringRule(CallSite site, object obj)
     /// <summary>
     /// The binder to get the value of an indexable object, e.g. $x[1]
     /// </summary>
-    internal class PSGetIndexBinder : GetIndexBinder
+    internal sealed class PSGetIndexBinder : GetIndexBinder
     {
         private static readonly Dictionary<Tuple<CallInfo, PSMethodInvocationConstraints, bool>, PSGetIndexBinder> s_binderCache
                 = new Dictionary<Tuple<CallInfo, PSMethodInvocationConstraints, bool>, PSGetIndexBinder>();
@@ -4517,7 +4517,7 @@ private Func<object, object, object> GetNonSlicingIndexer()
     /// <summary>
     /// The binder for setting the value of an indexable element, like $x[1] = 5.
     /// </summary>
-    internal class PSSetIndexBinder : SetIndexBinder
+    internal sealed class PSSetIndexBinder : SetIndexBinder
     {
         private static readonly Dictionary<Tuple<CallInfo, PSMethodInvocationConstraints>, PSSetIndexBinder> s_binderCache
                 = new Dictionary<Tuple<CallInfo, PSMethodInvocationConstraints>, PSSetIndexBinder>();
@@ -6508,7 +6508,7 @@ public override DynamicMetaObject FallbackInvoke(DynamicMetaObject target, Dynam
         }
     }
 
-    internal class PSInvokeMemberBinder : InvokeMemberBinder
+    internal sealed class PSInvokeMemberBinder : InvokeMemberBinder
     {
         internal enum MethodInvocationType
         {
diff --git a/src/System.Management.Automation/engine/runtime/ScriptBlockToPowerShell.cs b/src/System.Management.Automation/engine/runtime/ScriptBlockToPowerShell.cs
@@ -172,7 +172,7 @@ internal static void ThrowError(ScriptBlockToPowerShellNotSupportedException ex,
         }
     }
 
-    internal class UsingExpressionAstSearcher : AstSearcher
+    internal sealed class UsingExpressionAstSearcher : AstSearcher
     {
         internal static IEnumerable<Ast> FindAllUsingExpressions(Ast ast)
         {
@@ -207,7 +207,7 @@ public override AstVisitAction VisitFunctionDefinition(FunctionDefinitionAst ast
     /// Converts a ScriptBlock to a PowerShell object by traversing the
     /// given Ast.
     /// </summary>
-    internal class ScriptBlockToPowerShellConverter
+    internal sealed class ScriptBlockToPowerShellConverter
     {
         private readonly PowerShell _powershell;
         private ExecutionContext _context;
diff --git a/src/System.Management.Automation/help/AliasHelpInfo.cs b/src/System.Management.Automation/help/AliasHelpInfo.cs
@@ -8,7 +8,7 @@ namespace System.Management.Automation
     /// <summary>
     /// Stores help information related to Alias Commands.
     /// </summary>
-    internal class AliasHelpInfo : HelpInfo
+    internal sealed class AliasHelpInfo : HelpInfo
     {
         /// <summary>
         /// Initializes a new instance of the AliasHelpInfo class.
diff --git a/src/System.Management.Automation/help/CommandHelpProvider.cs b/src/System.Management.Automation/help/CommandHelpProvider.cs
@@ -1359,7 +1359,7 @@ internal virtual CommandSearcher GetCommandSearcherForSearch(string pattern, Exe
     /// Legally, user-defined Help Data should be within the same file as the corresponding
     /// commandHelp and it should appear after the commandHelp.
     /// </summary>
-    internal class UserDefinedHelpData
+    internal sealed class UserDefinedHelpData
     {
         private UserDefinedHelpData()
         {
diff --git a/src/System.Management.Automation/help/HelpCommentsParser.cs b/src/System.Management.Automation/help/HelpCommentsParser.cs
@@ -20,7 +20,7 @@ namespace System.Management.Automation
     /// <summary>
     /// Parses help comments and turns them into HelpInfo objects.
     /// </summary>
-    internal class HelpCommentsParser
+    internal sealed class HelpCommentsParser
     {
         private HelpCommentsParser()
         {
diff --git a/src/System.Management.Automation/help/HelpFileHelpInfo.cs b/src/System.Management.Automation/help/HelpFileHelpInfo.cs
@@ -9,7 +9,7 @@ namespace System.Management.Automation
     /// Class HelpFileHelpInfo keeps track of help information to be returned by
     /// command help provider.
     /// </summary>
-    internal class HelpFileHelpInfo : HelpInfo
+    internal sealed class HelpFileHelpInfo : HelpInfo
     {
         /// <summary>
         /// Constructor for HelpFileHelpInfo.
diff --git a/src/System.Management.Automation/help/MUIFileSearcher.cs b/src/System.Management.Automation/help/MUIFileSearcher.cs
@@ -10,7 +10,7 @@
 
 namespace System.Management.Automation
 {
-    internal class MUIFileSearcher
+    internal sealed class MUIFileSearcher
     {
         /// <summary>
         /// Constructor. It is private so that MUIFileSearcher is used only internal for this class.
diff --git a/src/System.Management.Automation/help/ProviderHelpInfo.cs b/src/System.Management.Automation/help/ProviderHelpInfo.cs
diff --git a/src/System.Management.Automation/help/SyntaxHelpInfo.cs b/src/System.Management.Automation/help/SyntaxHelpInfo.cs
diff --git a/src/System.Management.Automation/utils/CryptoUtils.cs b/src/System.Management.Automation/utils/CryptoUtils.cs
diff --git a/src/System.Management.Automation/utils/GraphicalHostReflectionWrapper.cs b/src/System.Management.Automation/utils/GraphicalHostReflectionWrapper.cs

Original file line number	Diff line number	Diff line change
`@@ -19,7 +19,7 @@ namespace Microsoft.PowerShell.Cmdletization.Cim`
`19`	`19`	`/// <summary>`
`20`	`20`	`/// Tracks (per-session) terminating errors in a given cmdlet invocation.`
`21`	`21`	`/// </summary>`
`22`		`- internal class TerminatingErrorTracker`
	`22`	`+ internal sealed class TerminatingErrorTracker`
`23`	`23`	`{`
`24`	`24`	`#region Getting tracker for a given cmdlet invocation`
`25`	`25`
Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,7 @@`
`14`	`14`
`15`	`15`	`namespace Microsoft.PowerShell.Cmdletization.Cim`
`16`	`16`	`{`
`17`		`- internal class CimCustomOptionsDictionary`
	`17`	`+ internal sealed class CimCustomOptionsDictionary`
`18`	`18`	`{`
`19`	`19`	`private readonly IDictionary<string, object> _dict;`
`20`	`20`	`private readonly object _dictModificationLock = new();`
Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,7 @@ namespace System.Management.Automation`
`15`	`15`	`/// <summary>`
`16`	`16`	`/// The powershell custom AssemblyLoadContext implementation.`
`17`	`17`	`/// </summary>`
`18`		`- internal partial class PowerShellAssemblyLoadContext`
	`18`	`+ internal sealed partial class PowerShellAssemblyLoadContext`
`19`	`19`	`{`
`20`	`20`	`#region Resource_Strings`
`21`	`21`
Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,7 @@ namespace Microsoft.PowerShell.Commands`
`13`	`13`	`/// <summary>`
`14`	`14`	`/// Helper class for writing formatting directives to XML.`
`15`	`15`	`/// </summary>`
`16`		`- internal class FormatXmlWriter`
	`16`	`+ internal sealed class FormatXmlWriter`
`17`	`17`	`{`
`18`	`18`	`private XmlWriter _writer;`
`19`	`19`	`private bool _exportScriptBlock;`
Original file line number	Diff line number	Diff line change
`@@ -928,7 +928,7 @@ public PSMemberNameAndType(string name, PSTypeName typeName, object value = null`
`928`	`928`	`/// but can be used where a real type might not be available, in which case the name of the type can be used.`
`929`	`929`	`/// The type encodes the members of dynamic objects in the type name.`
`930`	`930`	`/// </summary>`
`931`		`- internal class PSSyntheticTypeName : PSTypeName`
	`931`	`+ internal sealed class PSSyntheticTypeName : PSTypeName`
`932`	`932`	`{`
`933`	`933`	`internal static PSSyntheticTypeName Create(string typename, IList<PSMemberNameAndType> membersTypes) => Create(new PSTypeName(typename), membersTypes);`
`934`	`934`
Original file line number	Diff line number	Diff line change
`@@ -642,7 +642,7 @@ private static bool GetModuleEntryFromCache(string modulePath, out DateTime last`
`642`	`642`	`}`
`643`	`643`	`}`
`644`	`644`
`645`		`- internal class AnalysisCacheData`
	`645`	`+ internal sealed class AnalysisCacheData`
`646`	`646`	`{`
`647`	`647`	`private static byte[] GetHeader()`
`648`	`648`	`{`
Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@ namespace System.Management.Automation`
`10`	`10`	`/// <summary>`
`11`	`11`	`/// A class representing a name that is qualified by the PSSnapin name.`
`12`	`12`	`/// </summary>`
`13`		`- internal class PSSnapinQualifiedName`
	`13`	`+ internal sealed class PSSnapinQualifiedName`
`14`	`14`	`{`
`15`	`15`	`private PSSnapinQualifiedName(string[] splitName)`
`16`	`16`	`{`
Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,7 @@`
`15`	`15`
`16`	`16`	`namespace System.Management.Automation.Interpreter`
`17`	`17`	`{`
`18`		`- internal class UpdatePositionInstruction : Instruction`
	`18`	`+ internal sealed class UpdatePositionInstruction : Instruction`
`19`	`19`	`{`
`20`	`20`	`private readonly int _sequencePoint;`
`21`	`21`	`private readonly bool _checkBreakpoints;`
Original file line number	Diff line number	Diff line change
`@@ -356,7 +356,7 @@ public object VisitParenExpression(ParenExpressionAst parenExpressionAst)`
`356`	`356`	`* except in the case of handling the unary operator`
`357`	`357`	`* ExecutionContext is provided to ensure we can resolve variables`
`358`	`358`	`*/`
`359`		`- internal class GetSafeValueVisitor : ICustomAstVisitor2`
	`359`	`+ internal sealed class GetSafeValueVisitor : ICustomAstVisitor2`
`360`	`360`	`{`
`361`	`361`	`internal enum SafeValueContext`
`362`	`362`	`{`
Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@`
`17`	`17`
`18`	`18`	`namespace System.Management.Automation.Language`
`19`	`19`	`{`
`20`		`- internal class SemanticChecks : AstVisitor2, IAstPostVisitHandler`
	`20`	`+ internal sealed class SemanticChecks : AstVisitor2, IAstPostVisitHandler`
`21`	`21`	`{`
`22`	`22`	`private readonly Parser _parser;`
`23`	`23`