Merge pull request #275 from dtoma/develop

Update hexify to use array lookup instead of ternary (#270)

Merge pull request #275 from dtoma/develop
Update hexify to use array lookup instead of ternary (#270)
814fb31d · Niels · GitHub · 7ffa07e3 · 5c129c89 · 814fb31d
Commit 814fb31d authored Jul 01, 2016 by Niels Committed by GitHub Jul 01, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 3 deletions

src/json.hpp.re2c src/json.hpp.re2c +5 -3

No files found.
--- a/src/json.hpp.re2c
+++ b/src/json.hpp.re2c
@@ -5959,9 +5959,11 @@ class basic_json
                        // (0..f)
                        const auto hexify = [](const int v) -> char
                        {
-                            return (v < 10)
+						    static const char hex[16] = { '0', '1', '2', '3',
-                            ? ('0' + static_cast<char>(v))
+                                                          '4', '5', '6', '7',
-                            : ('a' + static_cast<char>((v - 10) & 0x1f));
+                                                          '8', '9', 'a', 'b',
+                                                          'c', 'd', 'e', 'f' };
+						    return hex[v];
                        };
                        // print character c as \uxxxx