Fix ArgumentNullException messages passed as parameter names (#2889)

Fixed several ArgumentNullExceptions that throw with the message as the parameter name, leading to potential confusion for developers.

Author: hughbe

src/mscorlib/src/System/AppDomain.cs

@@ -3289,7 +3289,7 @@ internal static AppDomain InternalCreateDomain(String friendlyName,
                                       AppDomainSetup info)
         {
             if (friendlyName == null)
-                throw new ArgumentNullException(Environment.GetResourceString("ArgumentNull_String"));
+                throw new ArgumentNullException("friendlyName", Environment.GetResourceString("ArgumentNull_String"));
 
             Contract.EndContractBlock();
 

src/mscorlib/src/System/AppDomainManager.cs

@@ -59,7 +59,7 @@ protected static AppDomain CreateDomainHelper (string friendlyName,
                                                        Evidence securityInfo,
                                                        AppDomainSetup appDomainInfo) {
             if (friendlyName == null)
-                throw new ArgumentNullException(Environment.GetResourceString("ArgumentNull_String"));
+                throw new ArgumentNullException("friendlyName", Environment.GetResourceString("ArgumentNull_String"));
 
             Contract.EndContractBlock();
             // If evidence is provided, we check to make sure that is allowed.

src/mscorlib/src/System/Globalization/DateTimeFormatInfo.cs

@@ -2056,7 +2056,7 @@ public void SetAllDateTimePatterns(String[] patterns, char format)
             {
                 if (patterns[i] == null)
                 {
-                    throw new ArgumentNullException(Environment.GetResourceString("ArgumentNull_ArrayValue"));
+                    throw new ArgumentNullException("patterns[" + i + "]", Environment.GetResourceString("ArgumentNull_ArrayValue"));
                 }
             }
 

src/mscorlib/src/System/Reflection/FieldInfo.cs

@@ -36,7 +36,7 @@ public abstract class FieldInfo : MemberInfo, _FieldInfo
         public static FieldInfo GetFieldFromHandle(RuntimeFieldHandle handle)
         {
             if (handle.IsNullHandle())
-                throw new ArgumentException(Environment.GetResourceString("Argument_InvalidHandle"));
+                throw new ArgumentException(Environment.GetResourceString("Argument_InvalidHandle"), "handle");
 
             FieldInfo f = RuntimeType.GetFieldInfo(handle.GetRuntimeFieldInfo());
 

src/mscorlib/src/System/Runtime/InteropServices/Marshal.cs

@@ -1060,7 +1060,7 @@ public static IntPtr GetHINSTANCE(Module m)
             }
 
             if (rtModule == null)
-                throw new ArgumentNullException(Environment.GetResourceString("Argument_MustBeRuntimeModule"));
+                throw new ArgumentNullException("m", Environment.GetResourceString("Argument_MustBeRuntimeModule"));
 
             return GetHINSTANCE(rtModule.GetNativeHandle());
         }    

src/mscorlib/src/System/Security/Policy/ApplicationTrust.cs

@@ -91,7 +91,7 @@ public ApplicationTrust(PermissionSet defaultGrantSet, IEnumerable<StrongName> f
             List<StrongName> fullTrustList = new List<StrongName>();
             foreach (StrongName strongName in fullTrustAssemblies) {
                 if (strongName == null) {
-                    throw new ArgumentException(Environment.GetResourceString("Argument_NullFullTrustAssembly"));
+                    throw new ArgumentException(Environment.GetResourceString("Argument_NullFullTrustAssembly"), "fullTrustAssemblies");
                 }
 
                 fullTrustList.Add(new StrongName(strongName.PublicKey, strongName.Name, strongName.Version));
@@ -120,7 +120,7 @@ public ApplicationIdentity ApplicationIdentity {
             }
             set {
                 if (value == null)
-                    throw new ArgumentNullException(Environment.GetResourceString("Argument_InvalidAppId"));
+                    throw new ArgumentNullException("value", Environment.GetResourceString("Argument_InvalidAppId"));
                 Contract.EndContractBlock();
                 m_appId = value;
             }

src/mscorlib/src/System/StubHelpers.cs

@@ -1670,7 +1670,7 @@ static internal IntPtr SafeHandleAddRef(SafeHandle pHandle, ref bool success)
         {
             if (pHandle == null)
             {
-                throw new ArgumentNullException(Environment.GetResourceString("ArgumentNull_SafeHandle"));
+                throw new ArgumentNullException("pHandle", Environment.GetResourceString("ArgumentNull_SafeHandle"));
             }
             Contract.EndContractBlock();
 
@@ -1686,7 +1686,7 @@ static internal void SafeHandleRelease(SafeHandle pHandle)
         {
             if (pHandle == null)
             {
-                throw new ArgumentNullException(Environment.GetResourceString("ArgumentNull_SafeHandle"));
+                throw new ArgumentNullException("pHandle", Environment.GetResourceString("ArgumentNull_SafeHandle"));
             }
             Contract.EndContractBlock();
 

src/mscorlib/src/System/Text/StringBuilder.cs

@@ -1108,7 +1108,7 @@ public StringBuilder Insert(int index, char[] value, int startIndex, int charCou
                 {
                     return this;
                 }
-                throw new ArgumentNullException(Environment.GetResourceString("ArgumentNull_String"));
+                throw new ArgumentNullException("value", Environment.GetResourceString("ArgumentNull_String"));
             }
 
             //Range check the array.

src/mscorlib/src/System/Threading/WaitHandle.cs

@@ -281,7 +281,7 @@ public static bool WaitAll(WaitHandle[] waitHandles, int millisecondsTimeout, bo
         {
             if (waitHandles == null)
             {
-                throw new ArgumentNullException(Environment.GetResourceString("ArgumentNull_Waithandles"));
+                throw new ArgumentNullException("waitHandles", Environment.GetResourceString("ArgumentNull_Waithandles"));
             }
             if(waitHandles.Length == 0)
             {
@@ -297,7 +297,7 @@ public static bool WaitAll(WaitHandle[] waitHandles, int millisecondsTimeout, bo
 #if FEATURE_CORECLR
                 throw new ArgumentException(Environment.GetResourceString("Argument_EmptyWaithandleArray"));
 #else
-                throw new ArgumentNullException(Environment.GetResourceString("Argument_EmptyWaithandleArray"));
+                throw new ArgumentNullException("waitHandles", Environment.GetResourceString("Argument_EmptyWaithandleArray"));
 #endif
             }
             if (waitHandles.Length > MAX_WAITHANDLES)
@@ -315,7 +315,7 @@ public static bool WaitAll(WaitHandle[] waitHandles, int millisecondsTimeout, bo
                 WaitHandle waitHandle = waitHandles[i];
 
                 if (waitHandle == null)
-                    throw new ArgumentNullException(Environment.GetResourceString("ArgumentNull_ArrayElement"));
+                    throw new ArgumentNullException("waitHandles[" + i + "]", Environment.GetResourceString("ArgumentNull_ArrayElement"));
 
 #if FEATURE_REMOTING        
                 if (RemotingServices.IsTransparentProxy(waitHandle))
@@ -394,7 +394,7 @@ public static int WaitAny(WaitHandle[] waitHandles, int millisecondsTimeout, boo
         {
             if (waitHandles==null)
             {
-                throw new ArgumentNullException(Environment.GetResourceString("ArgumentNull_Waithandles"));
+                throw new ArgumentNullException("waitHandles", Environment.GetResourceString("ArgumentNull_Waithandles"));
             }
             if(waitHandles.Length == 0)
             {
@@ -415,7 +415,7 @@ public static int WaitAny(WaitHandle[] waitHandles, int millisecondsTimeout, boo
                 WaitHandle waitHandle = waitHandles[i];
 
                 if (waitHandle == null)
-                    throw new ArgumentNullException(Environment.GetResourceString("ArgumentNull_ArrayElement"));
+                    throw new ArgumentNullException("waitHandles[" + i + "]", Environment.GetResourceString("ArgumentNull_ArrayElement"));
 
 #if FEATURE_REMOTING        
                 if (RemotingServices.IsTransparentProxy(waitHandle))