pre-allocate for Set() new component for lower memory usage

[JoshuaC215]

Extra allocations are done when using an empty bytes.Buffer here in createInsertComponents(). bytes.Buffer uses a normal grow strategy of successively doubling in size. This could be avoided by counting the length of the needed keys and the final set value and starting with a Buffer that is about the right size.

[buger]

Yes, it is a good idea!

I also think in terms of implementing multiple key set which will lead us to full marshaling implementation. When we set multiple keys, it make the first iteration, and find all keys first, and estimate how big new buffer should be, and on second iteration fills it with data.

[JoshuaC215]

Actually I dug into source and realized the bytes.Buffer provides a 64 byte initial slice for writing into. Given that, I think it's a tossup whether the complexity is worth it to pre-allocate.

[buger]

Fixed in v1.4.0 — Set now pre-allocates the output buffer using calcAllocateSpace, reducing allocations from 6 to 1 per call. See: https://github.com/buger/jsonparser/releases/tag/v1.4.0