>>>>> "BB" == Brian Behlendorf <brian@hyperreal.org> writes:
BB> Correct me if I'm wrong, but shouldn't "$a" be "$temp" in the second
BB> and third while line?
Hmmm is right. A patch must have been botched sometime in the past.
Fortunately it makes very, very little difference because nobody ever uses
spaces in the places where that code removes them. It just takes care of
one of the many nasty cases in RFC822.
BB> Also, I "get" the point of a rule further down:
BB> # Look for addresses of the form rouilj@cs if (/\@/ && !/\./) {
BB> However, it appears this routine allows through email addresses without
BB> any "@" sign at all.
Yes, it's been that way since 1.92 or maybe earlier. I've never understood
whether it was intended to allow such an address through or not.
- J<
References:
-
hmmm..
From: Brian Behlendorf <brian@hyperreal.org>
|
|
