stephen-gardner/MOTHER: convinit.go; 7 LoC

Found a possible issue in [stephen-gardner/MOTHER](https://www.github.com/stephen-gardner/MOTHER) at [convinit.go](https://github.com/stephen-gardner/MOTHER/blob/3b05a98cfd46ba64f821c0897ae075dd0889948d/convinit.go#L119-L125)

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.

> reference to prev is reassigned at line 121

[Click here to see the code in its original context.](https://github.com/stephen-gardner/MOTHER/blob/3b05a98cfd46ba64f821c0897ae075dd0889948d/convinit.go#L119-L125)

Click here to show the 7 line(s) of Go which triggered the analyzer.

```go
for _, prev := range ctx.mom.Conversations {
if prev.Active && prev.DirectID == ctx.conv.DirectID {
ctx.prev = &prev
ctx.switched = true
return
}
}

```

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: 3b05a98cfd46ba64f821c0897ae075dd0889948d

Start with convinit.go lines 119-125 and review how the range-loop variable prev is referenced when assigning ctx.prev. Inspect the surrounding conversation-switching logic and Go range-variable behavior, then classify the finding as Bug, Mitigated, or Desirable Behavior; no patch target is specified.

Written by the indexing model from the issue text.