Re: Extended profile info

On 29.08.2012 07:00, Oto BREZINA wrote:
> On 2012-08-28 20:44, Stefan Küng wrote:
>> On 28.08.2012 14:37, Oto BREZINA wrote:
>>> Here is conceptual patch - may need some tweaks and fix - with extended
>>> Profile info. It adds User, Kernel and Wall time to report.
>>> User and Kernel Time are multiplied by threads. Windows returns their
>>> count is in 100ns units, but step may be quite bigger. Usualy I use this
>>> three vaules to determine efficiency.
>>>
>>> Pls review it and comment.
>>>
>> Looks good. But there's a small issue with the PROFILE_RUNTIMES define:
>>
>> if it's defined, all is well. But if it's not, you're still writing the
>> data to the file.
> Would you like to keep that switch ?

Not really, no. Unless the additional profiling slows down too much?

> Why CRecordProfileEvent::CRecordProfileEvent is defined once as _inline_
> for release and not for Debug ?
> Most debug setting disables inlining anyway, so there is no difference
> between those two. I'm I Right?

Not really, but it depends on the compiler used.
Stefan Fuhrmann wrote that class, so maybe he knows why he chose to do that.

Stefan

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.net
------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=757&dsMessageId=3002001
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].