Pull request: Dependency-based trim

[Carl Worth]

José Fonseca <jose.r.fonseca at gmail.com> writes:
> Sorry for not replying sooner.

No problem at all.

> I don't object merging this, even if it's not perfect. Just two requests:
> - move all trim dependency tracking to a separate module (e.g.,
> cli_trim_foo.cpp)
> - issue a warning that dependency tracking is experimental and might
> not work in all cases

OK. Those two are easy enough. I'll let you know when I've got that done
this afternoon, and then you can pull the whole mess.

> I'll look into more detail as time permits.

Thanks, I'll look forward to any comments you might have. Obviously,
most everything new is inside the new module, so almost totally isolated
From everything else. The things that touch other modules are mostly
features I needed while adding testing and are all quite uncontroversial
I think:

  common/trace_option:
  cli/dump:
	Share code for parsing Boolean command-line options

  common/os_posix:
	Print attempted command line if execvp fails

  common/trace_callset:
	Track first and last call over callset for early rejection

  common/trace_model:
  common/trace_parser:
  common/trace_parser_flags:
	Rename FLAG_VERBOSE to FLAG_UNINTERESTING

  common/trace_wrapper:
        Add findProgram() function like findWrapper() and findScript()

  retrace/glws_glx:
        Quit on keypress of 'Q' or 'ESC'

  retrace/retrace_main:
	Convert from ad-hoc command-line parsing to getopt

  cli/dump-images:
	Add --call-nos=no option for image names without call numbers

  scripts/tracediff:
        Don't colorize if not targeting a tty

So there's a little quick-access information for your review without the
need to dive into all ~50 commits.

-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/20120817/3d51e4ff/attachment.pgp>