use Sync.once opt the lazy init func

[imxyb]

No description provided.

[stevenh]

Cleaner but does it result in better performance?

Could you create a benchmark to prove either way?

[imxyb]

Cleaner but does it result in better performance?

Could you create a benchmark to prove either way?

I have used BenchmarkPoolGet to compare, and the performance is almost the same as before. This change can make the code cleaner and more concise

[FZambia]

btw, sync.Once internally also has a fast path based on the atomic comparison.

[imxyb]

btw, sync.Once internally also has a fast path based on the atomic comparison.

yes, i know, this change is only make the code more cleaner and readable

[stevenh]

Thanks for confirming.