On Mon, Mar 29, 2010 at 05:10:50AM +0400, Dmitry V. Levin wrote: > On Sun, Mar 28, 2010 at 02:39:38PM +0100, Adrien Kunysz wrote: > > Back in August I posted this patch that implements a new -C option. I > > got some feedback and reworked the patch accordingly but I got no follow > > up after the third iteration. Can you please consider this again or tell > > me what's wrong with this? This version applies cleanly against the > > current master branch. > > > > This combine regular output with -c. I would find that useful to be > > integrated in the master branch in order to get rid of some custom, > > unreliable scripts I (and doubtlessy others) use to analyze regular > > strace output. > > The patch looked OK, but, unfortunately, only after the first glance. > There are a lot of severe problems in trace_syscall() part of it. > You can find a fixed patch at > http://strace.git.sourceforge.net/git/gitweb.cgi?p=strace/strace;a=commitdiff;h=ldv/option-C
No comments yet? -- ldv
pgpuGr01HdRAu.pgp
Description: PGP signature
------------------------------------------------------------------------------ Download Intel® Parallel Studio Eval Try the new software tools for yourself. Speed compiling, find bugs proactively, and fine-tune applications for parallel performance. See why Intel Parallel Studio got high marks during beta. http://p.sf.net/sfu/intel-sw-dev
_______________________________________________ Strace-devel mailing list Strace-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/strace-devel
