Event Profiler Patch

Andreas Ericsson ae at op5.se
Mon Jun 8 17:30:23 CEST 2009


Steven D. Morrey wrote:
> I made it a compile time option in my patch for ease of insertion in
> the existing code base. It would be trivial to convert it to a config
> file option. Just add a new option and track it with a global
> variable, then search and replace all of the #ifdef EVENT_PROFILE
> tags with a simple if(gEvent_Profile_Enabled) After that it's just a
> matter of tracking down the unmatched #endif's and that should be it.
>  If there is interest in it, I can update the code to do this and
> submit a new patch.
> 

That would be preferrable, but please stick to the current style of
naming variables. "event_profiling_enabled" would be an acceptable
variable name.

-- 
Andreas Ericsson                   andreas.ericsson at op5.se
OP5 AB                             www.op5.se
Tel: +46 8-230225                  Fax: +46 8-230231

Considering the successes of the wars on alcohol, poverty, drugs and
terror, I think we should give some serious thought to declaring war
on peace.

------------------------------------------------------------------------------
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensing option that enables unlimited
royalty-free distribution of the report engine for externally facing 
server and web deployment.
http://p.sf.net/sfu/businessobjects




More information about the Developers mailing list