fasthttp icon indicating copy to clipboard operation
fasthttp copied to clipboard

Published 20 hours ago • verified publisher icon valyala

Copy maps using Go 1.21's new "maps" package

Open 3JoB opened this issue 7 months ago • 1 comments

An attempted implementation of #1599 .

Also updated lint to 1.21, it seemed to have some issues before?

3JoB avatar Nov 08 '23 10:11 3JoB

What is the benefit of this? It just complicates the code and increases the size. And map.Copy just does exactly the same so there is no speed difference. I think it's only useful if we don't have to support older Go versions anymore which isn't soon.

erikdubbelboer avatar Nov 08 '23 22:11 erikdubbelboer

What is the benefit of this? It just complicates the code and increases the size. And map.Copy just does exactly the same so there is no speed difference. I think it's only useful if we don't have to support older Go versions anymore which isn't soon.

True! I didn't check out the code of maps.Copy when opening my issue and think this can be closed now after looking at it.

leonklingele avatar Feb 29 '24 08:02 leonklingele