[PATCH] tracedump: add --help and --function-histogram

Carl Worth cworth at cworth.org
Thu Oct 20 09:24:18 PDT 2011


On Thu, 20 Oct 2011 15:58:18 +0200, José Fonseca <jose.r.fonseca at gmail.com> wrote:
> Portability fixes will be needed as you admited, talloc dependency
> removed as Zack mentioned, but implementation details apart, this is
> the direction I'd like to take. So thanks for taking the initiative.

You're quite welcome. For portability, I will need more specific
guidance on what the other platforms need. So I'll wait to hear details,
(which can probably wait until I have a closer-to-ready breanch).

> I'd prefer that you work on a branch on the main apitrace repos, so
> that I can also collaborate, in particular non-Linux OS portability
> fixes.  Do you have a github account?

I'm glad to work pretty much anywhere, but what difference does it make
which repository the branch is in? If you do:

	git remote add cworth git://git.cworth.org/git/apitrace

Then you can just do:

	git remote update

to update your code, (from both the upstream as well as my repository),
and it should be every bit as easy as if my branch lived in the same
repository. About the only difference is that my branch would be named
cworth/master rather than some other name. (And I can start using names
other than just "master" if I focus branches on specific topics.)

For what it's worth, I don't have a github account.     

> About talloc replacement, the best is a simple C++ class for path
> manipulation.  I actually started writing one to fix issue 46.
> 
> Getopt dependency is fine -- I'll just add the BSD getopt_long for
> Windows as done in Mesa.

Thanks for offering some portability fixes. I'll be quite happy to
collaborate on this stuff.

-Carl

-- 
carl.d.worth at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/apitrace/attachments/20111020/40508d6e/attachment.pgp>


More information about the apitrace mailing list