2007-12-18 Carlos Alberto Cortez <calberto.cortez@gmail.com>

[mono.git] / mcs / class / Mono.Data / ProviderCollection.cs

diff --git a/mcs/class/Mono.Data/ProviderCollection.cs b/mcs/class/Mono.Data/ProviderCollection.cs
index c82b36e44958c14b15e7f53af5e3d24602265f78..95411824d62659c66635262b8a9764b1083d498b 100644 (file)
--- a/mcs/class/Mono.Data/ProviderCollection.cs
+++ b/mcs/class/Mono.Data/ProviderCollection.cs
@@ -42,7 +42,7 @@ namespace Mono.Data
        ///    </para>\r
        /// </summary>\r
        /// <seealso cref='.ProviderCollection'/>\r
-       [Serializable()]\r
+       [Serializable ()]\r
        public class ProviderCollection : DictionaryBase  \r
        {\r
                \r
@@ -51,7 +51,8 @@ namespace Mono.Data
                ///       Initializes a new instance of <see cref='.ProviderCollection'/>.\r
                ///    </para>\r
                /// </summary>\r
-               public ProviderCollection() \r
+               public ProviderCollection ()\r
+\r
                {\r
                }\r
                \r
@@ -63,9 +64,12 @@ namespace Mono.Data
                /// <param name='value'>\r
                ///       A <see cref='.ProviderCollection'/> from which the contents are copied\r
                /// </param>\r
-               public ProviderCollection(ProviderCollection value) \r
+               public ProviderCollection (ProviderCollection value) \r
                {\r
-                       this.AddRange(value);\r
+                       if (value == null) \r
+                               throw new System.ArgumentNullException ("value");\r
+\r
+                       this.AddRange (value);\r
                }\r
                \r
                /// <summary>\r
@@ -76,9 +80,12 @@ namespace Mono.Data
                /// <param name='value'>\r
                ///       A array of <see cref='.Provider'/> objects with which to intialize the collection\r
                /// </param>\r
-               public ProviderCollection(Provider[] value) \r
+               public ProviderCollection (Provider[] value) \r
                {\r
-                       this.AddRange(value);\r
+                       if (value == null) \r
+                               throw new System.ArgumentNullException ("value");\r
+\r
+                       this.AddRange (value);\r
                }\r
                \r
                /// <summary>\r
@@ -89,46 +96,50 @@ namespace Mono.Data
                ///    <para> The entry at the specified index of the collection.</para>\r
                /// </value>\r
                /// <exception cref='System.ArgumentOutOfRangeException'><paramref name='index'/> is outside the valid range of indexes for the collection.</exception>\r
-               public Provider this[string Name] \r
+               public Provider this [string Name] \r
                {\r
-                       get \r
-                       {\r
-                               return ((Provider)(Dictionary[Name]));\r
+                       get {                           \r
+                               return ((Provider)(Dictionary [Name]));\r
                        }\r
-                       set \r
-                       {\r
-                               Dictionary[Name] = value;\r
+                       set {\r
+                               Dictionary [Name] = value;\r
                        }\r
                }\r
 \r
                public Provider FindByCommandType(Type CommandType)\r
                {\r
-                       foreach (Provider p in this)\r
-                       {\r
-                               if (p.CommandType==CommandType)\r
+                       if (CommandType == null) \r
+                               throw new System.ArgumentNullException ("CommandType");\r
+\r
+                       foreach (Provider p in this) {\r
+                               if (p.CommandType == CommandType)\r
                                        return p;\r
                        }\r
-                       throw new IndexOutOfRangeException();\r
+                       throw new IndexOutOfRangeException ();\r
                }\r
 \r
                public Provider FindByDataAdapterType(Type DataAdapterType)\r
                {\r
-                       foreach (Provider p in this)\r
-                       {\r
-                               if (p.DataAdapterType==DataAdapterType)\r
+                       if (DataAdapterType == null) \r
+                               throw new System.ArgumentNullException ("DataAdapterType");\r
+\r
+                       foreach (Provider p in this) {\r
+                               if (p.DataAdapterType == DataAdapterType)\r
                                        return p;\r
                        }\r
-                       throw new IndexOutOfRangeException();\r
+                       throw new IndexOutOfRangeException ();\r
                }\r
 \r
                public Provider FindByConnectionType(Type ConnectionType)\r
                {\r
-                       foreach (Provider p in this)\r
-                       {\r
-                               if (p.ConnectionType==ConnectionType)\r
+                       if (ConnectionType == null) \r
+                               throw new System.ArgumentNullException("ConnectionType");\r
+\r
+                       foreach (Provider p in this) {\r
+                               if (p.ConnectionType == ConnectionType)\r
                                        return p;\r
                        }\r
-                       throw new IndexOutOfRangeException();\r
+                       throw new IndexOutOfRangeException ();\r
                }\r
 \r
                /// <summary>\r
@@ -142,7 +153,10 @@ namespace Mono.Data
                /// <seealso cref='.ProviderCollection.AddRange'/>\r
                public void Add(Provider value) \r
                {\r
-                       Dictionary.Add(value.Name, value);\r
+                       if (value == null) \r
+                               throw new System.ArgumentNullException ("value");\r
+\r
+                       Dictionary.Add (value.Name, value);\r
                }\r
                \r
                /// <summary>\r
@@ -155,12 +169,13 @@ namespace Mono.Data
                ///   <para>None.</para>\r
                /// </returns>\r
                /// <seealso cref='.ProviderCollection.Add'/>\r
-               public void AddRange(Provider[] value) \r
+               public void AddRange (Provider[] value) \r
                {\r
-                       for (int i = 0; (i < value.Length); i = (i + 1)) \r
-                       {\r
-                               this.Add(value[i]);\r
-                       }\r
+                       if (value == null) \r
+                               throw new System.ArgumentNullException ("value");\r
+\r
+                       for (int i = 0; i < value.Length; i++) \r
+                               this.Add (value [i]);\r
                }\r
                \r
                /// <summary>\r
@@ -177,10 +192,11 @@ namespace Mono.Data
                /// <seealso cref='.ProviderCollection.Add'/>\r
                public void AddRange(ProviderCollection value) \r
                {\r
+                       if (value == null) \r
+                               throw new System.ArgumentNullException ("value");\r
+\r
                        foreach (Provider p in value)\r
-                       {\r
-                               this.Add(p);\r
-                       }\r
+                               this.Add (p);\r
                }\r
                \r
                /// <summary>\r
@@ -193,9 +209,12 @@ namespace Mono.Data
                ///   otherwise, <see langword='false'/>.</para>\r
                /// </returns>\r
                /// <seealso cref='.ProviderCollection.IndexOf'/>\r
-               public bool Contains(Provider value) \r
+               public bool Contains (Provider value) \r
                {\r
-                       return Dictionary.Contains(value);\r
+                       if (value == null) \r
+                               throw new System.ArgumentNullException("value");\r
+\r
+                       return Dictionary.Contains (value);\r
                }\r
                \r
                /// <summary>\r
@@ -213,6 +232,9 @@ namespace Mono.Data
                /// <seealso cref='System.Array'/>\r
                public void CopyTo(Provider[] array, int index) \r
                {\r
+                       if (array == null) \r
+                               throw new System.ArgumentNullException ("array");\r
+\r
                        Dictionary.CopyTo(array, index);\r
                }\r
                \r
@@ -222,9 +244,9 @@ namespace Mono.Data
                /// </summary>\r
                /// <returns><para>None.</para></returns>\r
                /// <seealso cref='System.Collections.IEnumerator'/>\r
-               public new ProviderEnumerator GetEnumerator() \r
+               public new ProviderEnumerator GetEnumerator () \r
                {\r
-                       return new ProviderEnumerator(this);\r
+                       return new ProviderEnumerator (this);\r
                }\r
                \r
                /// <summary>\r
@@ -236,6 +258,9 @@ namespace Mono.Data
                /// <exception cref='System.ArgumentException'><paramref name='value'/> is not found in the Collection. </exception>\r
                public void Remove(Provider value) \r
                {\r
+                       if (value == null) \r
+                               throw new System.ArgumentNullException ("value");\r
+\r
                        Dictionary.Remove(value);\r
                }\r
                \r
@@ -248,6 +273,9 @@ namespace Mono.Data
                        \r
                        public ProviderEnumerator(ProviderCollection mappings) \r
                        {\r
+                               if (mappings == null) \r
+                                       throw new System.ArgumentNullException ("mappings");\r
+\r
                                this.temp = ((IEnumerable)(mappings));\r
                                this.baseEnumerator = temp.GetEnumerator();\r
                        }\r