min_site_separation_ is unused outside of ChooseInitialSites
Open
- Dominant language
- C++
- Stars
- 2.7k
- Forks
- 357
- Avg merge
- 11h 4m
- Merged PRs (30d)
- 1
Description
I noticed that since c84fc2a0cea28d98fc005057754f58bb439506f3 `s2builder.cc` stopped using `min_site_separation_` outside `ChooseInitialSites`.
If that's on purpose then `min_site_separation_` could be a local variable inside inside `ChooseInitialSites` instead of class scoped.
Contributor guide
Assessment
This issue has not been assessed yet.