diff options
| author | Aleksandr Nogikh <nogikh@google.com> | 2021-07-14 17:01:48 +0000 |
|---|---|---|
| committer | Aleksandr Nogikh <wp32pw@gmail.com> | 2021-07-20 13:39:45 +0200 |
| commit | b5969a9bdd1769ae7dc56682c4dbb8bc8f2ab7c3 (patch) | |
| tree | 1859559654383d8f1b771ece935aca6a1b6371c0 /pkg/build/gvisor.go | |
| parent | bc48c9ab65ed47d707bedef64ca52e1a5c383250 (diff) | |
all: remove pointers to pkg.build.Params
The struct pkg.build.Params is currently primarily passed on as a
pointer, which leads make it hard to see the places, where it can
(and should) actually be modified.
Make it all more explicit by only passing pointer references to objects
of this type when the object is expected to be modified by the function.
In fact, at this moment there are no such situations.
Diffstat (limited to 'pkg/build/gvisor.go')
| -rw-r--r-- | pkg/build/gvisor.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/pkg/build/gvisor.go b/pkg/build/gvisor.go index f26a09346..0925a7b7c 100644 --- a/pkg/build/gvisor.go +++ b/pkg/build/gvisor.go @@ -19,7 +19,7 @@ type gvisor struct{} var bazelTargetPath = regexp.MustCompile(`(?sm:.*^)\s*Outputs: \[(.*)\](?sm:$.*)`) -func (gvisor gvisor) build(params *Params) error { +func (gvisor gvisor) build(params Params) error { if params.Compiler == "" { params.Compiler = "bazel" } |
