You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@spamassassin.apache.org by mm...@apache.org on 2010/03/30 03:29:47 UTC

svn commit: r928958 - /spamassassin/trunk/lib/Mail/SpamAssassin/Dns.pm

Author: mmartinec
Date: Tue Mar 30 01:29:47 2010
New Revision: 928958

URL: http://svn.apache.org/viewvc?rev=928958&view=rev
Log:
defined(%hash) is not the way to test a hash for being empty

Modified:
    spamassassin/trunk/lib/Mail/SpamAssassin/Dns.pm

Modified: spamassassin/trunk/lib/Mail/SpamAssassin/Dns.pm
URL: http://svn.apache.org/viewvc/spamassassin/trunk/lib/Mail/SpamAssassin/Dns.pm?rev=928958&r1=928957&r2=928958&view=diff
==============================================================================
--- spamassassin/trunk/lib/Mail/SpamAssassin/Dns.pm (original)
+++ spamassassin/trunk/lib/Mail/SpamAssassin/Dns.pm Tue Mar 30 01:29:47 2010
@@ -750,7 +750,7 @@ sub enter_helper_run_mode {
   dbg("dns: entering helper-app run mode");
   $self->{old_slash} = $/;              # Razor pollutes this
   %{$self->{old_env}} = ();
-  if ( defined %ENV ) {
+  if ( %ENV ) {
     # undefined values in %ENV can result due to autovivification elsewhere,
     # this prevents later possible warnings when we restore %ENV
     while (my ($key, $value) = each %ENV) {