Re: [PATCH] Add new print style for repl trace

[Nala Ginrut]

hi Wingo!

Yes,  it's a simpler solution for this, and less codes added.

But I still worried that folks may need to track the procedure call-stack depth for debugging purpose. If we just limit '|||...' counting,  it could be implicit for the tracing, only ellipsis left...

What do you think?

On Wed, Jan 16, 2013 at 8:22 PM, Andy Wingo <address@hidden> wrote:

On Tue 28 Feb 2012 11:19, Nala Ginrut <address@hidden> writes:

> I got extremely painful when I's tracing a complicated procedure. So I
> decided to do something to alleviate this pain.
> And I added a new print style for the REPL trace. It'll show level count
> number instead lots of "|  |  |......" which makes me drag my console
> window very very long. But this patch won't eliminate all the tracing
> pains as all guys think. 
> And Andy Wingo encourage me to make this count style as default, so I
> did. Anyway, there's an option for old style I'll show you later.

It seems to me that adding a "mode" argument is a way of avoiding a
decision that we should in fact be making.  How about the following
patch, which simply limits the length of the "|  " prefix to the width,
falling back to the counting method if the indentation prefix is too
wide:



--
http://wingolog.org/