tree 9090e3b104c1c9f0f32b515da2db5166e7433c0e
parent 2005adbe0cf62429a4256148b0c8afd8f6feeca4
author Michael Stapelberg <stapelberg@golang.org> 1737119898 +0100
committer Gopher Robot <gobot@golang.org> 1737122505 -0800

internal_gengo: avoid allocations in rawDescGZIP() accessors

Use unsafeBytes in rawDescGZIP() as well, which is safe because our
CompressGZIP() does not write to the byte slice.

Store the result of CompressGZIP as a byte slice so that subsequent
calls do not cause any more allocations.

In http://go.dev/cl/638135, I asserted that rawDescGZIP() is rarely-used,
but after rolling out the change Google-internally, affected users made
me aware of a few programs that heavily access these descriptors.

Change-Id: Ieb5010ddc7b9ac6be88970321ff01a3d29e846bf
Reviewed-on: https://go-review.googlesource.com/c/protobuf/+/643276
Reviewed-by: Chressie Himpel <chressie@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Auto-Submit: Nicolas Hillegeer <aktau@google.com>
Reviewed-by: Nicolas Hillegeer <aktau@google.com>
Commit-Queue: Nicolas Hillegeer <aktau@google.com>