diff options
| author | Matthew Dempsky <mdempsky@google.com> | 2015-02-25 14:41:21 +0900 |
|---|---|---|
| committer | Dmitry Vyukov <dvyukov@google.com> | 2015-02-26 08:59:24 +0000 |
| commit | 3c8a89daf3a05a1dd98075a733db7a20bef2dc5c (patch) | |
| tree | db477dc0bc924ff94ea7b334c5a9233df9c65a17 /src/runtime/extern.go | |
| parent | a32dd832530a176b5b45f06c3a97f52383227480 (diff) | |
| download | go-3c8a89daf3a05a1dd98075a733db7a20bef2dc5c.tar.gz go-3c8a89daf3a05a1dd98075a733db7a20bef2dc5c.zip | |
runtime: simplify CPU profiling code
This makes Go's CPU profiling code somewhat more idiomatic; e.g.,
using := instead of forward declaring variables, using "int" for
element counts instead of "uintptr", and slices instead of C-style
pointer+length. This makes the code easier to read and eliminates a
lot of type conversion clutter.
Additionally, in sigprof we can collect just maxCPUProfStack stack
frames, as cpuprof won't use more than that anyway.
Change-Id: I0235b5ae552191bcbb453b14add6d8c01381bd06
Reviewed-on: https://go-review.googlesource.com/6072
Run-TryBot: Matthew Dempsky <mdempsky@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Dmitry Vyukov <dvyukov@google.com>
Diffstat (limited to 'src/runtime/extern.go')
| -rw-r--r-- | src/runtime/extern.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/runtime/extern.go b/src/runtime/extern.go index cd90390b34..540d7b5124 100644 --- a/src/runtime/extern.go +++ b/src/runtime/extern.go @@ -114,7 +114,7 @@ func Caller(skip int) (pc uintptr, file string, line int, ok bool) { // and what it called, so that we can see if it // "called" sigpanic. var rpc [2]uintptr - if callers(1+skip-1, &rpc[0], 2) < 2 { + if callers(1+skip-1, rpc[:]) < 2 { return } f := findfunc(rpc[1]) @@ -161,7 +161,7 @@ func Callers(skip int, pc []uintptr) int { if len(pc) == 0 { return 0 } - return callers(skip, &pc[0], len(pc)) + return callers(skip, pc) } // GOROOT returns the root of the Go tree. |