https://bz.apache.org/SpamAssassin/show_bug.cgi?id=8470

Sidney Markowitz <[email protected]> changed:

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

--- Comment #2 from Sidney Markowitz <[email protected]> ---
I did it with in-core scalar filehandle at first but IO::String does it twice
as fast. I made the call to have the speed at the expense of the added
dependency. But looking at it more carefully I find that I don't care about
saving only up to 15 milliseconds on very large files, or even 150 ms on
specially crafted large malicious files, especially at the expense of all the
problems that you have pointed out it would cause.

+1 to commit patch 6162 for the 4.0.3 release

When you commit this, remove the mention of IO::String in INSTALL, UPGRADE, and
its installation in
.github/actions/install_cpan_dependencies_for_sa/action.yml. Votes are not
needed for those changes.

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

Reply via email to