Avoid creating byte strings explicitly.

Previously, calls like njs_string_new(,,,0) produced byte strings by
explicitly providing zero length argument.

diff --git a/src/njs_builtin.c b/src/njs_builtin.c
index e4c8d73ce49ec93419306e2368eeb6ef18aa6370..6c38cbee07c54e0bf32e396bfa6a412c3a74cc7e 100644 (file)
--- a/src/njs_builtin.c
+++ b/src/njs_builtin.c
@@ -388,7 +388,7 @@ njs_builtin_traverse(njs_vm_t *vm, njs_traverse_t *traverse, void *data)
         return NJS_ERROR;
     }
 
-    ret = njs_string_new(vm, &prop->name, buf, p - buf, 0);
+    ret = njs_string_create(vm, &prop->name, buf, p - buf);
     if (njs_slow_path(ret != NJS_OK)) {
         return ret;
     }
@@ -590,7 +590,7 @@ njs_ext_dump(njs_vm_t *vm, njs_value_t *args, njs_uint_t nargs,
         return NJS_ERROR;
     }
 
-    return njs_string_new(vm, retval, str.start, str.length, 0);
+    return njs_string_create(vm, retval, str.start, str.length);
 }
 
 

diff --git a/src/njs_json.c b/src/njs_json.c
index a7a6c9fdf7e80eaead37a344bd281bf4eb86883b..4fef58a4bacc0d87bbd62dc022a96b5ad3244826 100644 (file)
--- a/src/njs_json.c
+++ b/src/njs_json.c
@@ -548,7 +548,6 @@ njs_json_parse_string(njs_json_parse_ctx_t *ctx, njs_value_t *value,
 {
     u_char        ch, *s, *dst;
     size_t        size, surplus;
-    ssize_t       length;
     uint32_t      utf, utf_low;
     njs_int_t     ret;
     const u_char  *start, *last;
@@ -742,12 +741,7 @@ njs_json_parse_string(njs_json_parse_ctx_t *ctx, njs_value_t *value,
         start = dst;
     }
 
-    length = njs_utf8_length(start, size);
-    if (njs_slow_path(length < 0)) {
-        length = 0;
-    }
-
-    ret = njs_string_new(ctx->vm, value, (u_char *) start, size, length);
+    ret = njs_string_create(ctx->vm, value, (u_char *) start, size);
     if (njs_slow_path(ret != NJS_OK)) {
         return NULL;
     }

diff --git a/src/njs_vm.c b/src/njs_vm.c
index 7183a24c7287c1961eff5e790a182002a73491be..b0fe353b518a1a43c37477756c88ed6a7c6e7ca6 100644 (file)
--- a/src/njs_vm.c
+++ b/src/njs_vm.c
@@ -874,7 +874,7 @@ njs_vm_bind2(njs_vm_t *vm, const njs_str_t *var_name, njs_object_prop_t *prop,
     njs_lvlhsh_t        *hash;
     njs_lvlhsh_query_t  lhq;
 
-    ret = njs_string_new(vm, &prop->name, var_name->start, var_name->length, 0);
+    ret = njs_string_create(vm, &prop->name, var_name->start, var_name->length);
     if (njs_slow_path(ret != NJS_OK)) {
         return NJS_ERROR;
     }