[PATCH v4 1/4] Option to print only failing syscalls

[Dmitry V. Levin]

On Tue, Jun 18, 2019 at 08:53:31AM +0000, Kohl, Burkhard wrote:
[...]
> > @@ -1585,7 +1587,7 @@ init(int argc, char *argv[])
> >  #ifdef ENABLE_STACKTRACE
> >  	    "k"
> >  #endif
> > -	    "a:Ab:cCdDe:E:fFhiI:o:O:p:P:qrs:S:tTu:vVwxX:yz")) != EOF) {
> > +	    "a:Ab:cCdDe:E:fFhiI:o:O:p:P:qrs:S:tTu:vVwxX:yzZ")) != EOF) {
> >  		switch (c) {
> >  		case 'a':
> >  			acolumn = string_to_uint(optarg);
> > @@ -1716,6 +1718,9 @@ init(int argc, char *argv[])
> >  		case 'z':
> >  			not_failing_only = 1;
> >  			break;
> > +		case 'Z':
> > +			failing_only = 1;
> > +			break;
> >  		default:
> >  			error_msg_and_help(NULL);
> >  			break;
> 
> This allows specifying both option -zZ / -Zz, but then the results depend on the
> sequence in which these options appear on the cmdline,

I don't think this is the case when the whole patchset is applied.

This patch alone is of course incomplete, as well as the second patch
in the series.  Only the third patch makes things work properly.

> e.g.:
> 
>     wc -l *.trc:

Sorry, what is *.trc?

> These two options should be made mutually exclusive

Why should they?


-- 
ldv
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://lists.strace.io/pipermail/strace-devel/attachments/20190618/6cfb7b76/attachment.bin>