Cleaned up code.

mtamme · Jan 4, 2018 · ee0b197 · ee0b197
1 parent 94973c9
commit ee0b197
Show file tree

Hide file tree

Showing 5 changed files with 18 additions and 19 deletions.
diff --git a/...e/Internal/Caching/AnonymousDisposable.cs → ...Proxy.Core/Internal/Caching/Disposable.cs b/...e/Internal/Caching/AnonymousDisposable.cs → ...Proxy.Core/Internal/Caching/Disposable.cs
@@ -19,25 +19,25 @@
 namespace NProxy.Core.Internal.Caching
 {
     /// <summary>
-    /// Represents an anonymous disposable.
+    /// Represents a disposable.
     /// </summary>
-    internal sealed class AnonymousDisposable : IDisposable
+    internal sealed class Disposable : IDisposable
     {
         /// <summary>
         /// The dispose action.
         /// </summary>
         private readonly Action<bool> _dispose;
 
         /// <summary>
-        /// A value indicating whether this <see cref="AnonymousDisposable"/> was already disposed.
+        /// A value indicating whether this <see cref="Disposable"/> was already disposed.
         /// </summary>
         private bool _disposed;
 
         /// <summary>
-        /// Initializes a new instance of the <see cref="AnonymousDisposable"/> class.
+        /// Initializes a new instance of the <see cref="Disposable"/> class.
         /// </summary>
         /// <param name="dispose">The dispose action.</param>
-        public AnonymousDisposable(Action<bool> dispose)
+        public Disposable(Action<bool> dispose)
         {
             if (dispose == null)
                 throw new ArgumentNullException("dispose");
@@ -49,15 +49,15 @@ public AnonymousDisposable(Action<bool> dispose)
 
         /// <summary>
         /// Releases unmanaged resources and performs other cleanup operations before the
-        /// <see cref="AnonymousDisposable"/> is reclaimed by garbage collection.
+        /// <see cref="Disposable"/> is reclaimed by garbage collection.
         /// </summary>
-        ~AnonymousDisposable()
+        ~Disposable()
         {
             Dispose(false);
         }
 
         /// <summary>
-        /// Dispose this <see cref="AnonymousDisposable"/>.
+        /// Dispose this <see cref="Disposable"/>.
         /// </summary>
         /// <param name="disposing">A value indicating whether disposing is in progress.</param>
         private void Dispose(bool disposing)

diff --git a/Source/Main/NProxy.Core/Internal/Caching/ReadWriteLock.cs b/Source/Main/NProxy.Core/Internal/Caching/ReadWriteLock.cs
@@ -54,7 +54,7 @@ public ReadWriteLock()
         }
 
         /// <summary>
-        /// Dispose this <see cref="AnonymousDisposable"/>.
+        /// Dispose this <see cref="ReadWriteLock"/>.
         /// </summary>
         /// <param name="disposing">A value indicating whether disposing is in progress.</param>
         private void Dispose(bool disposing)
@@ -76,7 +76,7 @@ public IDisposable UpgradeableRead()
         {
             _lock.EnterUpgradeableReadLock();
 
-            return new AnonymousDisposable(_ => _lock.ExitUpgradeableReadLock());
+            return new Disposable(_ => _lock.ExitUpgradeableReadLock());
         }
 
         /// <summary>
@@ -87,7 +87,7 @@ public IDisposable Read()
         {
             _lock.EnterReadLock();
 
-            return new AnonymousDisposable(_ => _lock.ExitReadLock());
+            return new Disposable(_ => _lock.ExitReadLock());
         }
 
         /// <summary>
@@ -98,7 +98,7 @@ public IDisposable Write()
         {
             _lock.EnterWriteLock();
 
-            return new AnonymousDisposable(_ => _lock.ExitWriteLock());
+            return new Disposable(_ => _lock.ExitWriteLock());
         }
 
         #region IDisposable Members

diff --git a/Source/Main/NProxy.Core/Internal/Reflection/EventInfoExtensions.cs b/Source/Main/NProxy.Core/Internal/Reflection/EventInfoExtensions.cs
@@ -57,7 +57,6 @@ public static IEnumerable<MethodInfo> GetMethods(this EventInfo eventInfo)
                 eventInfo.GetAddMethod(true),
                 eventInfo.GetRemoveMethod(true)
             };
-
             var raiseMethodInfo = eventInfo.GetRaiseMethod(true);
 
             if (raiseMethodInfo != null)

diff --git a/Source/Main/NProxy.Core/NProxy.Core.csproj b/Source/Main/NProxy.Core/NProxy.Core.csproj
@@ -73,7 +73,7 @@
     <Compile Include="Interceptors\InvocationContext.cs" />
     <Compile Include="Interceptors\ProxyFactoryExtensions.cs" />
     <Compile Include="Interceptors\DynamicTargetInterceptor.cs" />
-    <Compile Include="Internal\Caching\AnonymousDisposable.cs" />
+    <Compile Include="Internal\Caching\Disposable.cs" />
     <Compile Include="Internal\Caching\ReadWriteLock.cs" />
     <Compile Include="Interceptors\MixinInvocationHandler.cs" />
     <Compile Include="Interceptors\Language\IFluent.cs" />

diff --git a/Source/Main/NProxy.Core/ProxyGenerator.cs b/Source/Main/NProxy.Core/ProxyGenerator.cs
@@ -95,19 +95,19 @@ public IProxyTemplate GenerateProxyTemplate(IProxyDefinition proxyDefinition)
         #region IProxyDefinitionVisitor Members
 
         /// <inheritdoc/>
-        public void VisitInterface(Type interfaceType)
+        void IProxyDefinitionVisitor.VisitInterface(Type interfaceType)
         {
             _typeBuilder.AddInterface(interfaceType);
         }
 
         /// <inheritdoc/>
-        public void VisitConstructor(ConstructorInfo constructorInfo)
+        void IProxyDefinitionVisitor.VisitConstructor(ConstructorInfo constructorInfo)
         {
             _typeBuilder.BuildConstructor(constructorInfo);
         }
 
         /// <inheritdoc/>
-        public void VisitEvent(EventInfo eventInfo)
+        void IProxyDefinitionVisitor.VisitEvent(EventInfo eventInfo)
         {
             if (_typeBuilder.IsConcreteEvent(eventInfo) && !_interceptionFilter.AcceptEvent(eventInfo))
                 return;
@@ -117,7 +117,7 @@ public void VisitEvent(EventInfo eventInfo)
         }
 
         /// <inheritdoc/>
-        public void VisitProperty(PropertyInfo propertyInfo)
+        void IProxyDefinitionVisitor.VisitProperty(PropertyInfo propertyInfo)
         {
             if (_typeBuilder.IsConcreteProperty(propertyInfo) && !_interceptionFilter.AcceptProperty(propertyInfo))
                 return;
@@ -127,7 +127,7 @@ public void VisitProperty(PropertyInfo propertyInfo)
         }
 
         /// <inheritdoc/>
-        public void VisitMethod(MethodInfo methodInfo)
+        void IProxyDefinitionVisitor.VisitMethod(MethodInfo methodInfo)
         {
             if (_typeBuilder.IsConcreteMethod(methodInfo) && !_interceptionFilter.AcceptMethod(methodInfo))
                 return;