From 0abf443513a1399141a2fdac109c63630abf4d4b Mon Sep 17 00:00:00 2001 From: Russ Cox Date: Fri, 11 Dec 2015 02:49:16 -0500 Subject: [PATCH] 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 --- src/runtime/runtime2.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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 -- 2.50.0