http://issues.apache.org/SpamAssassin/show_bug.cgi?id=4901





------- Additional Comments From [EMAIL PROTECTED]  2006-05-22 14:41 -------
(In reply to comment #4)
> theo _did_ change the summary line to include [review], so probably meant to
> vote +1. still, better to get that explicitly ;)

Actually I was just moving it to the 3.1.2 queue, and since it had a patch I 
added in the review state.  I 
was planning to review and vote shortly thereafter but got distracted by other 
things.  :(

The patch generally looks good, but is there a reason to use $! and not 
$resolver->errorstring ?  
Something similar to the code in the Net::DNS POD:

             warn "query failed: ", $res->errorstring, "\n";




------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

Reply via email to