qbit/go
1
0
Fork 0
mirror of https://github.com/golang/go synced 2024-09-30 02:14:29 -06:00
Code Releases Activity

runtime: remove incorrect TODO added in CL 16035

Browse Source 
I've already turned away one attempt to remove this field.
As the comment above the struct says, many tools know the layout.
The field cannot simply be removed.

It was one thing to remove the fields name, but the TODO should
not have been added.

Change-Id: If40eacf0eb35835082055e129e2b88333a0731b9
Reviewed-on: https://go-review.googlesource.com/17741
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
This commit is contained in:
Russ Cox 2015-12-11 02:49:16 -05:00
parent bdd5d6e257
commit 0abf443513
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/runtime/runtime2.go
View File

@ -497,7 +497,7 @@ type _func struct {
nameoff int32 // function name
args int32 // in/out args size
_ int32 // Previously: legacy frame size. TODO: Remove this.
_ int32 // previously legacy frame size; kept for layout compatibility
pcsp int32
pcfile int32