github-vet / github-vet/rangeloop-pointer-findings
miuer/ncepu-work: architecture/c2/component/component.go; 14 LoC
- Dominant language
- No language data
- Stars
- 0
- Forks
- 0
- PR merge metrics
- PR metrics pending
Description
Found a possible issue in [miuer/ncepu-work](https://www.github.com/miuer/ncepu-work) at [architecture/c2/component/component.go](https://github.com/miuer/ncepu-work/blob/aa2dd7bfb438e20e727ee4180dd82060be0dca26/architecture/c2/component/component.go#L16-L29)
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first issue it finds, so please do not limit your consideration to the contents of the below message.
> range-loop variable cal used in defer or goroutine at line 20
[Click here to see the code in its original context.](https://github.com/miuer/ncepu-work/blob/aa2dd7bfb438e20e727ee4180dd82060be0dca26/architecture/c2/component/component.go#L16-L29)
Click here to show the 14 line(s) of Go which triggered the analyzer.
```go
for _, cal := range c2.Calculation {
// create a calculation goroutinue
go func() {
log.Print("the calcution is:", *cal)
ch <- cal.Run(c2.Result)
}()
// The main process captures the value of calculation component
// then create a new calculation goroutinue
c2.Result = <-ch
log.Println("the result is:", c2.Result)
}
```
Leave a reaction on this issue to contribute to the project by classifying this instance as a **Bug** :-1:, **Mitigated** :+1:, or **Desirable Behavior** :rocket:
See the descriptions of the classifications [here](https://github.com/github-vet/rangeclosure-findings#how-can-i-help) for more information.
commit ID: aa2dd7bfb438e20e727ee4180dd82060be0dca26
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.