[Webkit-unassigned] [Bug 42637] new-run-webkit-tests: Support auto indentation based on call stack depth for debugging

Mon Jul 26 03:50:17 PDT 2010

https://bugs.webkit.org/show_bug.cgi?id=42637

--- Comment #6 from Hayato Ito <hayato at chromium.org>  2010-07-26 03:50:17 PST ---
Hi Dirk,
Thank you for the comments.

(In reply to comment #3)
> I agree that we shouldn't add another command line option for this. I am fine with just making this the default (in the verbose case; the non-verbose case shouldn't change at all as a result of this patch).
> 
> There's a small part of me that wonders if just the debug statements should be indented, so that ERROR/WARN/INFO stays on the left. I'd sort of have to see the different output and compare.

Okay. I've attached the two log files for the comparison:
1. All log level statements are indented.
2. Only debug level statements are indented.

I am not sure which is better. IMHO, it would be easily viewable if all log level statements are equally indented.
What do you think about them?

I don't have any particular opinion about that this auto-indent feature should be default for '--verbose'.
I am just afraid that someone does not like such an auto-indent.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.