[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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

From: Nala Ginrut
Subject: Re: [PATCH] Add new print style for repl trace
Date: Wed, 16 Jan 2013 20:55:20 +0800

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


reply via email to

[Prev in Thread] Current Thread [Next in Thread]