From: Russ Cox Date: Fri, 11 Dec 2015 07:49:16 +0000 (-0500) Subject: runtime: remove incorrect TODO added in CL 16035 X-Git-Tag: go1.6beta1~57 X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=0abf443513a1399141a2fdac109c63630abf4d4b;p=gostls13.git runtime: remove incorrect TODO added in CL 16035 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 Reviewed-by: Ian Lance Taylor --- diff --git a/src/runtime/runtime2.go b/src/runtime/runtime2.go index 86ed846064..4a4d5f81ba 100644 --- a/src/runtime/runtime2.go +++ b/src/runtime/runtime2.go @@ -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