Housekeeping

src/Ryujinx.Graphics.Metal/Auto.cs

@@ -5,7 +5,7 @@ using System.Threading;
 
 namespace Ryujinx.Graphics.Metal
 {
-    public interface IAuto
+    interface IAuto
     {
         bool HasCommandBufferDependency(CommandBufferScoped cbs);
 
@@ -14,13 +14,13 @@ namespace Ryujinx.Graphics.Metal
         void DecrementReferenceCount();
     }
 
-    public interface IAutoPrivate : IAuto
+    interface IAutoPrivate : IAuto
     {
         void AddCommandBufferDependencies(CommandBufferScoped cbs);
     }
 
     [SupportedOSPlatform("macos")]
-    public class Auto<T> : IAutoPrivate, IDisposable where T : IDisposable
+    class Auto<T> : IAutoPrivate, IDisposable where T : IDisposable
     {
         private int _referenceCount;
         private T _value;

src/Ryujinx.Graphics.Metal/BackgroundResources.cs

@@ -26,7 +26,7 @@ namespace Ryujinx.Graphics.Metal
             if (_pool == null)
             {
                 MTLCommandQueue queue = _renderer.BackgroundQueue;
-                _pool = new CommandBufferPool(queue.Device, queue);
+                _pool = new CommandBufferPool(queue);
             }
 
             return _pool;

src/Ryujinx.Graphics.Metal/BufferHolder.cs

@@ -8,7 +8,7 @@ using System.Threading;
 namespace Ryujinx.Graphics.Metal
 {
     [SupportedOSPlatform("macos")]
-    public class BufferHolder : IDisposable
+    class BufferHolder : IDisposable
     {
         private CacheByRange<BufferHolder> _cachedConvertedBuffers;
 

src/Ryujinx.Graphics.Metal/BufferManager.cs

@@ -8,7 +8,7 @@ using System.Runtime.Versioning;
 
 namespace Ryujinx.Graphics.Metal
 {
-    public readonly struct ScopedTemporaryBuffer : IDisposable
+    readonly struct ScopedTemporaryBuffer : IDisposable
     {
         private readonly BufferManager _bufferManager;
         private readonly bool _isReserved;
@@ -39,7 +39,7 @@ namespace Ryujinx.Graphics.Metal
     }
 
     [SupportedOSPlatform("macos")]
-    public class BufferManager : IDisposable
+    class BufferManager : IDisposable
     {
         private readonly IdList<BufferHolder> _buffers;
 

src/Ryujinx.Graphics.Metal/CacheByRange.cs

@@ -1,11 +1,10 @@
-using SharpMetal.Metal;
 using System;
 using System.Collections.Generic;
 using System.Runtime.Versioning;
 
 namespace Ryujinx.Graphics.Metal
 {
-    public interface ICacheKey : IDisposable
+    interface ICacheKey : IDisposable
     {
         bool KeyEqual(ICacheKey other);
     }
@@ -116,7 +115,7 @@ namespace Ryujinx.Graphics.Metal
     }
 
     [SupportedOSPlatform("macos")]
-    public readonly struct Dependency
+    readonly struct Dependency
     {
         private readonly BufferHolder _buffer;
         private readonly int _offset;

src/Ryujinx.Graphics.Metal/CommandBufferPool.cs

@@ -7,14 +7,12 @@ using System.Runtime.Versioning;
 namespace Ryujinx.Graphics.Metal
 {
     [SupportedOSPlatform("macos")]
-    public class CommandBufferPool : IDisposable
+    class CommandBufferPool : IDisposable
     {
         public const int MaxCommandBuffers = 16;
 
         private readonly int _totalCommandBuffers;
         private readonly int _totalCommandBuffersMask;
-
-        private readonly MTLDevice _device;
         private readonly MTLCommandQueue _queue;
 
         [SupportedOSPlatform("macos")]
@@ -45,9 +43,8 @@ namespace Ryujinx.Graphics.Metal
         private int _queuedCount;
         private int _inUseCount;
 
-        public CommandBufferPool(MTLDevice device, MTLCommandQueue queue)
+        public CommandBufferPool(MTLCommandQueue queue)
         {
-            _device = device;
             _queue = queue;
 
             _totalCommandBuffers = MaxCommandBuffers;

src/Ryujinx.Graphics.Metal/CommandBufferScoped.cs

@@ -5,7 +5,7 @@ using System.Runtime.Versioning;
 namespace Ryujinx.Graphics.Metal
 {
     [SupportedOSPlatform("macos")]
-    public readonly struct CommandBufferScoped : IDisposable
+    readonly struct CommandBufferScoped : IDisposable
     {
         private readonly CommandBufferPool _pool;
         public MTLCommandBuffer CommandBuffer { get; }

src/Ryujinx.Graphics.Metal/EncoderState.cs

@@ -5,7 +5,7 @@ using System.Runtime.Versioning;
 
 namespace Ryujinx.Graphics.Metal
 {
-    public struct DirtyFlags
+    struct DirtyFlags
     {
         public bool RenderPipeline = false;
         public bool ComputePipeline = false;
@@ -21,7 +21,7 @@ namespace Ryujinx.Graphics.Metal
         }
     }
 
-    public record struct BufferRef
+    record struct BufferRef
     {
         public Auto<DisposableBuffer> Buffer;
         public int Index;

src/Ryujinx.Graphics.Metal/HelperShader.cs

@@ -10,7 +10,7 @@ using System.Runtime.Versioning;
 namespace Ryujinx.Graphics.Metal
 {
     [SupportedOSPlatform("macos")]
-    public class HelperShader : IDisposable
+    class HelperShader : IDisposable
     {
         private const int ConvertElementsPerWorkgroup = 32 * 100; // Work group size of 32 times 100 elements.
         private const string ShadersSourcePath = "/Ryujinx.Graphics.Metal/Shaders";

src/Ryujinx.Graphics.Metal/MetalRenderer.cs

@@ -21,7 +21,6 @@ namespace Ryujinx.Graphics.Metal
 
         public event EventHandler<ScreenCaptureImageInfo> ScreenCaptured;
         public bool PreferThreading => true;
-
         public IPipeline Pipeline => _pipeline;
         public IWindow Window => _window;
 
@@ -54,7 +53,7 @@ namespace Ryujinx.Graphics.Metal
             layer.Device = _device;
             layer.FramebufferOnly = false;
 
-            CommandBufferPool = new CommandBufferPool(_device, _queue);
+            CommandBufferPool = new CommandBufferPool(_queue);
             _window = new Window(this, layer);
             _pipeline = new Pipeline(_device, this);
             BufferManager = new BufferManager(_device, this, _pipeline);

src/Ryujinx.Graphics.Metal/MultiFenceHolder.cs

@@ -8,7 +8,7 @@ namespace Ryujinx.Graphics.Metal
     /// Holder for multiple host GPU fences.
     /// </summary>
     [SupportedOSPlatform("macos")]
-    public class MultiFenceHolder
+    class MultiFenceHolder
     {
         private const int BufferUsageTrackingGranularity = 4096;
 

src/Ryujinx.Graphics.Metal/Pipeline.cs

@@ -41,7 +41,7 @@ namespace Ryujinx.Graphics.Metal
             CommandBuffer = (Cbs = _renderer.CommandBufferPool.Rent()).CommandBuffer;
         }
 
-        public void InitEncoderStateManager(BufferManager bufferManager)
+        internal void InitEncoderStateManager(BufferManager bufferManager)
         {
             _encoderStateManager = new EncoderStateManager(_device, bufferManager, this);
         }
@@ -142,7 +142,7 @@ namespace Ryujinx.Graphics.Metal
             }
         }
 
-        public MTLRenderCommandEncoder BeginRenderPass()
+        private MTLRenderCommandEncoder BeginRenderPass()
         {
             EndCurrentPass();
 
@@ -154,7 +154,7 @@ namespace Ryujinx.Graphics.Metal
             return renderCommandEncoder;
         }
 
-        public MTLBlitCommandEncoder BeginBlitPass()
+        private MTLBlitCommandEncoder BeginBlitPass()
         {
             EndCurrentPass();
 
@@ -166,7 +166,7 @@ namespace Ryujinx.Graphics.Metal
             return blitCommandEncoder;
         }
 
-        public MTLComputeCommandEncoder BeginComputePass()
+        private MTLComputeCommandEncoder BeginComputePass()
         {
             EndCurrentPass();
 
@@ -536,7 +536,7 @@ namespace Ryujinx.Graphics.Metal
             _encoderStateManager.UpdateStorageBuffers(buffers);
         }
 
-        public void SetStorageBuffers(int first, ReadOnlySpan<Auto<DisposableBuffer>> buffers)
+        internal void SetStorageBuffers(int first, ReadOnlySpan<Auto<DisposableBuffer>> buffers)
         {
             _encoderStateManager.UpdateStorageBuffers(first, buffers);
         }

src/Ryujinx.Graphics.Metal/StagingBuffer.cs

@@ -8,7 +8,7 @@ using System.Runtime.Versioning;
 
 namespace Ryujinx.Graphics.Metal
 {
-    public readonly struct StagingBufferReserved
+    readonly struct StagingBufferReserved
     {
         public readonly BufferHolder Buffer;
         public readonly int Offset;
@@ -23,7 +23,7 @@ namespace Ryujinx.Graphics.Metal
     }
 
     [SupportedOSPlatform("macos")]
-    public class StagingBuffer : IDisposable
+    class StagingBuffer : IDisposable
     {
         private const int BufferSize = 32 * 1024 * 1024;