> +void reparent(struct tcb *tcp, struct tcb *tcpchild) Put the function name on a new line, like all the other code in strace. Otherwise this looks fine to me. (It's just moving existing code without change.) Thanks, Roland