Remove redundant is check in It

... because `Match` matchers will already have checked `value is T` by the time the predicate delegate gets invoked.
devlooped · Aug 29, 2019 · 3fbed15 · 3fbed15
1 parent ca229eb
commit 3fbed15
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/src/Moq/It.cs b/src/Moq/It.cs
@@ -50,8 +50,7 @@ public static TValue IsAny<TValue>()
 		{
 			return Match<TValue>.Create(
 #if FEATURE_COM
-				value => value == null || (typeof(TValue).IsAssignableFrom(value.GetType())
-				                           || (IsComObject(value) && value is TValue)),
+				value => value == null || (typeof(TValue).IsAssignableFrom(value.GetType()) || IsComObject(value)),
 #else
 				value => value == null || typeof(TValue).IsAssignableFrom(value.GetType()),
 #endif
@@ -73,8 +72,7 @@ public static TValue IsNotNull<TValue>()
 		{
 			return Match<TValue>.Create(
 #if FEATURE_COM
-				value => value != null && (typeof(TValue).IsAssignableFrom(value.GetType())
-				                           || (IsComObject(value) && value is TValue)),
+				value => value != null && (typeof(TValue).IsAssignableFrom(value.GetType()) || IsComObject(value)),
 #else
 				value => value != null && typeof(TValue).IsAssignableFrom(value.GetType()),
 #endif