Back after a little break
Carl Worth
cworth at cworth.org
Sat Jan 26 16:16:09 PST 2013
Hi apitrace folks,
I've been quiet around here for a while, but I'm changing that now.
José, thanks for merging so much of my trim code at the end of last
year. I'll be following up with some other code I have on hand (a big
optimization for the dependency tracking and a --trim-spec option so
that the user can specify a subset of what kinds of automatic trimming
should be performed).
I've been using commits for both of those for quite some time now, so I
think they're ready. But I'll rebase them onto the latest master along
with changes to the test suite to track them carefully.
But first, I want to seriously propose "apitrace replay" as the name for
the interface that replays a trace. I submitted this originally as
"replay" and José changed it afterwards to "retrace". He had mentioned
earlier in email that he preferred "retrace" for consistency with the
existing implementation.
I suggest that for a user, the name of the interface should be as "good"
as possible and "replay" is objectively better than "retrace". The user
also doesn't see the implementation so will not be aware of any
inconsistency.
But if consistency is what's preventing a good name for the interface,
I'm happy to fix that as well. I've written patches to rename everything
in the implementation and the test suite from "retrace" to "replay" and
the test suite still passes.
I'll email these patches[*] out now, and I'll be interested in feedback.
-Carl
[*] I'm using email for this since I don't have good network
connectivity at the moment and it's much easier for me to queue up email
(which will just go out as soon as I'm connected again) than to queue up
git pushes.
That said, José, I'm willing to revisit other collaboration
strategies. Earlier you said you would prefer to work together on
feature branches within the main repository. I'm willing to experiment
with that, so let me know what you need from me to get that setup.
-------------- 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/20130127/588d481a/attachment.pgp>
More information about the apitrace
mailing list