Fix inspections

ppy · Apr 15, 2024 · 45e3395 · 45e3395
1 parent 0ce3c91
commit 45e3395
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/osu.Server.QueueProcessor/ConnectionMultiplexerExtensions.cs b/osu.Server.QueueProcessor/ConnectionMultiplexerExtensions.cs
@@ -34,7 +34,7 @@ public static void ClearCurrentSchema(this ConnectionMultiplexer connection)
         /// <summary>
         /// Get all active schemas (including past or future).
         /// </summary>
-        public static string[] GetActiveSchemas(this ConnectionMultiplexer connection)
+        public static string?[] GetActiveSchemas(this ConnectionMultiplexer connection)
         {
             return connection.GetDatabase().SetMembers(allActiveSchemasKey).ToStringArray();
         }
@@ -44,7 +44,7 @@ public static string[] GetActiveSchemas(this ConnectionMultiplexer connection)
         /// </summary>
         public static string GetCurrentSchema(this ConnectionMultiplexer connection)
         {
-            return connection.GetDatabase().StringGet(mainSchemaKey).ToString() ?? string.Empty;
+            return connection.GetDatabase().StringGet(mainSchemaKey).ToString();
         }
 
         /// <summary>

diff --git a/osu.Server.QueueProcessor/QueueProcessor.cs b/osu.Server.QueueProcessor/QueueProcessor.cs
@@ -107,6 +107,7 @@ public void Run(CancellationToken cancellation = default)
 
                             var redisItems = database.ListRightPop(QueueName, config.BatchSize);
 
+                            // ReSharper disable once ConditionIsAlwaysTrueOrFalseAccordingToNullableAPIContract (https://github.com/StackExchange/StackExchange.Redis/issues/2697)
                             // queue doesn't exist.
                             if (redisItems == null)
                             {