You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@spamassassin.apache.org by Theo Van Dinter <fe...@apache.org> on 2005/12/19 09:50:49 UTC

Re: svn commit: r357661 - /spamassassin/branches/3.1/lib/Mail/SpamAssassin/DnsResolver.pm

Errr.  Me thinks the patch committed is not what was intended...

On Mon, Dec 19, 2005 at 07:27:16AM -0000, jm@apache.org wrote:
> bug 4719: long paragraphs should be split on word boundaries where possible
> 
> Modified:
>     spamassassin/branches/3.1/lib/Mail/SpamAssassin/DnsResolver.pm
> 
[...]
> -A wrapper for C<Net::DNS::Packet::new()> which traps a die thrown by it
> +A wrapper for C<Net::DNS::Packet::new()> which traps a die thrown by it.
[...]
> +  # construct a PTR query if it looks like an IPv4 address
> +  if ((!defined($type) || $type eq 'PTR') && $host =~ /^(\d+)\.(\d+)\.(\d+)\.(\d+)$/) {
> +    $host = "$4.$3.$2.$1.in-addr.arpa.";
> +    $type = 'PTR';
> +  }

-- 
Randomly Generated Tagline:
"I don't think Microsoft is evil in itself; I just think they make really
 crappy operating systems."            - Linus Torvalds

Re: svn commit: r357661 - /spamassassin/branches/3.1/lib/Mail/SpamAssassin/DnsResolver.pm

Posted by Sidney Markowitz <si...@sidney.com>.
Theo Van Dinter wrote:
> Errr.  Me thinks the patch committed is not what was intended...

All fixed up :-)

 -- sidney