Reuse Encoder/Decoder instance?

[msgpack]

Reuse Encoder/Decoder instance?

[gfx]

That's a good question. There's an example in test/reuse-instances.test.ts, but they only accepts positional parameters right now, so I might change their interfaces to accept named parameters just like as encode({...}) and decode({...}).

[buu700]

Oh, just to clarify, I wasn't asking about the documentation. I was suggesting an optimization to the default encode and decode functions / wondering whether there was a non-obvious reason that that hadn't been done.

[gfx]

Ah, sorry! Misunderstood.

There're some reasons that we cannot memorize them:

• instance.*Async() are not async-await safe (or not thread-safe)
• the instance has lots of configuration parameters so it's difficult to memoize

[buu700]

Got it, that makes sense. If memoizing isn't an option, what about just pre-initializing a default instance that gets used when options is undefined? That would at least cover the most common case (I'm assuming).

[gfx]

Sounds good. Will try it.

[buu700]

Nice, thanks! I'd be happy to submit a PR if that would be helpful.