Stowage/go
2
0
Fork 0
mirror of https://github.com/golang/go.git synced 2025-12-08 06:10:04 +00:00
Code Issues Projects Releases Packages Wiki Activity

cmd/go: refactor usage of initialized

Browse source 
This commit refactors usage of the global variable `initialized` to
the global LoaderState field of the same name.

This commit is part of the overall effort to eliminate global
modloader state.

[git-generate]
cd src/cmd/go/internal/modload
rf 'ex { initialized -> LoaderState.initialized }'
rf 'rm initialized'

Change-Id: I97e35bab00f4c22661670b01b69425fc25efe6df
Reviewed-on: https://go-review.googlesource.com/c/go/+/698056
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Michael Matloob <matloob@golang.org>
Reviewed-by: Michael Matloob <matloob@google.com>
This commit is contained in:
Ian Alexander 2025-08-20 19:16:36 -04:00
parent 1d62e92567
commit 7bfeb43509
1 changed files with 7 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

15
src/cmd/go/internal/modload/init.go
View file

@ -60,7 +60,6 @@ var (
// Variables set in Init.
var (
initialized bool
// These are primarily used to initialize the MainModules, and should be
// eventually superseded by them but are still used in cases where the module
@ -406,7 +405,7 @@ func Reset() {
func setState(s State) State {
oldState := State{
initialized: initialized,
initialized: LoaderState.initialized,
forceUseModules: ForceUseModules,
rootMode: RootMode,
modRoots: modRoots,
@ -414,7 +413,7 @@ func setState(s State) State {
mainModules: MainModules,
requirements: requirements,
}
initialized = s.initialized
LoaderState.initialized = s.initialized
ForceUseModules = s.forceUseModules
RootMode = s.rootMode
modRoots = s.modRoots
@ -450,10 +449,10 @@ var LoaderState = NewState()
// configures the cfg, codehost, load, modfetch, and search packages for use
// with modules.
func Init() {
if initialized {
if LoaderState.initialized {
return
}
initialized = true
LoaderState.initialized = true
fips140.Init()
@ -573,7 +572,7 @@ func WillBeEnabled() bool {
// Already enabled.
return true
}
if initialized {
if LoaderState.initialized {
// Initialized, not enabled.
return false
}
@ -640,7 +639,7 @@ func VendorDir() string {
}
func inWorkspaceMode() bool {
if !initialized {
if !LoaderState.initialized {
panic("inWorkspaceMode called before modload.Init called")
}
if !Enabled() {
@ -1253,7 +1252,7 @@ func fixVersion(ctx context.Context, fixed *bool) modfile.VersionFixer {
// This function affects the default cfg.BuildMod when outside of a module,
// so it can only be called prior to Init.
func AllowMissingModuleImports() {
if initialized {
if LoaderState.initialized {
panic("AllowMissingModuleImports after Init")
}
allowMissingModuleImports = true