[Bug 7698] Prevent loading warnings.pm every eval statement
https://bz.apache.org/SpamAssassin/show_bug.cgi?id=7698 Todd Rinaldo changed: What|Removed |Added Status|NEW |RESOLVED Resolution|--- |WONTFIX --- Comment #5 from Todd Rinaldo --- Closing for now. -- You are receiving this mail because: You are the assignee for the bug.
[Bug 7698] Prevent loading warnings.pm every eval statement
https://bz.apache.org/SpamAssassin/show_bug.cgi?id=7698 Kevin A. McGrail changed: What|Removed |Added CC||kmcgr...@apache.org --- Comment #4 from Kevin A. McGrail --- Yeah, we are back on make test working on 3.4 and trunk and I'm trying to sneak one more feature in to roll 3.4.3 -- You are receiving this mail because: You are the assignee for the bug.
[Bug 7698] Prevent loading warnings.pm every eval statement
https://bz.apache.org/SpamAssassin/show_bug.cgi?id=7698 --- Comment #3 from Henrik Krohns --- Future plan is to comb through all the eval stuff, perhaps get rid of it all. There's already a big change coming though for security reasons very soon. I know the code pretty well since I was forced to learn it for a month to get a grasp for it, so I'd like to make it much more simple, but we'll see.. -- You are receiving this mail because: You are the assignee for the bug.
[Bug 7698] Prevent loading warnings.pm every eval statement
https://bz.apache.org/SpamAssassin/show_bug.cgi?id=7698 Todd Rinaldo changed: What|Removed |Added CC||to...@cpanel.net --- Comment #2 from Todd Rinaldo --- Those are fair points. I think the fix I probably want is to get rid of the need for no warnings. Do you think that would be achievable? -- You are receiving this mail because: You are the assignee for the bug.
[Bug 7698] Prevent loading warnings.pm every eval statement
https://bz.apache.org/SpamAssassin/show_bug.cgi?id=7698 Henrik Krohns changed: What|Removed |Added CC||h...@hege.li --- Comment #1 from Henrik Krohns --- Did you actually benchmark this? I can not see any difference over 1% margin of error in debug timings with a massive ruleset. Also the original "no warnings" only appear in do_head_tests, but your patch affects _all_ calls of run_generic_tests. So it doesn't seem to make much sense to commit at this time. -- You are receiving this mail because: You are the assignee for the bug.