[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