Merge pull request #14847 from tjwalkr3/warnings-5

Fix CA1051 and CA1815 warnings, Change public fields to auto properties
This commit is contained in:
Bond-009 2026-05-08 17:55:25 +02:00 committed by GitHub
commit de64a69c7a
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
4 changed files with 10 additions and 5 deletions

View file

@ -1,3 +1,5 @@
#pragma warning disable CA1815
namespace Emby.Naming.AudioBook
{
/// <summary>

View file

@ -1,3 +1,5 @@
#pragma warning disable CA1815
namespace Emby.Naming.Video
{
/// <summary>

View file

@ -40,11 +40,6 @@ namespace MediaBrowser.Controller.Net
/// </summary>
private readonly List<(IWebSocketConnection Connection, CancellationTokenSource CancellationTokenSource, TStateType State)> _activeConnections = new();
/// <summary>
/// The logger.
/// </summary>
protected readonly ILogger<BasePeriodicWebSocketListener<TReturnDataType, TStateType>> Logger;
private readonly Task _messageConsumerTask;
protected BasePeriodicWebSocketListener(ILogger<BasePeriodicWebSocketListener<TReturnDataType, TStateType>> logger)
@ -56,6 +51,11 @@ namespace MediaBrowser.Controller.Net
_messageConsumerTask = HandleMessages();
}
/// <summary>
/// Gets the Logger.
/// </summary>
protected ILogger<BasePeriodicWebSocketListener<TReturnDataType, TStateType>> Logger { get; }
/// <summary>
/// Gets the type used for the messages sent to the client.
/// </summary>

View file

@ -1,4 +1,5 @@
#pragma warning disable CS1591
#pragma warning disable CA1815
using System.Globalization;