On Tue, Mar 08, 2016 at 10:55:47AM +0530, haris iqbal wrote:
> if (tcp->qual_flg & QUAL_RAW) {
> /* sys_res = printargs(tcp); - but it's nop on sysexit */
> } else {
> /* FIXME: not_failing_only (IOW, option -z) is broken:
> * failure of syscall is known only after syscall return.
> * Thus we end up with something like this on, say, ENOENT:
> * open("doesnt_exist", O_RDONLY <unfinished ...>
> * {next syscall decode}
> * whereas the intended result is that open(...) line
> * is not shown at all.
> */
> if (not_failing_only && tcp->u_error)
> goto ret; /* ignore failed syscalls */
> if (tcp->sys_func_rval & RVAL_DECODED)
> sys_res = tcp->sys_func_rval;
> else
> sys_res = tcp->s_ent->sys_func(tcp);
> }
>
> This condition check in if seems redundant. Should I remove it withThe code you are talking about is a comment. > if (!(tcp->qual_flg & QUAL_RAW)) Why do you think this would be any different from the above? -- ldv
pgpClNKdKfh9a.pgp
Description: PGP signature
------------------------------------------------------------------------------ Transform Data into Opportunity. Accelerate data analysis in your applications with Intel Data Analytics Acceleration Library. Click to learn more. http://pubads.g.doubleclick.net/gampad/clk?id=278785111&iu=/4140
_______________________________________________ Strace-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/strace-devel
