json_udf slowdown

don't call strlen() in the loop

json_udf slowdown
don't call strlen() in the loop
79fc519e · Sergei Golubchik · ef125e23 · 79fc519e
Commit 79fc519e authored Jul 12, 2016 by Sergei Golubchik
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

storage/connect/json.cpp storage/connect/json.cpp +2 -2

No files found.
--- a/storage/connect/json.cpp
+++ b/storage/connect/json.cpp
@@ -767,7 +767,7 @@ bool JOUTSTR::Escape(const char *s)
 {
  WriteChr('"');
-  for (unsigned int i = 0; i < strlen(s); i++)
+  for (unsigned int i = 0; s[i]; i++)
    switch (s[i]) {
      case '"':  
      case '\\':
@@ -816,7 +816,7 @@ bool JOUTFILE::Escape(const char *s)
  // This is temporary
  fputc('"', Stream);
-  for (unsigned int i = 0; i < strlen(s); i++)
+  for (unsigned int i = 0; s[i]; i++)
    switch (s[i]) {
      case '"':  fputs("\\\"", Stream); break;
      case '\\': fputs("\\\\", Stream); break;