infra: chore: fix/silence compile warnings

Ryubing · Dec 24, 2024 · 2f540dc · 2f540dc
1 parent 3cb996b
commit 2f540dc
Show file tree

Hide file tree

Showing 5 changed files with 8 additions and 2 deletions.
diff --git a/src/Ryujinx.Graphics.Gpu/Shader/ShaderCache.cs b/src/Ryujinx.Graphics.Gpu/Shader/ShaderCache.cs
@@ -838,6 +838,7 @@ private static TranslationOptions CreateTranslationOptions(TargetApi api, Transl
                 TargetApi.OpenGL => TargetLanguage.Glsl,
                 TargetApi.Vulkan => GraphicsConfig.EnableSpirvCompilationOnVulkan ? TargetLanguage.Spirv : TargetLanguage.Glsl,
                 TargetApi.Metal => TargetLanguage.Msl,
+                _ => throw new NotImplementedException()
             };
 
             return new TranslationOptions(lang, api, flags);

diff --git a/src/Ryujinx.Graphics.Metal/EncoderStateManager.cs b/src/Ryujinx.Graphics.Metal/EncoderStateManager.cs
@@ -1767,6 +1767,7 @@ private static uint SetIndexToBindingIndex(uint setIndex)
                 Constants.StorageBuffersSetIndex => Constants.StorageBuffersIndex,
                 Constants.TexturesSetIndex => Constants.TexturesIndex,
                 Constants.ImagesSetIndex => Constants.ImagesIndex,
+                _ => throw new NotImplementedException()
             };
         }
 

diff --git a/src/Ryujinx.Graphics.Metal/MetalRenderer.cs b/src/Ryujinx.Graphics.Metal/MetalRenderer.cs
@@ -21,9 +21,12 @@ public sealed class MetalRenderer : IRenderer
         private Pipeline _pipeline;
         private Window _window;
 
-        public uint ProgramCount { get; set; } = 0;
+        public uint ProgramCount { get; set; }
 
+#pragma warning disable CS0067 // The event is never used
         public event EventHandler<ScreenCaptureImageInfo> ScreenCaptured;
+#pragma warning restore CS0067
+
         public bool PreferThreading => true;
         public IPipeline Pipeline => _pipeline;
         public IWindow Window => _window;

diff --git a/src/Ryujinx.Graphics.Shader/CodeGen/Msl/Instructions/InstGen.cs b/src/Ryujinx.Graphics.Shader/CodeGen/Msl/Instructions/InstGen.cs
@@ -125,6 +125,7 @@ private static string GetExpression(CodeGenContext context, AstOperation operati
                                 Instruction.Add => "PreciseFAdd",
                                 Instruction.Subtract => "PreciseFSub",
                                 Instruction.Multiply => "PreciseFMul",
+                                _ => throw new NotImplementedException()
                             };
 
                             return $"{func}({expr[0]}, {expr[1]})";

diff --git a/src/Ryujinx/AppHost.cs b/src/Ryujinx/AppHost.cs
@@ -917,7 +917,7 @@ private void InitializeSwitchInstance()
 
             if (ShouldInitMetal)
             {
-#pragma warning disable CA1416 This call site is reachable on all platforms
+#pragma warning disable CA1416 // This call site is reachable on all platforms
                 // The condition does a check for Mac, on top of checking if it's an ARM Mac. This isn't a problem.
                 renderer = new MetalRenderer((RendererHost.EmbeddedWindow as EmbeddedWindowMetal)!.CreateSurface);
 #pragma warning restore CA1416