From 0e30e9088fbb4b80dd244a4c49944c1bbfefa3f0 Mon Sep 17 00:00:00 2001
From: Lucian Wischik <lu@wischik.com>
Date: Wed, 31 Aug 2016 09:32:31 -0700
Subject: [PATCH] Followed convention of "(object)t != null" for types

---
 .../AsyncMethodBuilderMemberCollection.cs     | 19 +++++++++----------
 1 file changed, 9 insertions(+), 10 deletions(-)

diff --git a/src/Compilers/CSharp/Portable/Lowering/AsyncRewriter/AsyncMethodBuilderMemberCollection.cs b/src/Compilers/CSharp/Portable/Lowering/AsyncRewriter/AsyncMethodBuilderMemberCollection.cs
index d05d1164deb71..155e9ed445b54 100644
--- a/src/Compilers/CSharp/Portable/Lowering/AsyncRewriter/AsyncMethodBuilderMemberCollection.cs
+++ b/src/Compilers/CSharp/Portable/Lowering/AsyncRewriter/AsyncMethodBuilderMemberCollection.cs
@@ -145,7 +145,7 @@ internal static bool TryCreate(SyntheticBoundNodeFactory F, MethodSymbol method,
                 if (customBuilder)
                 {
                     builderType = ValidateBuilderType(F, builderArgument, returnType.DeclaredAccessibility, desiredArity:0);
-                    if (builderType != null)
+                    if ((object)builderType != null)
                     {
                         taskProperty = GetCustomTaskProperty(F, builderType);
                         createBuilderMethod = GetCustomCreateMethod(F, builderType);
@@ -168,7 +168,8 @@ internal static bool TryCreate(SyntheticBoundNodeFactory F, MethodSymbol method,
                         customBuilder,
                         out taskProperty);
                 }
-                if ((object)createBuilderMethod == null ||
+                if ((object)builderType == null ||
+                    (object)createBuilderMethod == null ||
                     (object)taskProperty == null)
                 {
                     collection = default(AsyncMethodBuilderMemberCollection);
@@ -209,16 +210,10 @@ internal static bool TryCreate(SyntheticBoundNodeFactory F, MethodSymbol method,
 
                 object builderArgument;
                 bool customBuilder = returnType.IsCustomTaskType(out builderArgument);
-                if (!customBuilder)
-                {
-                    builderType = F.WellKnownType(WellKnownType.System_Runtime_CompilerServices_AsyncTaskMethodBuilder_T);
-                    Debug.Assert((object)builderType != null);
-                    builderType = builderType.Construct(resultType);
-                }
                 if (customBuilder)
                 {
                     builderType = ValidateBuilderType(F, builderArgument, returnType.DeclaredAccessibility, 1);
-                    if (builderType != null)
+                    if ((object)builderType != null)
                     {
                         builderType = builderType.ConstructedFrom.Construct(resultType);
                         taskProperty = GetCustomTaskProperty(F, builderType);
@@ -227,6 +222,9 @@ internal static bool TryCreate(SyntheticBoundNodeFactory F, MethodSymbol method,
                 }
                 else
                 {
+                    builderType = F.WellKnownType(WellKnownType.System_Runtime_CompilerServices_AsyncTaskMethodBuilder_T);
+                    Debug.Assert((object)builderType != null);
+                    builderType = builderType.Construct(resultType);
                     TryGetBuilderMember<MethodSymbol>(
                         F,
                         WellKnownMember.System_Runtime_CompilerServices_AsyncTaskMethodBuilder_T__Create,
@@ -240,7 +238,8 @@ internal static bool TryCreate(SyntheticBoundNodeFactory F, MethodSymbol method,
                         customBuilder,
                         out taskProperty);
                 }
-                if ((object)taskProperty == null ||
+                if ((object)builderType == null ||
+                    (object)taskProperty == null ||
                     (object)createBuilderMethod == null)
                 {
                     collection = default(AsyncMethodBuilderMemberCollection);