Enable nullable throughout solution (#2102)

Author: mgravell

perf/benchmarkapps/QpsWorker/Infrastructure/ClientRunner.cs

@@ -304,7 +304,7 @@ private Task RunClientAsync(GrpcChannel channel, IInterarrivalTimer timer)
             return RunGenericStreamingAsync(channel, timer);
         }
 
-        GrpcPreconditions.CheckNotNull(_payloadConfig.SimpleParams);
+        GrpcPreconditions.CheckNotNull(_payloadConfig.SimpleParams, "SimpleParams");
         if (_clientType == ClientType.SyncClient)
         {
             GrpcPreconditions.CheckArgument(_rpcType == RpcType.Unary, "Sync client can only be used for Unary calls in C#");
@@ -326,7 +326,7 @@ private Task RunClientAsync(GrpcChannel channel, IInterarrivalTimer timer)
 
     private SimpleRequest CreateSimpleRequest()
     {
-        GrpcPreconditions.CheckNotNull(_payloadConfig.SimpleParams);
+        GrpcPreconditions.CheckNotNull(_payloadConfig.SimpleParams, "SimpleParams");
         return new SimpleRequest
         {
             Payload = CreateZerosPayload(_payloadConfig.SimpleParams.ReqSize),

src/Grpc.Auth/GoogleAuthInterceptors.cs

@@ -16,9 +16,6 @@
 
 #endregion
 
-using System.Threading;
-using System.Threading.Tasks;
-
 using Google.Apis.Auth.OAuth2;
 using Grpc.Core;
 using Grpc.Core.Utils;
@@ -83,7 +80,7 @@ public static AsyncAuthInterceptor FromCredential(ITokenAccessWithHeaders creden
     /// <returns>The interceptor.</returns>
     public static AsyncAuthInterceptor FromAccessToken(string accessToken)
     {
-        GrpcPreconditions.CheckNotNull(accessToken);
+        GrpcPreconditions.CheckNotNull(accessToken, nameof(accessToken));
         return new AsyncAuthInterceptor((context, metadata) =>
         {
             metadata.Add(CreateBearerTokenHeader(accessToken));

src/Grpc.Core.Api/AsyncAuthInterceptor.cs

@@ -16,11 +16,7 @@
 
 #endregion
 
-using System;
-using System.Collections.Generic;
 using System.Threading.Tasks;
-
-using Grpc.Core.Internal;
 using Grpc.Core.Utils;
 
 namespace Grpc.Core;
@@ -46,8 +42,8 @@ public class AuthInterceptorContext
     /// </summary>
     public AuthInterceptorContext(string serviceUrl, string methodName)
     {
-        this.serviceUrl = GrpcPreconditions.CheckNotNull(serviceUrl);
-        this.methodName = GrpcPreconditions.CheckNotNull(methodName);
+        this.serviceUrl = GrpcPreconditions.CheckNotNull(serviceUrl, nameof(serviceUrl));
+        this.methodName = GrpcPreconditions.CheckNotNull(methodName, nameof(methodName));
     }
 
     /// <summary>

src/Grpc.Core.Api/AuthContext.cs

@@ -16,7 +16,6 @@
 
 #endregion
 
-using System;
 using System.Collections.Generic;
 using System.Linq;
 using Grpc.Core.Utils;
@@ -31,8 +30,8 @@ namespace Grpc.Core;
 /// </summary>
 public class AuthContext
 {
-    string? peerIdentityPropertyName;
-    Dictionary<string, List<AuthProperty>> properties;
+    private readonly string? peerIdentityPropertyName;
+    private readonly Dictionary<string, List<AuthProperty>> properties;
 
     /// <summary>
     /// Initializes a new instance of the <see cref="T:Grpc.Core.AuthContext"/> class.
@@ -42,7 +41,7 @@ public class AuthContext
     public AuthContext(string? peerIdentityPropertyName, Dictionary<string, List<AuthProperty>> properties)
     {
         this.peerIdentityPropertyName = peerIdentityPropertyName;
-        this.properties = GrpcPreconditions.CheckNotNull(properties);
+        this.properties = GrpcPreconditions.CheckNotNull(properties, nameof(properties));
     }
 
     /// <summary>
@@ -101,8 +100,7 @@ public IEnumerable<AuthProperty> Properties
     /// </summary>
     public IEnumerable<AuthProperty> FindPropertiesByName(string propertyName)
     {
-        List<AuthProperty>? result;
-        if (!properties.TryGetValue(propertyName, out result))
+        if (!properties.TryGetValue(propertyName, out var result))
         {
             return Enumerable.Empty<AuthProperty>();
         }

src/Grpc.Core.Api/AuthProperty.cs

@@ -29,14 +29,15 @@ namespace Grpc.Core;
 public class AuthProperty
 {
     static readonly Encoding EncodingUTF8 = System.Text.Encoding.UTF8;
-    string name;
-    byte[] valueBytes;
-    string? lazyValue;
+
+    private readonly string name;
+    private readonly byte[] valueBytes;
+    private string? lazyValue;
 
     private AuthProperty(string name, byte[] valueBytes)
     {
-        this.name = GrpcPreconditions.CheckNotNull(name);
-        this.valueBytes = GrpcPreconditions.CheckNotNull(valueBytes);
+        this.name = GrpcPreconditions.CheckNotNull(name, nameof(name));
+        this.valueBytes = GrpcPreconditions.CheckNotNull(valueBytes, nameof(valueBytes));
     }
 
     /// <summary>
@@ -57,7 +58,7 @@ public string Value
     {
         get
         {
-            return lazyValue ?? (lazyValue = EncodingUTF8.GetString(this.valueBytes));
+            return lazyValue ??= EncodingUTF8.GetString(this.valueBytes);
         }
     }
 
@@ -81,7 +82,7 @@ public byte[] ValueBytes
     /// <param name="valueBytes">the binary value of the property</param>
     public static AuthProperty Create(string name, byte[] valueBytes)
     {
-        GrpcPreconditions.CheckNotNull(valueBytes);
+        GrpcPreconditions.CheckNotNull(valueBytes, nameof(valueBytes));
         var valueCopy = new byte[valueBytes.Length];
         Buffer.BlockCopy(valueBytes, 0, valueCopy, 0, valueBytes.Length);
         return new AuthProperty(name, valueCopy);

src/Grpc.Core.Api/CallCredentials.cs

@@ -17,9 +17,6 @@
 #endregion
 
 using System.Collections.Generic;
-using System.Collections.ObjectModel;
-
-using Grpc.Core.Internal;
 using Grpc.Core.Utils;
 
 namespace Grpc.Core;
@@ -78,7 +75,7 @@ private class AsyncAuthInterceptorCredentials : CallCredentials
 
         public AsyncAuthInterceptorCredentials(AsyncAuthInterceptor interceptor)
         {
-            this.interceptor = GrpcPreconditions.CheckNotNull(interceptor);
+            this.interceptor = GrpcPreconditions.CheckNotNull(interceptor, nameof(interceptor));
         }
 
         public override void InternalPopulateConfiguration(CallCredentialsConfiguratorBase configurator, object? state)

src/Grpc.Core.Api/ChannelCredentials.cs

@@ -16,11 +16,6 @@
 
 #endregion
 
-using System;
-using System.Collections.Generic;
-using System.Globalization;
-using System.Threading.Tasks;
-using Grpc.Core.Internal;
 using Grpc.Core.Utils;
 
 namespace Grpc.Core;
@@ -121,8 +116,8 @@ private sealed class CompositeChannelCredentials : ChannelCredentials
         /// <param name="callCredentials">channelCredentials to compose</param>
         public CompositeChannelCredentials(ChannelCredentials channelCredentials, CallCredentials callCredentials)
         {
-            this.channelCredentials = GrpcPreconditions.CheckNotNull(channelCredentials);
-            this.callCredentials = GrpcPreconditions.CheckNotNull(callCredentials);
+            this.channelCredentials = GrpcPreconditions.CheckNotNull(channelCredentials, nameof(channelCredentials));
+            this.callCredentials = GrpcPreconditions.CheckNotNull(callCredentials, nameof(callCredentials));
         }
 
         public override void InternalPopulateConfiguration(ChannelCredentialsConfiguratorBase configurator, object state)

src/Grpc.Core.Api/ClientBase.cs

@@ -212,7 +212,7 @@ internal ClientBaseConfigurationInfo(string? host, CallOptions callOptions)
 
         internal ClientBaseConfiguration(CallInvoker undecoratedCallInvoker, string? host)
         {
-            this.undecoratedCallInvoker = GrpcPreconditions.CheckNotNull(undecoratedCallInvoker);
+            this.undecoratedCallInvoker = GrpcPreconditions.CheckNotNull(undecoratedCallInvoker, nameof(undecoratedCallInvoker));
             this.host = host;
         }
 

src/Grpc.Core.Api/Grpc.Core.Api.csproj

@@ -9,7 +9,6 @@
     <PackageReadmeFile>README.md</PackageReadmeFile>
 
     <AllowUnsafeBlocks>true</AllowUnsafeBlocks>
-    <Nullable>enable</Nullable>
     <!-- grpc-dotnet global usings break the build. -->
     <ImplicitUsings>disable</ImplicitUsings>
   </PropertyGroup>

src/Grpc.Core.Api/Metadata.cs

@@ -149,7 +149,6 @@ public void Add(string key, byte[] valueBytes)
 
     #region IList members
 
-
     /// <summary>
     /// <see cref="T:IList`1"/>
     /// </summary>
@@ -163,7 +162,7 @@ public int IndexOf(Metadata.Entry item)
     /// </summary>
     public void Insert(int index, Metadata.Entry item)
     {
-        GrpcPreconditions.CheckNotNull(item);
+        GrpcPreconditions.CheckNotNull(item, nameof(item));
         CheckWriteable();
         entries.Insert(index, item);
     }
@@ -189,7 +188,7 @@ public Metadata.Entry this[int index]
 
         set
         {
-            GrpcPreconditions.CheckNotNull(value);
+            GrpcPreconditions.CheckNotNull(value, nameof(value));
             CheckWriteable();
             entries[index] = value;
         }
@@ -200,7 +199,7 @@ public Metadata.Entry this[int index]
     /// </summary>
     public void Add(Metadata.Entry item)
     {
-        GrpcPreconditions.CheckNotNull(item);
+        GrpcPreconditions.CheckNotNull(item, nameof(item));
         CheckWriteable();
         entries.Add(item);
     }
@@ -462,8 +461,10 @@ private static bool IsValidKey(string input, out bool isLowercase)
                     '0' <= c && c <= '9' ||
                     c == '.' ||
                     c == '_' ||
-                    c == '-' )
+                    c == '-')
+                {
                     continue;
+                }
 
                 if ('A' <= c && c <= 'Z')
                 {