Skip to content

Commit 27f527e

Browse files
stephentoubtarekghteo-tsirpanis
authored
Seal a few more extensions types with only a private ctor (#117368)
* Seal a few more extensions types with only a private ctor * Apply suggestions from code review Co-authored-by: Theodore Tsirpanis <[email protected]> --------- Co-authored-by: Tarek Mahmoud Sayed <[email protected]> Co-authored-by: Theodore Tsirpanis <[email protected]>
1 parent 10a0619 commit 27f527e

File tree

4 files changed

+4
-4
lines changed

4 files changed

+4
-4
lines changed

src/libraries/Microsoft.Extensions.FileProviders.Abstractions/ref/Microsoft.Extensions.FileProviders.Abstractions.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ public NotFoundFileInfo(string name) { }
4646
[System.Diagnostics.CodeAnalysis.DoesNotReturn]
4747
public System.IO.Stream CreateReadStream() { throw null; }
4848
}
49-
public partial class NullChangeToken : Microsoft.Extensions.Primitives.IChangeToken
49+
public sealed partial class NullChangeToken : Microsoft.Extensions.Primitives.IChangeToken
5050
{
5151
internal NullChangeToken() { }
5252
public bool ActiveChangeCallbacks { get { throw null; } }

src/libraries/Microsoft.Extensions.FileProviders.Abstractions/src/NullChangeToken.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ namespace Microsoft.Extensions.FileProviders
99
/// <summary>
1010
/// An empty change token that doesn't raise any change callbacks.
1111
/// </summary>
12-
public class NullChangeToken : IChangeToken
12+
public sealed class NullChangeToken : IChangeToken
1313
{
1414
/// <summary>
1515
/// Gets a singleton instance of <see cref="NullChangeToken"/>.

src/libraries/Microsoft.Extensions.Logging.Abstractions/ref/Microsoft.Extensions.Logging.Abstractions.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -193,7 +193,7 @@ public void AddProvider(Microsoft.Extensions.Logging.ILoggerProvider provider) {
193193
public Microsoft.Extensions.Logging.ILogger CreateLogger(string name) { throw null; }
194194
public void Dispose() { }
195195
}
196-
public partial class NullLoggerProvider : Microsoft.Extensions.Logging.ILoggerProvider, System.IDisposable
196+
public sealed partial class NullLoggerProvider : Microsoft.Extensions.Logging.ILoggerProvider, System.IDisposable
197197
{
198198
internal NullLoggerProvider() { }
199199
public static Microsoft.Extensions.Logging.Abstractions.NullLoggerProvider Instance { get { throw null; } }

src/libraries/Microsoft.Extensions.Logging.Abstractions/src/NullLoggerProvider.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ namespace Microsoft.Extensions.Logging.Abstractions
66
/// <summary>
77
/// Provider for the <see cref="NullLogger"/>.
88
/// </summary>
9-
public class NullLoggerProvider : ILoggerProvider
9+
public sealed class NullLoggerProvider : ILoggerProvider
1010
{
1111
/// <summary>
1212
/// Returns an instance of <see cref="NullLoggerProvider"/>.

0 commit comments

Comments
 (0)