[dev] [patch] Bug#698 -- visual event status cues in Kronolith
views
Chuck Hagenbuch
chuck at horde.org
Sat Nov 27 21:30:43 PST 2004
Quoting "Derek P. Moore" <derekm at hackunix.org>:
> Here's a resubmittal of my kronolith/lib/Driver.php patch.
I take it this is the version you uploaded to the ticket?
> I suppose this amounts to a minor performance improvement, short
> circuiting test cases for a majority of events.
Please justify these with benchmarks. Unless there is a measurable difference,
you're likely either slowing things down with the extra lines of code, or just
confusing things for later authors. I'm guessing it doesn't impact measurable
speed one way or another (this would be different if there were significant
calculations that were being avoided, of course).
-chuck
--
"But she goes not abroad in search of monsters to destroy." - John
Quincy Adams
More information about the dev
mailing list