Atsushi, Yes, the argument name in the ArgumentNullException does not match the name of the method argument.
This is probably because the argument check itself is not performed in the Find method, but bubbled up. This is a detail not worth discussing, but also one that was not worth reverting. Gert -----Original Message----- From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On Behalf Of Atsushi Eno Sent: vrijdag 27 juni 2008 23:47 To: Gert Driesen Cc: 'mono-devel-list' Subject: Re: [Mono-dev] [Mono-patches] r106777 - in trunk/mcs/class/System: System.ComponentModel Test/System.ComponentModel Um, you're right, sadly. Then; check the name of the argument. Atsushi Eno > Atsushi, > > I did not ask you privately. > > Gert > > -----Original Message----- > From: Atsushi Eno [mailto:[EMAIL PROTECTED] > Sent: vrijdag 27 juni 2008 23:34 > To: Gert Driesen > Subject: Re: [Mono-patches] r106777 - in trunk/mcs/class/System: > System.ComponentModel Test/System.ComponentModel > > I never give any replies privately. Though if you ask it on any public > list, you > might end up to feel ashamed, so I wouldn't recommend it. > > >> Atsushi, >> >> Your changes introduce test failures on MS. >> >> Please explain why you need to revert that change! >> >> Gert >> >> -----Original Message----- >> From: [EMAIL PROTECTED] >> [mailto:[EMAIL PROTECTED] On Behalf Of Atsushi >> > Enomoto > >> ([EMAIL PROTECTED]) >> Sent: vrijdag 27 juni 2008 22:56 >> To: [EMAIL PROTECTED]; [EMAIL PROTECTED]; >> [EMAIL PROTECTED] >> Subject: [Mono-patches] r106777 - in trunk/mcs/class/System: >> System.ComponentModel Test/System.ComponentModel >> >> Author: atsushi >> Date: 2008-06-27 16:56:26 -0400 (Fri, 27 Jun 2008) >> New Revision: 106777 >> >> Modified: >> trunk/mcs/class/System/System.ComponentModel/ChangeLog >> >> >> > trunk/mcs/class/System/System.ComponentModel/PropertyDescriptorCollection.cs > >> trunk/mcs/class/System/Test/System.ComponentModel/ChangeLog >> >> >> > trunk/mcs/class/System/Test/System.ComponentModel/PropertyDescriptorCollecti > >> onTests.cs >> Log: >> too silly to document. >> >> >> >> Modified: trunk/mcs/class/System/System.ComponentModel/ChangeLog >> =================================================================== >> --- trunk/mcs/class/System/System.ComponentModel/ChangeLog 2008-06-27 >> 20:45:49 UTC (rev 106776) >> +++ trunk/mcs/class/System/System.ComponentModel/ChangeLog 2008-06-27 >> 20:56:26 UTC (rev 106777) >> @@ -1,3 +1,7 @@ >> +2008-06-27 Atsushi Enomoto <[EMAIL PROTECTED]> >> + >> + * PropertyDescriptorCollection.cs : huh. >> + >> 2008-06-27 Gert Driesen <[EMAIL PROTECTED]> >> >> * EventDescriptorCollection.cs: Fixed Empty to return read-only >> >> Modified: >> >> > trunk/mcs/class/System/System.ComponentModel/PropertyDescriptorCollection.cs > >> =================================================================== >> --- >> >> > trunk/mcs/class/System/System.ComponentModel/PropertyDescriptorCollection.cs > >> 2008-06-27 20:45:49 UTC (rev 106776) >> +++ >> >> > trunk/mcs/class/System/System.ComponentModel/PropertyDescriptorCollection.cs > >> 2008-06-27 20:56:26 UTC (rev 106777) >> @@ -140,7 +140,7 @@ >> public virtual PropertyDescriptor Find (string name, bool >> ignoreCase) >> { >> if (name == null) >> - throw new ArgumentNullException ("key"); >> + throw new ArgumentNullException ("name"); >> >> foreach (PropertyDescriptor p in properties) { >> #if NET_2_0 >> >> Modified: trunk/mcs/class/System/Test/System.ComponentModel/ChangeLog >> =================================================================== >> --- trunk/mcs/class/System/Test/System.ComponentModel/ChangeLog >> > 2008-06-27 > >> 20:45:49 UTC (rev 106776) >> +++ trunk/mcs/class/System/Test/System.ComponentModel/ChangeLog >> > 2008-06-27 > >> 20:56:26 UTC (rev 106777) >> @@ -1,3 +1,7 @@ >> +2008-06-27 Atsushi Enomoto <[EMAIL PROTECTED]> >> + >> + * PropertyDescriptorCollectionTests.cs : huh. >> + >> 2008-06-27 Gert Driesen <[EMAIL PROTECTED]> >> >> * EventDescriptorCollectionTests.cs: Enabled tests for Empty, Find >> and >> >> Modified: >> >> > trunk/mcs/class/System/Test/System.ComponentModel/PropertyDescriptorCollecti > >> onTests.cs >> =================================================================== >> --- >> >> > trunk/mcs/class/System/Test/System.ComponentModel/PropertyDescriptorCollecti > >> onTests.cs 2008-06-27 20:45:49 UTC (rev 106776) >> +++ >> >> > trunk/mcs/class/System/Test/System.ComponentModel/PropertyDescriptorCollecti > >> onTests.cs 2008-06-27 20:56:26 UTC (rev 106777) >> @@ -92,7 +92,7 @@ >> Assert.AreEqual (typeof >> (ArgumentNullException), ex.GetType (), "#A2"); >> Assert.IsNull (ex.InnerException, "#A3"); >> Assert.IsNotNull (ex.Message, "#A4"); >> - Assert.AreEqual ("key", ex.ParamName, >> "#A5"); >> + Assert.AreEqual ("name", ex.ParamName, >> "#A5"); >> } >> >> try { >> @@ -102,7 +102,7 @@ >> Assert.AreEqual (typeof >> (ArgumentNullException), ex.GetType (), "#B2"); >> Assert.IsNull (ex.InnerException, "#B3"); >> Assert.IsNotNull (ex.Message, "#B4"); >> - Assert.AreEqual ("key", ex.ParamName, >> "#B5"); >> + Assert.AreEqual ("name", ex.ParamName, >> "#B5"); >> } >> >> descriptors = PropertyDescriptorCollection.Empty; >> @@ -114,7 +114,7 @@ >> Assert.AreEqual (typeof >> (ArgumentNullException), ex.GetType (), "#C2"); >> Assert.IsNull (ex.InnerException, "#C3"); >> Assert.IsNotNull (ex.Message, "#C4"); >> - Assert.AreEqual ("key", ex.ParamName, >> "#C5"); >> + Assert.AreEqual ("name", ex.ParamName, >> "#C5"); >> } >> >> try { >> @@ -124,7 +124,7 @@ >> Assert.AreEqual (typeof >> (ArgumentNullException), ex.GetType (), "#D2"); >> Assert.IsNull (ex.InnerException, "#D3"); >> Assert.IsNotNull (ex.Message, "#D4"); >> - Assert.AreEqual ("key", ex.ParamName, >> "#D5"); >> + Assert.AreEqual ("name", ex.ParamName, >> "#D5"); >> } >> } >> >> @@ -235,7 +235,7 @@ >> Assert.AreEqual (typeof >> (ArgumentNullException), ex.GetType (), "#A2"); >> Assert.IsNull (ex.InnerException, "#A3"); >> Assert.IsNotNull (ex.Message, "#A4"); >> - Assert.AreEqual ("key", ex.ParamName, >> "#A5"); >> + Assert.AreEqual ("name", ex.ParamName, >> "#A5"); >> } >> >> descriptors = PropertyDescriptorCollection.Empty; >> @@ -247,7 +247,7 @@ >> Assert.AreEqual (typeof >> (ArgumentNullException), ex.GetType (), "#B2"); >> Assert.IsNull (ex.InnerException, "#B3"); >> Assert.IsNotNull (ex.Message, "#B4"); >> - Assert.AreEqual ("key", ex.ParamName, >> "#B5"); >> + Assert.AreEqual ("name", ex.ParamName, >> "#B5"); >> } >> } >> >> >> _______________________________________________ >> Mono-patches maillist - [EMAIL PROTECTED] >> http://lists.ximian.com/mailman/listinfo/mono-patches >> >> >> > > > > _______________________________________________ Mono-devel-list mailing list Mono-devel-list@lists.ximian.com http://lists.ximian.com/mailman/listinfo/mono-devel-list _______________________________________________ Mono-devel-list mailing list Mono-devel-list@lists.ximian.com http://lists.ximian.com/mailman/listinfo/mono-devel-list
