qbit/go
1
0
Fork 0
mirror of https://github.com/golang/go synced 2024-11-25 19:47:58 -07:00
Code Releases Activity

internal/pkgbits: add DerivedInfoNeeded

Browse Source 
So next CL can use it to remove unnecessary derivedInfo needed field.

Updates #68778

Change-Id: Ia4e0f638beaf4a448fbf10a9aa1bc9425349a5e8
Reviewed-on: https://go-review.googlesource.com/c/go/+/608215
Auto-Submit: Cuong Manh Le <cuong.manhle.vn@gmail.com>
Reviewed-by: Tim King <taking@google.com>
Reviewed-by: Robert Griesemer <gri@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
This commit is contained in:
Cuong Manh Le 2024-08-25 21:05:00 +07:00 committed by Gopher Robot
parent 2b0a157d68
commit c586dbb8b7
2 changed files with 9 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/internal/pkgbits/pkgbits_test.go
View File

@ -45,6 +45,8 @@ func TestVersions(t *testing.T) {
{pkgbits.V1, pkgbits.HasInit}, {pkgbits.V1, pkgbits.HasInit},
{pkgbits.V0, pkgbits.DerivedFuncInstance}, {pkgbits.V0, pkgbits.DerivedFuncInstance},
{pkgbits.V1, pkgbits.DerivedFuncInstance}, {pkgbits.V1, pkgbits.DerivedFuncInstance},
{pkgbits.V0, pkgbits.DerivedInfoNeeded},
{pkgbits.V1, pkgbits.DerivedInfoNeeded},
{pkgbits.V2, pkgbits.AliasTypeParamNames}, {pkgbits.V2, pkgbits.AliasTypeParamNames},
} { } {
if !c.v.Has(c.f) { if !c.v.Has(c.f) {
@ -57,6 +59,7 @@ func TestVersions(t *testing.T) {
{pkgbits.V0, pkgbits.Flags}, {pkgbits.V0, pkgbits.Flags},
{pkgbits.V2, pkgbits.HasInit}, {pkgbits.V2, pkgbits.HasInit},
{pkgbits.V2, pkgbits.DerivedFuncInstance}, {pkgbits.V2, pkgbits.DerivedFuncInstance},
{pkgbits.V2, pkgbits.DerivedInfoNeeded},
{pkgbits.V0, pkgbits.AliasTypeParamNames}, {pkgbits.V0, pkgbits.AliasTypeParamNames},
{pkgbits.V1, pkgbits.AliasTypeParamNames}, {pkgbits.V1, pkgbits.AliasTypeParamNames},
} { } {

6
src/internal/pkgbits/version.go
View File

@ -25,6 +25,7 @@ const (
// - remove the legacy "has init" bool from the public root // - remove the legacy "has init" bool from the public root
// - remove obj's "derived func instance" bool // - remove obj's "derived func instance" bool
// - add a TypeParamNames field to ObjAlias // - add a TypeParamNames field to ObjAlias
// - remove derived info "needed" bool
V2 V2
numVersions = iota numVersions = iota
@ -56,6 +57,10 @@ const (
// ObjAlias has a list of TypeParamNames. // ObjAlias has a list of TypeParamNames.
AliasTypeParamNames AliasTypeParamNames
// Deprecated: DerivedInfoNeeded was a bool indicating
// whether a type was a derived type.
DerivedInfoNeeded
numFields = iota numFields = iota
) )
@ -71,6 +76,7 @@ var introduced = [numFields]Version{
var removed = [numFields]Version{ var removed = [numFields]Version{
HasInit: V2, HasInit: V2,
DerivedFuncInstance: V2, DerivedFuncInstance: V2,
DerivedInfoNeeded: V2,
} }
// Has reports whether field f is present in a bitstream at version v. // Has reports whether field f is present in a bitstream at version v.