search

matt-42 / iod

Meta programming utilities for C++14. Merged in matt-42/lithium
MIT License
727 stars 58 forks source link

Fix JSON encoding of integers (issue #14) #15

Closed Kazuo256 closed 7 years ago

Kazuo256 commented 7 years ago

Fix #14.

This implementation relies on C++11's std::to_string. Is that alright?

In fact, it's possible to generalize this implementation to all types accepted by std::to_string.

matt-42 commented 7 years ago

Let's accept this event if this is 3x slower than the current implementation. I suggest that we switch to the faster std::to_chars (C++17) once it's available.