Malte S. Stretz said: > If nobody object I will apply the patch tomorrow or so.
I'm fine with the concept. I always thought the name "search" was a bit strange for what it was. I don't see the need for keeping a search sub for "backward compatibility". Isn't DnsResolver a new class we introduced with the new DNs code that has not been released yet? Just grep through the source to make sure that you haven't missed any other references to it. Regarding Mail::SPF::Query, are you saying that you are going to write something that will use our backgrounded resolver insstead of directly calling Net::DNS? That has some advantages of reducing the number of sockets/filehandles used and letting the queries run in parallel with the other processing the way our DNSBL processing does. I looked at doing that but it seemed that the DNS queries were too enmeshed in the rest of the SPF::Query code to separate out easily. -- sidney
