https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6369

Kevin A. McGrail <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]

--- Comment #1 from Kevin A. McGrail <[email protected]> 2010-03-11 17:46:33 
UTC ---
I don't like the way the sleep 2 is written inside the if loop

Would suggest the following minor change for readability purposes.

if (Mail::SpamAssassin::Util::am_running_on_windows()) {
    sleep 2; # Need this on win32 so that a child can get a signal
}

Otherwise, I'm +1 though I'm not testing the code on a win32 system.

KAM

-- 
Configure bugmail: 
https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

Reply via email to