tree 77c69c02ce475f32b2095e71b5cb8407868ac926
parent 9fd125535ba7ed3f3667a19aa42c340b1c9849af
author adonovan <adonovan@google.com> 1580246566 -0800
committer Copybara-Service <copybara-worker@google.com> 1580246623 -0800

bazel syntax: eliminate calls to StarlarkThread.getGlobals during evaluation

Calls during evaluation want the innermost enclosing Starlark source
file on the call stack, which is now provided by the ugly helper function
Module.ofInnermostEnclosingStarlarkFunction.

All remaining calls to getGlobals are called before or after
execution, so StarlarkThread call/return no longer needs to
save/restore this.globalFrame (now called this.module). These calls
will be eliminated once the StarlarkThread and Module construction API
is simplified.

This is a breaking API change for Copybara. References to StarlarkThread
have been replaced by Module where the one appeared to be a proxy for the other.

No behavior change.

PiperOrigin-RevId: 292000083