[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