A-sync-RedisClient icon indicating copy to clipboard operation
A-sync-RedisClient copied to clipboard

Avoid temporary allocations

Open davidfowl opened this issue 6 years ago • 1 comments

The message parsing is all string based. While this is convenient it allocates strings to parse the redis command which is then turned into the resulting object. It would be better to parse the ReadOnlySequence directly in each of the result processors.

davidfowl avatar Sep 11 '19 14:09 davidfowl

@davidfowl I've changed it to only allocate a string if I'm going to use it for returning etc. Is this what you mean?

thomhurst avatar Sep 18 '19 10:09 thomhurst