diff options
| author | Russ Cox <rsc@golang.org> | 2014-04-02 23:00:40 -0400 |
|---|---|---|
| committer | Russ Cox <rsc@golang.org> | 2014-04-02 23:00:40 -0400 |
| commit | f98f22011b10d475bf903d6acd14f84d5e33f7dc (patch) | |
| tree | 022b43bae9c770dc3d9cca083793e2b29b8dba98 /src/pkg/runtime/traceback_arm.c | |
| parent | 2eaff7c80cce076ec47b8a92ad82f29e9ff3d7be (diff) | |
| download | go-f98f22011b10d475bf903d6acd14f84d5e33f7dc.tar.gz | |
runtime: print up to 10 words of arguments
The old limit of 5 was chosen because we didn't actually know how
many bytes of arguments there were; 5 was a halfway point between
printing some useful information and looking ridiculous.
Now we know how many bytes of arguments there are, and we stop
the printing when we reach that point, so the "looking ridiculous" case
doesn't happen anymore: we only print actual argument words.
The cutoff now serves only to truncate very long (but real) argument lists.
In multiple debugging sessions recently (completely unrelated bugs)
I have been frustrated by not seeing more of the long argument lists:
5 words is only 2.5 interface values or strings, and not even 2 slices.
Double the max amount we'll show.
LGTM=bradfitz
R=golang-codereviews, bradfitz
CC=golang-codereviews, iant, r
https://codereview.appspot.com/83850043
Diffstat (limited to 'src/pkg/runtime/traceback_arm.c')
| -rw-r--r-- | src/pkg/runtime/traceback_arm.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/pkg/runtime/traceback_arm.c b/src/pkg/runtime/traceback_arm.c index f5cd4133d..dd77fcdfd 100644 --- a/src/pkg/runtime/traceback_arm.c +++ b/src/pkg/runtime/traceback_arm.c @@ -156,7 +156,7 @@ runtime·gentraceback(uintptr pc0, uintptr sp0, uintptr lr0, G *gp, int32 skip, tracepc -= sizeof(uintptr); runtime·printf("%s(", runtime·funcname(f)); for(i = 0; i < frame.arglen/sizeof(uintptr); i++) { - if(i >= 5) { + if(i >= 10) { runtime·prints(", ..."); break; } |