From: Keith Randall <khr@golang.org>
Date: Sat, 9 Nov 2024 03:58:26 +0000 (-0800)
Subject: internal/runtime/maps: get rid of a few obsolete TODOs
X-Git-Tag: go1.24rc1~434
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=f9159b11e501f1b4cb79fe6807b97b1a1bbd2544;p=gostls13.git

internal/runtime/maps: get rid of a few obsolete TODOs

Change-Id: I7b3d95c0861ae2b6e0721b65aa75cda036435e9c
Reviewed-on: https://go-review.googlesource.com/c/go/+/625903
Reviewed-by: Keith Randall <khr@google.com>
Reviewed-by: Michael Pratt <mpratt@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
---

diff --git a/src/internal/runtime/maps/table.go b/src/internal/runtime/maps/table.go
index 7b3895c0a1..55c9879c3f 100644
--- a/src/internal/runtime/maps/table.go
+++ b/src/internal/runtime/maps/table.go
@@ -68,16 +68,11 @@ type table struct {
 	// locality, but it comes at the expense of wasted space for some types
 	// (consider uint8 key, uint64 element). Consider placing all keys
 	// together in these cases to save space.
-	//
-	// TODO(prattmic): Support indirect keys/values? This means storing
-	// keys/values as pointers rather than inline in the slot. This avoid
-	// bloating the table size if either type is very large.
 	groups groupsReference
 }
 
 func newTable(typ *abi.SwissMapType, capacity uint64, index int, localDepth uint8) *table {
 	if capacity < abi.SwissMapGroupSlots {
-		// TODO: temporary until we have a real map type.
 		capacity = abi.SwissMapGroupSlots
 	}