------- You are receiving this mail because: -------
You are on the CC list for the bug.

http://bugs.exim.org/show_bug.cgi?id=1403

Phil Pennock <[email protected]> changed:

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




--- Comment #5 from Phil Pennock <[email protected]>  2013-10-25 01:54:53 ---
Be careful here: studying a regexp can be quite slow, which is why it's not
done by default.  In Perl, for instance, you have to be invoking a regexp a
potentially large number of times before the costs are paid for by improved
performance.

So studying all regular expressions always may result in performance
regressions.

It might be worth adding a match_study expansion condition; or pushing this
change after 4.82 goes out, so that people have more time to explore the impact
on them.

Other than the potential for performance regressions, the patch looks good to
me.


-- 
Configure bugmail: http://bugs.exim.org/userprefs.cgi?tab=email

-- 
## List details at https://lists.exim.org/mailman/listinfo/exim-dev Exim 
details at http://www.exim.org/ ##

Reply via email to