Seal/abstract/virtual everything (space-wizards#6739)

Author: mirrorcult

Content.Benchmarks/ColorInterpolateBenchmark.cs

@@ -12,7 +12,7 @@
 namespace Content.Benchmarks
 {
     [DisassemblyDiagnoser]
-    public class ColorInterpolateBenchmark
+    public sealed class ColorInterpolateBenchmark
     {
 #if NETCOREAPP
         private const MethodImplOptions AggressiveOpt = MethodImplOptions.AggressiveOptimization;

Content.Benchmarks/ComponentFetchBenchmark.cs

@@ -6,7 +6,7 @@
 namespace Content.Benchmarks
 {
     [SimpleJob]
-    public class ComponentFetchBenchmark
+    public sealed class ComponentFetchBenchmark
     {
         [Params(5000)] public int NEnt { get; set; }
 
@@ -218,39 +218,39 @@ private abstract class BComponent
             public EntityUid Uid;
         }
 
-        private class BComponent1 : BComponent
+        private sealed class BComponent1 : BComponent
         {
         }
 
-        private class BComponent2 : BComponent
+        private sealed class BComponent2 : BComponent
         {
         }
 
-        private class BComponent3 : BComponent
+        private sealed class BComponent3 : BComponent
         {
         }
 
-        private class BComponent4 : BComponent
+        private sealed class BComponent4 : BComponent
         {
         }
 
-        private class BComponentLookup : BComponent
+        private sealed class BComponentLookup : BComponent
         {
         }
 
-        private class BComponent6 : BComponent
+        private sealed class BComponent6 : BComponent
         {
         }
 
-        private class BComponent7 : BComponent
+        private sealed class BComponent7 : BComponent
         {
         }
 
-        private class BComponent8 : BComponent
+        private sealed class BComponent8 : BComponent
         {
         }
 
-        private class BComponent9 : BComponent
+        private sealed class BComponent9 : BComponent
         {
         }
     }

Content.Benchmarks/DependencyInjectBenchmark.cs

@@ -6,7 +6,7 @@ namespace Content.Benchmarks
 {
     // To actually run this benchmark you'll have to make DependencyCollection public so it's accessible.
 
-    public class DependencyInjectBenchmark
+    public sealed class DependencyInjectBenchmark
     {
         [Params(InjectMode.Reflection, InjectMode.DynamicMethod)]
         public InjectMode Mode { get; set; }

Content.Benchmarks/DynamicTreeBenchmark.cs

@@ -5,7 +5,7 @@
 namespace Content.Benchmarks
 {
     [SimpleJob, MemoryDiagnoser]
-    public class DynamicTreeBenchmark
+    public sealed class DynamicTreeBenchmark
     {
         private static readonly Box2[] _aabbs1 =
         {

Content.Benchmarks/EntityFetchBenchmark.cs

@@ -6,7 +6,7 @@
 namespace Content.Benchmarks
 {
     [SimpleJob]
-    public class EntityFetchBenchmark
+    public sealed class EntityFetchBenchmark
     {
         [Params(1000)] public int N { get; set; }
 

Content.Benchmarks/EntityManagerGetAllComponents.cs

@@ -10,7 +10,7 @@
 
 namespace Content.Benchmarks
 {
-    public class EntityManagerGetAllComponents
+    public sealed class EntityManagerGetAllComponents
     {
         private IEntityManager _entityManager;
 
@@ -87,7 +87,7 @@ public int Noop()
             return count;
         }
 
-        private class DummyComponent : Component
+        private sealed class DummyComponent : Component
         {
         }
     }

Content.Benchmarks/NetSerializerIntBenchmark.cs

@@ -6,7 +6,7 @@
 namespace Content.Benchmarks
 {
     [SimpleJob]
-    public class NetSerializerIntBenchmark
+    public sealed class NetSerializerIntBenchmark
     {
         private MemoryStream _writeStream;
         private MemoryStream _readStream;

Content.Benchmarks/NetSerializerStringBenchmark.cs

@@ -13,7 +13,7 @@ namespace Content.Benchmarks
     // Code for the *Slow and *Unsafe implementations taken from NetSerializer, licensed under the MIT license.
 
     [MemoryDiagnoser]
-    public class NetSerializerStringBenchmark
+    public sealed class NetSerializerStringBenchmark
     {
         private const int StringByteBufferLength = 256;
         private const int StringCharBufferLength = 128;

Content.Benchmarks/StereoToMonoBenchmark.cs

@@ -3,7 +3,7 @@
 
 namespace Content.Benchmarks
 {
-    public class StereoToMonoBenchmark
+    public sealed class StereoToMonoBenchmark
     {
         [Params(128, 256, 512)]
         public int N { get; set; }

Content.Client/AI/ClientAiDebugSystem.cs

@@ -13,7 +13,7 @@
 namespace Content.Client.AI
 {
 #if DEBUG
-    public class ClientAiDebugSystem : EntitySystem
+    public sealed class ClientAiDebugSystem : EntitySystem
     {
         [Dependency] private readonly IEyeManager _eyeManager = default!;
 

Content.Client/AI/ClientPathfindingDebugSystem.cs

@@ -16,7 +16,7 @@
 namespace Content.Client.AI
 {
 #if DEBUG
-    public class ClientPathfindingDebugSystem : EntitySystem
+    public sealed class ClientPathfindingDebugSystem : EntitySystem
     {
         [Dependency] private readonly IPrototypeManager _prototypeManager = default!;
         [Dependency] private readonly IEyeManager _eyeManager = default!;

Content.Client/AME/UI/AMEControllerBoundUserInterface.cs

@@ -7,7 +7,7 @@
 namespace Content.Client.AME.UI
 {
     [UsedImplicitly]
-    public class AMEControllerBoundUserInterface : BoundUserInterface
+    public sealed class AMEControllerBoundUserInterface : BoundUserInterface
     {
         private AMEWindow? _window;
 

Content.Client/AME/UI/AMEWindow.xaml.cs

@@ -12,7 +12,7 @@
 namespace Content.Client.AME.UI
 {
     [GenerateTypedNameReferences]
-    public partial class AMEWindow : DefaultWindow
+    public sealed partial class AMEWindow : DefaultWindow
     {
         public AMEWindow(AMEControllerBoundUserInterface ui)
         {

Content.Client/AME/Visualizers/AMEControllerVisualizer.cs

@@ -7,7 +7,7 @@
 namespace Content.Client.AME.Visualizers
 {
     [UsedImplicitly]
-    public class AMEControllerVisualizer : AppearanceVisualizer
+    public sealed class AMEControllerVisualizer : AppearanceVisualizer
     {
         public override void InitializeEntity(EntityUid entity)
         {

Content.Client/AME/Visualizers/AMEVisualizer.cs

@@ -7,7 +7,7 @@
 namespace Content.Client.AME.Visualizers
 {
     [UsedImplicitly]
-    public class AMEVisualizer : AppearanceVisualizer
+    public sealed class AMEVisualizer : AppearanceVisualizer
     {
         public override void InitializeEntity(EntityUid entity)
         {

Content.Client/Access/UI/IdCardConsoleBoundUserInterface.cs

@@ -7,7 +7,7 @@
 
 namespace Content.Client.Access.UI
 {
-    public class IdCardConsoleBoundUserInterface : BoundUserInterface
+    public sealed class IdCardConsoleBoundUserInterface : BoundUserInterface
     {
         [Dependency] private readonly IPrototypeManager _prototypeManager = default!;
         [Dependency] private readonly IEntityManager _entityManager = default!;

Content.Client/Access/UI/IdCardConsoleWindow.xaml.cs

@@ -13,7 +13,7 @@
 namespace Content.Client.Access.UI
 {
     [GenerateTypedNameReferences]
-    public partial class IdCardConsoleWindow : DefaultWindow
+    public sealed partial class IdCardConsoleWindow : DefaultWindow
     {
         private readonly IdCardConsoleBoundUserInterface _owner;
 

Content.Client/Actions/ActionsSystem.cs

@@ -12,7 +12,7 @@
 namespace Content.Client.Actions
 {
     [UsedImplicitly]
-    public class ActionsSystem : EntitySystem
+    public sealed class ActionsSystem : EntitySystem
     {
         [Dependency] private readonly IPlayerManager _playerManager = default!;
 

Content.Client/Actions/Assignments/ActionAssignments.cs

@@ -9,7 +9,7 @@ namespace Content.Client.Actions.Assignments
     /// <summary>
     /// Tracks and manages the hotbar assignments for actions.
     /// </summary>
-    public class ActionAssignments
+    public sealed class ActionAssignments
     {
         // the slots and assignments fields hold client's assignments (what action goes in what slot),
         // which are completely client side and independent of what actions they've actually been granted and

Content.Client/Actions/UI/ActionAlertTooltip.cs

@@ -11,7 +11,7 @@ namespace Content.Client.Actions.UI
     /// <summary>
     /// Tooltip for actions or alerts because they are very similar.
     /// </summary>
-    public class ActionAlertTooltip : PanelContainer
+    public sealed class ActionAlertTooltip : PanelContainer
     {
         private const float TooltipTextMaxWidth = 350;
 

Content.Client/Actions/UI/ActionMenu.cs

@@ -27,7 +27,7 @@ namespace Content.Client.Actions.UI
     /// Action selection menu, allows filtering and searching over all possible
     /// actions and populating those actions into the hotbar.
     /// </summary>
-    public class ActionMenu : DefaultWindow
+    public sealed class ActionMenu : DefaultWindow
     {
         private const string ItemTag = "item";
         private const string NotItemTag = "not item";

Content.Client/Actions/UI/ActionMenuItem.cs

@@ -10,7 +10,7 @@ namespace Content.Client.Actions.UI
     /// <summary>
     /// An individual action visible in the action menu.
     /// </summary>
-    public class ActionMenuItem : ContainerButton
+    public sealed class ActionMenuItem : ContainerButton
     {
         // shorter than default tooltip delay so user can
         // quickly explore what each action is

Content.Client/Actions/UI/ActionSlot.cs

@@ -25,7 +25,7 @@ namespace Content.Client.Actions.UI
     /// A slot in the action hotbar. Not extending BaseButton because
     /// its needs diverged too much.
     /// </summary>
-    public class ActionSlot : PanelContainer
+    public sealed class ActionSlot : PanelContainer
     {
         // shorter than default tooltip delay so user can more easily
         // see what actions they've been given

Content.Client/Administration/AdminNameOverlay.cs

@@ -7,7 +7,7 @@
 
 namespace Content.Client.Administration
 {
-    internal class AdminNameOverlay : Overlay
+    internal sealed class AdminNameOverlay : Overlay
     {
         private readonly AdminSystem _system;
         private readonly IEntityManager _entityManager;

Content.Client/Administration/AdminSystem.Menu.cs

@@ -19,7 +19,7 @@
 
 namespace Content.Client.Administration
 {
-    public partial class AdminSystem
+    public sealed partial class AdminSystem
     {
         [Dependency] private readonly INetManager _netManager = default!;
         [Dependency] private readonly IInputManager _inputManager = default!;

Content.Client/Administration/AdminSystem.Overlay.cs

@@ -5,7 +5,7 @@
 
 namespace Content.Client.Administration
 {
-    public partial class AdminSystem
+    public sealed partial class AdminSystem
     {
         [Dependency] private readonly IClientAdminManager _adminManager = default!;
         [Dependency] private readonly IEyeManager _eyeManager = default!;

Content.Client/Administration/AdminSystem.cs

@@ -9,7 +9,7 @@
 
 namespace Content.Client.Administration
 {
-    public partial class AdminSystem : EntitySystem
+    public sealed partial class AdminSystem : EntitySystem
     {
         public event Action<List<PlayerInfo>>? PlayerListChanged;
 

Content.Client/Administration/AdminVerbSystem.cs

@@ -9,7 +9,7 @@ namespace Content.Client.Verbs
     /// <summary>
     ///     Client-side admin verb system. These usually open some sort of UIs.
     /// </summary>
-    class AdminVerbSystem : EntitySystem
+    sealed class AdminVerbSystem : EntitySystem
     {
         [Dependency] private readonly IClientConGroupController _clientConGroupController = default!;
         [Dependency] private readonly IClientConsoleHost _clientConsoleHost = default!;

Content.Client/Administration/BwoinkSystem.cs

@@ -20,7 +20,7 @@
 namespace Content.Client.Administration
 {
     [UsedImplicitly]
-    public class BwoinkSystem : SharedBwoinkSystem
+    public sealed class BwoinkSystem : SharedBwoinkSystem
     {
         [Dependency] private readonly IClientAdminManager _adminManager = default!;
         [Dependency] private readonly IPlayerManager _playerManager = default!;

Content.Client/Administration/Logs/AdminLogSystem.cs

@@ -2,6 +2,6 @@
 
 namespace Content.Client.Administration.Logs;
 
-public class AdminLogSystem : SharedAdminLogSystem
+public sealed class AdminLogSystem : SharedAdminLogSystem
 {
 }

Content.Client/Administration/Managers/ClientAdminManager.cs

@@ -11,7 +11,7 @@
 
 namespace Content.Client.Administration.Managers
 {
-    public class ClientAdminManager : IClientAdminManager, IClientConGroupImplementation, IPostInjectInit
+    public sealed class ClientAdminManager : IClientAdminManager, IClientConGroupImplementation, IPostInjectInit
     {
         [Dependency] private readonly IClientNetManager _netMgr = default!;
         [Dependency] private readonly IClientConGroupController _conGroup = default!;

Content.Client/Administration/Managers/GamePrototypeLoadManager.cs

@@ -8,7 +8,7 @@
 
 namespace Content.Client.Administration.Managers;
 
-public class GamePrototypeLoadManager : IGamePrototypeLoadManager
+public sealed class GamePrototypeLoadManager : IGamePrototypeLoadManager
 {
     [Dependency] private readonly IClientNetManager _netManager = default!;
     [Dependency] private readonly IPrototypeManager _prototypeManager = default!;

Content.Client/Administration/UI/AdminAnnounceWindow.xaml.cs

@@ -10,7 +10,7 @@
 namespace Content.Client.Administration.UI
 {
     [GenerateTypedNameReferences]
-    public partial class AdminAnnounceWindow : DefaultWindow
+    public sealed partial class AdminAnnounceWindow : DefaultWindow
     {
         [Dependency] private readonly ILocalizationManager _localization = default!;
 

Content.Client/Administration/UI/AdminMenuWindow.xaml.cs

@@ -9,7 +9,7 @@
 namespace Content.Client.Administration.UI
 {
     [GenerateTypedNameReferences]
-    public partial class AdminMenuWindow : DefaultWindow
+    public sealed partial class AdminMenuWindow : DefaultWindow
     {
         [Dependency] private readonly IGameHud? _gameHud = default!;
 

Content.Client/Administration/UI/AdminMenuWindowEui.cs

@@ -4,7 +4,7 @@
 
 namespace Content.Client.Administration.UI
 {
-    public class AdminAnnounceEui : BaseEui
+    public sealed class AdminAnnounceEui : BaseEui
     {
         private readonly AdminAnnounceWindow _window;
 

Content.Client/Administration/UI/BwoinkWindow.xaml.cs

@@ -19,7 +19,7 @@ namespace Content.Client.Administration.UI
     /// This window connects to a BwoinkSystem channel. BwoinkSystem manages the rest.
     /// </summary>
     [GenerateTypedNameReferences]
-    public partial class BwoinkWindow : DefaultWindow
+    public sealed partial class BwoinkWindow : DefaultWindow
     {
         [Dependency] private readonly IClientAdminManager _adminManager = default!;
         [Dependency] private readonly IClientConsoleHost _console = default!;

Content.Client/Administration/UI/CustomControls/AdminLogImpactButton.cs

@@ -4,7 +4,7 @@
 
 namespace Content.Client.Administration.UI.CustomControls;
 
-public class AdminLogImpactButton : Button
+public sealed class AdminLogImpactButton : Button
 {
     public AdminLogImpactButton(LogImpact impact)
     {

Content.Client/Administration/UI/CustomControls/AdminLogLabel.cs

@@ -4,7 +4,7 @@
 
 namespace Content.Client.Administration.UI.CustomControls;
 
-public class AdminLogLabel : RichTextLabel
+public sealed class AdminLogLabel : RichTextLabel
 {
     public AdminLogLabel(ref SharedAdminLog log, HSeparator separator)
     {