From: Philip Oakley <philipoakley@iee.org>
Date: Fri, 20 Jun 2014 22:49:56 GMT
Subject: Re: [PATCH v6 00/11] add performance tracing facility
Message-ID: <CEA564C177884D1D86D69200A9A43AC8@PhilipOakley>
In-Reply-To: <53A4A195.1070502@gmail.com>

From: "Karsten Blees" <karsten.blees@gmail.com>
> Changes since v5:
> [05/11]: GIT_TRACE_BARE=1 disables 'timestamp file:line' output for
>         unit tests that rely on trace output (t1510 and t5503)
> [08/11]: Align original trace output at col 40
> [09/11]: Dropped '(div 10e9)' from the commit message.
> [10/11]: Dropped trace_performance[_since]() return value and the
>         respective usage example. Renamed trace_performance_vfl to
>         trace_performance_vprintf_fl.
>
> The other patches are the versions from pu.
>
> Karsten Blees (11):
>  trace: move trace declarations from cache.h to new trace.h
>  trace: consistently name the format parameter
>  trace: remove redundant printf format attribute
>  trace: factor out printing to the trace file
>  trace: add infrastructure to augment trace output with additional 
> info
>  trace: add current timestamp to all trace output
>  trace: move code around, in preparation to file:line output
>  trace: add 'file:line' to all trace output
>  trace: add high resolution timer function to debug performance issues
>  trace: add trace_performance facility to debug performance issues
>  git: add performance tracing for git's main() function to debug
>    scripts
>
> Makefile              |   7 ++
> cache.h               |  13 +--
> config.mak.uname      |   1 +
> git-compat-util.h     |   4 +
> git.c                 |   2 +
> t/t1510-repo-setup.sh |   2 +-
> t/t5503-tagfollow.sh  |   8 +-
> trace.c               | 313 
> ++++++++++++++++++++++++++++++++++++++++++++------
> trace.h               |  90 +++++++++++++++
> 9 files changed, 387 insertions(+), 53 deletions(-)

Should there be some documentation as well? Perhaps in t/README, or in
Documentation/howto.


> create mode 100644 trace.h
>
> -- 
Philip 

-- 
-- 
*** Please reply-to-all at all times ***
*** (do not pretend to know who is subscribed and who is not) ***
*** Please avoid top-posting. ***
The msysGit Wiki is here: https://github.com/msysgit/msysgit/wiki - Github accounts are free.

You received this message because you are subscribed to the Google
Groups "msysGit" group.
To post to this group, send email to msysgit@googlegroups.com
To unsubscribe from this group, send email to
msysgit+unsubscribe@googlegroups.com
For more options, and view previous threads, visit this group at
http://groups.google.com/group/msysgit?hl=en_US?hl=en

--- 
You received this message because you are subscribed to the Google Groups "msysGit" group.
To unsubscribe from this group and stop receiving emails from it, send an email to msysgit+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.