Switch SubscribeAsync/UnsubscribeAsync to IEnumerable<string> (#2026)

The function currently requires an ICollection<string>, but doesn't actually depend on the ICollection<string> interface. Relax the requirement to be IEnumerable<string> for better flexibility.
dotnet · Jun 22, 2024 · b95d352 · b95d352
1 parent c5f4cf1
commit b95d352
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/Source/MQTTnet.Extensions.ManagedClient/IManagedMqttClient.cs b/Source/MQTTnet.Extensions.ManagedClient/IManagedMqttClient.cs
@@ -47,8 +47,8 @@ public interface IManagedMqttClient : IDisposable
 
         Task StopAsync(bool cleanDisconnect = true);
 
-        Task SubscribeAsync(ICollection<MqttTopicFilter> topicFilters);
+        Task SubscribeAsync(IEnumerable<MqttTopicFilter> topicFilters);
 
-        Task UnsubscribeAsync(ICollection<string> topics);
+        Task UnsubscribeAsync(IEnumerable<string> topics);
     }
 }
diff --git a/Source/MQTTnet.Extensions.ManagedClient/ManagedMqttClient.cs b/Source/MQTTnet.Extensions.ManagedClient/ManagedMqttClient.cs
@@ -288,7 +288,7 @@ public async Task StopAsync(bool cleanDisconnect = true)
             }
         }
 
-        public Task SubscribeAsync(ICollection<MqttTopicFilter> topicFilters)
+        public Task SubscribeAsync(IEnumerable<MqttTopicFilter> topicFilters)
         {
             ThrowIfDisposed();
 
@@ -316,7 +316,7 @@ public Task SubscribeAsync(ICollection<MqttTopicFilter> topicFilters)
             return CompletedTask.Instance;
         }
 
-        public Task UnsubscribeAsync(ICollection<string> topics)
+        public Task UnsubscribeAsync(IEnumerable<string> topics)
         {
             ThrowIfDisposed();