[Mono-devel-list] small arraylist patch

Lluis Sanchez lluis at ximian.com
Thu Jul 21 19:03:40 EDT 2005


Actually, the parameter of that ArgumentNullException constructor is the
parameter name, not the error message. So it should be:

throw new ArgumentNullException ("c");

or

throw new ArgumentNullException ("c", "Collection cannot be null.");

El dj 21 de 07 del 2005 a les 15:52 -0700, en/na Jackson Harper va
escriure:
> On Fri, 2005-07-22 at 00:20 +0200, Andreas Nahr wrote:
> > The best way is to not override the textual message at all, because it is 
> > 100% redundant.
> 
> Old message:
> "Argument cannot be null."
> 
> New message:
> "Collection cannot be null."
> 
> percentage of redundancy:
> 75
> 
> Jackson
> 
> 
> > ----- Original Message ----- 
> > From: "Jackson Harper" <jackson at ximian.com>
> > To: <mono-devel-list at lists.ximian.com>
> > Sent: Thursday, July 21, 2005 9:49 PM
> > Subject: Re: [Mono-devel-list] small arraylist patch
> > 
> > 
> > >
> > > This patch does the message correctly and has a better test case.
> > >
> > > Jackson
> > >
> > >
> > > On Thu, 2005-07-21 at 11:31 -0700, Jackson Harper wrote:
> > >> Does anyone have any problems with me committing this? Currently we are
> > >> getting a NullReferenceException (with no descriptive message) instead
> > >> of an ArgumentNullException like MS does.
> > >>
> > >> Speak within the next 24 hours or forever hold your piece.
> > >>
> > >> Love,
> > >> Jackson
> > >>
> > >>
> > >> _______________________________________________
> > >> Mono-devel-list mailing list
> > >> Mono-devel-list at lists.ximian.com
> > >> http://lists.ximian.com/mailman/listinfo/mono-devel-list
> > >
> > 
> > 
> > --------------------------------------------------------------------------------
> > 
> > 
> > > _______________________________________________
> > > Mono-devel-list mailing list
> > > Mono-devel-list at lists.ximian.com
> > > http://lists.ximian.com/mailman/listinfo/mono-devel-list
> > > 
> > 
> > _______________________________________________
> > Mono-devel-list mailing list
> > Mono-devel-list at lists.ximian.com
> > http://lists.ximian.com/mailman/listinfo/mono-devel-list
> 
> _______________________________________________
> Mono-devel-list mailing list
> Mono-devel-list at lists.ximian.com
> http://lists.ximian.com/mailman/listinfo/mono-devel-list




More information about the Mono-devel-list mailing list