golang / geo

S2 geometry library in Go
Apache License 2.0
1.69k stars 182 forks source link

gofmt and add doc to CrossingEdgeQuery #47

Closed regeda closed 5 years ago

regeda commented 5 years ago

I've run gofmt to fix the style. Also, I've added a comment that CrossingEdgeQuery is not concurrent safe.

dsymonds commented 5 years ago

Thanks, but approximately no types in this library are safe for concurrent use, and that's the Go norm, so the comment isn't necessary. I can do that gofmt -s myself the next time we have Google-internal code to export, so I'll close rather than merge this PR. Thanks anyway.