Ok;

In the process off adding a comment to the code, I see that you did it already. Thanks.

I downloaded also your adding to the wiki about exception handling: I'll have a look 
and edit it further if useful. Thanks again.

Vincenzo

> -----Original Message-----
> From: Noel J. Bergman [mailto:[EMAIL PROTECTED]
> Sent: venerd� 20 giugno 2003 18.49
> To: James Developers List
> Subject: RE: zero length array in AbstractRedirect
> 
> 
> Vincenzo,
> 
> We're talking about the same thing.  :-)
> 
> My reaction to seeing:
> 
>   apparentlyTo = new InternetAddress[0];
> 
> was that it was a bit of an odd construction, since 99 times out of a 100,
> someone would just assign a null.
> 
> > I wrote [because] later on the call
> >
> >    setTo(newMail, getTo(originalMail), originalMail);
> >
> > would behave differently if getTo(originalMail) returned null: in
> > this case the TO: value would be the original one (unchanged);
> > with an empty array it would, as you said, be converted to null
> 
> And my point was that we should explain that so it doesn't get 
> broken in the
> future when someone new goes to make changes, and says "Oh, 
> that's odd.  We
> don't need to create an object.  I'll just set this to null."
> 
>       --- Noel
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
> 


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to