From: Alex Brainman <alex.brainman@gmail.com>
Date: Wed, 18 Oct 2017 04:40:32 +0000 (+1100)
Subject: cmd/link: add TODO missing from CL 70310
X-Git-Tag: go1.10beta1~663
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=8ce8143f104a72718bdd8a53b4f8d3deba0df47e;p=gostls13.git

cmd/link: add TODO missing from CL 70310

CL 70310 dropped TODO while moving code.
Add TODO back, so we do not forget.

Change-Id: I3599ac02743bd35fb9556fdc238e9c72cf7f718f
Reviewed-on: https://go-review.googlesource.com/71590
Reviewed-by: David Crawshaw <crawshaw@golang.org>
---

diff --git a/src/cmd/link/internal/ld/pe.go b/src/cmd/link/internal/ld/pe.go
index fc97bfbaf1..f870d5d917 100644
--- a/src/cmd/link/internal/ld/pe.go
+++ b/src/cmd/link/internal/ld/pe.go
@@ -93,6 +93,7 @@ const (
 
 // TODO(crawshaw): add these constants to debug/pe.
 const (
+	// TODO: the Microsoft doco says IMAGE_SYM_DTYPE_ARRAY is 3 and IMAGE_SYM_DTYPE_FUNCTION is 2
 	IMAGE_SYM_TYPE_NULL      = 0
 	IMAGE_SYM_TYPE_STRUCT    = 8
 	IMAGE_SYM_DTYPE_FUNCTION = 0x20