Internaize utility classes, remove unused ones. (microsoft#161)

hongman · May 14, 2020 · 1d8a868 · 1d8a868
1 parent 20d5097
commit 1d8a868
Show file tree

Hide file tree

Showing 14 changed files with 12 additions and 349 deletions.
diff --git a/src/ReverseProxy/Abstractions/Time/IWallClockTimer.cs b/src/ReverseProxy/Abstractions/Time/IWallClockTimer.cs
diff --git a/src/ReverseProxy/EventIds.cs b/src/ReverseProxy/EventIds.cs
@@ -5,7 +5,7 @@
 
 namespace Microsoft.ReverseProxy
 {
-    public static class EventIds
+    internal static class EventIds
     {
         public static readonly EventId ApplyProxyConfig = new EventId(1, "ApplyProxyConfig");
         public static readonly EventId ApplyProxyConfigFailed = new EventId(2, "ApplyProxyConfigFailed");

diff --git a/src/ReverseProxy/Utilities/AsyncSemaphore.cs b/src/ReverseProxy/Utilities/AsyncSemaphore.cs
@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft Corporation.
+// Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
 
 using System.Collections.Generic;
@@ -12,7 +12,7 @@ namespace Microsoft.ReverseProxy.Utilities
     /// <remarks>
     /// Based on <c>https://blogs.msdn.microsoft.com/pfxteam/2012/02/12/building-async-coordination-primitives-part-5-asyncsemaphore/</c>.
     /// </remarks>
-    public sealed class AsyncSemaphore
+    internal sealed class AsyncSemaphore
     {
         private readonly Queue<TaskCompletionSource<bool>> _waiters = new Queue<TaskCompletionSource<bool>>();
         private int _count;

diff --git a/src/ReverseProxy/Utilities/Cache.cs b/src/ReverseProxy/Utilities/Cache.cs
diff --git a/src/ReverseProxy/Utilities/Contracts.cs b/src/ReverseProxy/Utilities/Contracts.cs
@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft Corporation.
+// Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
 
 using System;
@@ -9,7 +9,7 @@ namespace Microsoft.ReverseProxy.Utilities
     /// Simple argument validation helpers.
     /// </summary>
     // TODO: 6106627: find suitable replacement for CoreServicesBorrowed.Contracts
-    public static class Contracts
+    internal static class Contracts
     {
         /// <summary>
         /// Verifies that <paramref name="value"/> is not null, and throws

diff --git a/...seProxy/Extensions/ExceptionExtensions.cs → ...rseProxy/Utilities/ExceptionExtensions.cs b/...seProxy/Extensions/ExceptionExtensions.cs → ...rseProxy/Utilities/ExceptionExtensions.cs
@@ -14,7 +14,7 @@ namespace Microsoft.ReverseProxy.Utilities
     /// <summary>
     /// Extensions for the <see cref="Exception"/> class.
     /// </summary>
-    public static class ExceptionExtensions
+    internal static class ExceptionExtensions
     {
         /// <summary>
         /// Determines whether the provided <paramref name="exception"/> should be considered fatal. An exception

diff --git a/...ReverseProxy/Extensions/TaskExtensions.cs → src/ReverseProxy/Utilities/TaskExtensions.cs b/...ReverseProxy/Extensions/TaskExtensions.cs → src/ReverseProxy/Utilities/TaskExtensions.cs
@@ -14,7 +14,7 @@ namespace Microsoft.ReverseProxy.Utilities
     /// <summary>
     /// Extensions for the <see cref="Task"/> class.
     /// </summary>
-    public static class TaskExtensions
+    internal static class TaskExtensions
     {
         /// <summary>
         /// Runs an action on the current scheduler instead of the default scheduler.

diff --git a/src/ReverseProxy/Utilities/ThreadStaticRandom.cs b/src/ReverseProxy/Utilities/ThreadStaticRandom.cs
@@ -8,7 +8,7 @@ namespace Microsoft.ReverseProxy.Utilities
     /// <summary>
     /// Provides a thread static implementation of random numbers that optimizes not to lock on every invocation of random number generation.
     /// </summary>
-    public class ThreadStaticRandom
+    internal class ThreadStaticRandom
     {
         [ThreadStatic]
         private static Random t_inst;

diff --git a/src/ReverseProxy/Utilities/TimeSpanIso8601.cs b/src/ReverseProxy/Utilities/TimeSpanIso8601.cs
diff --git a/src/ReverseProxy/Utilities/TimeUtil.cs b/src/ReverseProxy/Utilities/TimeUtil.cs
@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft Corporation.
+// Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
 
 using System;
@@ -8,7 +8,7 @@ namespace Microsoft.ReverseProxy.Utilities
     /// <summary>
     /// Time and date utilities.
     /// </summary>
-    public static class TimeUtil
+    internal static class TimeUtil
     {
         /// <summary>
         /// This is the maximum <see cref="TimeSpan"/> that <see cref="System.Threading.CancellationTokenSource.CancelAfter(TimeSpan)"/> will accept.

diff --git a/src/ReverseProxy/Utilities/ValueStopwatch.cs b/src/ReverseProxy/Utilities/ValueStopwatch.cs
@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft Corporation.
+// Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
 
 using System;
@@ -12,7 +12,7 @@ namespace Microsoft.ReverseProxy.Utilities
     /// <remarks>
     /// Inspired on <seealso href="https://github.com/dotnet/extensions/blob/master/src/Shared/src/ValueStopwatch/ValueStopwatch.cs"/>.
     /// </remarks>
-    public struct ValueStopwatch
+    internal struct ValueStopwatch
     {
         private static readonly double _timestampToTicks = TimeSpan.TicksPerSecond / (double)Stopwatch.Frequency;
 

diff --git a/src/ReverseProxy/Utilities/WallClockTimer.cs b/src/ReverseProxy/Utilities/WallClockTimer.cs