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

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Jul 27 01:14:19 PDT 2010


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





--- Comment #10 from Hayato Ito <hayato at chromium.org>  2010-07-27 01:14:18 PST ---
Thank you for the comment.

(In reply to comment #8)
> Actually, now that I look at the output, it's not clear to me how this would help in debugging. It's not like it gives you a call stack or anything. What cases would this be helpful in?

I agree your concern.
For those who starts reading new-run-webkit-test code, such an indentation would be useful especially when a lot of _log.debug were inserted, as I actually did to understand the flow of new-run-webkit-test.

Once you are familiar with new-run-webkit-test code base, it would not be so useful than before.

I am afraid that making it default for '--verbose' is annoying for a lot of guys.
In a meantime, I'd like to hear more opinions. I am not confident that this patch is worth to be added.

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



More information about the webkit-unassigned mailing list