fusil  

Re: [fusil] [PATCH] Add -i option for displaying the instruction pointer

Mark Seaborn
Tue, 27 Jan 2009 11:29:03 -0800

Victor Stinner <victor.stin...@haypocalc.com> wrote:

> >          if self.options.show_pid:
> > -            text = "[%s] %s" % (syscall.process.pid, text)
> > +            text = "[pid %s] %s" % (syscall.process.pid, text)
> >          error(text)
>
> Why did you add "pid " prefix?  I prefer to keep a short prefix.

To distinguish it from the instruction pointer and for consistency
with regular strace.  But I don't have a strong preference.

Cheers,
Mark