details: https://hg.nginx.org/njs/rev/8851a78723dc branches: changeset: 2330:8851a78723dc user: Dmitry Volyntsev <xei...@nginx.com> date: Fri May 17 21:54:50 2024 -0700 description: Removing njs_string_set() from builtin functions.
This is the last function that produced byte strings. diffstat: src/njs_builtin.c | 9 ++++----- src/njs_error.c | 2 +- src/njs_extern.c | 5 +++-- src/njs_parser.c | 2 +- src/njs_regexp.c | 4 ++-- src/njs_string.c | 4 ++-- src/njs_string.h | 3 ++- src/njs_vm.c | 4 ++-- src/test/njs_benchmark.c | 2 +- 9 files changed, 18 insertions(+), 17 deletions(-) diffs (159 lines): diff -r 4e0553f7ea68 -r 8851a78723dc src/njs_builtin.c --- a/src/njs_builtin.c Fri May 17 21:54:50 2024 -0700 +++ b/src/njs_builtin.c Fri May 17 21:54:50 2024 -0700 @@ -1201,8 +1201,8 @@ njs_process_object_argv(njs_vm_t *vm, nj i = 0; for (arg = vm->options.argv; i < vm->options.argc; arg++) { - njs_string_set(vm, &argv->start[i++], (u_char *) *arg, - njs_strlen(*arg)); + njs_string_create(vm, &argv->start[i++], (u_char *) *arg, + njs_strlen(*arg)); } prop = njs_object_prop_alloc(vm, &argv_string, &njs_value_undefined, 1); @@ -1265,7 +1265,7 @@ njs_env_hash_init(njs_vm_t *vm, njs_lvlh continue; } - ret = njs_string_create(vm, &prop->name, (char *) entry, val - entry); + ret = njs_string_create(vm, &prop->name, entry, val - entry); if (njs_slow_path(ret != NJS_OK)) { return NJS_ERROR; } @@ -1285,8 +1285,7 @@ njs_env_hash_init(njs_vm_t *vm, njs_lvlh val++; - ret = njs_string_create(vm, njs_prop_value(prop), (char *) val, - njs_strlen(val)); + ret = njs_string_create(vm, njs_prop_value(prop), val, njs_strlen(val)); if (njs_slow_path(ret != NJS_OK)) { return NJS_ERROR; } diff -r 4e0553f7ea68 -r 8851a78723dc src/njs_error.c --- a/src/njs_error.c Fri May 17 21:54:50 2024 -0700 +++ b/src/njs_error.c Fri May 17 21:54:50 2024 -0700 @@ -147,7 +147,7 @@ njs_error_stack_new(njs_vm_t *vm, njs_ob return ret; } - return njs_string_set(vm, retval, string.start, string.length); + return njs_string_create(vm, retval, string.start, string.length); } diff -r 4e0553f7ea68 -r 8851a78723dc src/njs_extern.c --- a/src/njs_extern.c Fri May 17 21:54:50 2024 -0700 +++ b/src/njs_extern.c Fri May 17 21:54:50 2024 -0700 @@ -74,8 +74,9 @@ njs_external_add(njs_vm_t *vm, njs_arr_t lhq.key_hash = external->name.symbol; } else { - ret = njs_string_set(vm, &prop->name, external->name.string.start, - external->name.string.length); + ret = njs_string_create(vm, &prop->name, + external->name.string.start, + external->name.string.length); if (njs_slow_path(ret != NJS_OK)) { return NJS_ERROR; } diff -r 4e0553f7ea68 -r 8851a78723dc src/njs_parser.c --- a/src/njs_parser.c Fri May 17 21:54:50 2024 -0700 +++ b/src/njs_parser.c Fri May 17 21:54:50 2024 -0700 @@ -9242,7 +9242,7 @@ njs_parser_error(njs_vm_t *vm, njs_objec njs_value_property_set(vm, &error, njs_value_arg(&line_number), &value); if (file->length != 0) { - ret = njs_string_set(vm, &value, file->start, file->length); + ret = njs_string_create(vm, &value, file->start, file->length); if (ret == NJS_OK) { njs_value_property_set(vm, &error, njs_value_arg(&file_name), &value); diff -r 4e0553f7ea68 -r 8851a78723dc src/njs_regexp.c --- a/src/njs_regexp.c Fri May 17 21:54:50 2024 -0700 +++ b/src/njs_regexp.c Fri May 17 21:54:50 2024 -0700 @@ -1144,8 +1144,8 @@ njs_regexp_exec_result(njs_vm_t *vm, njs do { group = &pattern->groups[i]; - ret = njs_string_set(vm, &name, group->name.start, - group->name.length); + ret = njs_string_create(vm, &name, group->name.start, + group->name.length); if (njs_slow_path(ret != NJS_OK)) { goto fail; } diff -r 4e0553f7ea68 -r 8851a78723dc src/njs_string.c --- a/src/njs_string.c Fri May 17 21:54:50 2024 -0700 +++ b/src/njs_string.c Fri May 17 21:54:50 2024 -0700 @@ -127,7 +127,7 @@ njs_string_set(njs_vm_t *vm, njs_value_t njs_int_t -njs_string_create(njs_vm_t *vm, njs_value_t *value, const char *src, +njs_string_create(njs_vm_t *vm, njs_value_t *value, const u_char *src, size_t size) { u_char *p, *p_end; @@ -3152,7 +3152,7 @@ njs_string_get_substitution(njs_vm_t *vm p += 2; - ret = njs_string_create(vm, &name, (const char *) p, r - p); + ret = njs_string_create(vm, &name, p, r - p); if (njs_slow_path(ret != NJS_OK)) { goto exception; } diff -r 4e0553f7ea68 -r 8851a78723dc src/njs_string.h --- a/src/njs_string.h Fri May 17 21:54:50 2024 -0700 +++ b/src/njs_string.h Fri May 17 21:54:50 2024 -0700 @@ -103,13 +103,14 @@ typedef enum { } njs_trim_t; +/* njs_string_set() Is deprecated. */ njs_int_t njs_string_set(njs_vm_t *vm, njs_value_t *value, const u_char *start, uint32_t size); u_char *njs_string_alloc(njs_vm_t *vm, njs_value_t *value, uint64_t size, uint64_t length); njs_int_t njs_string_new(njs_vm_t *vm, njs_value_t *value, const u_char *start, uint32_t size, uint32_t length); -njs_int_t njs_string_create(njs_vm_t *vm, njs_value_t *value, const char *src, +njs_int_t njs_string_create(njs_vm_t *vm, njs_value_t *value, const u_char *src, size_t size); njs_int_t njs_string_create_chb(njs_vm_t *vm, njs_value_t *value, njs_chb_t *chain); diff -r 4e0553f7ea68 -r 8851a78723dc src/njs_vm.c --- a/src/njs_vm.c Fri May 17 21:54:50 2024 -0700 +++ b/src/njs_vm.c Fri May 17 21:54:50 2024 -0700 @@ -843,7 +843,7 @@ njs_vm_value(njs_vm_t *vm, const njs_str return NJS_ERROR; } - ret = njs_string_set(vm, &key, start, size); + ret = njs_string_create(vm, &key, start, size); if (njs_slow_path(ret != NJS_OK)) { return NJS_ERROR; } @@ -977,7 +977,7 @@ njs_int_t njs_vm_value_string_create(njs_vm_t *vm, njs_value_t *value, const u_char *start, uint32_t size) { - return njs_string_create(vm, value, (const char *) start, size); + return njs_string_create(vm, value, start, size); } diff -r 4e0553f7ea68 -r 8851a78723dc src/test/njs_benchmark.c --- a/src/test/njs_benchmark.c Fri May 17 21:54:50 2024 -0700 +++ b/src/test/njs_benchmark.c Fri May 17 21:54:50 2024 -0700 @@ -715,7 +715,7 @@ njs_benchmark_string(njs_vm_t *vm, njs_v } else if (memcmp(mode.start, "create", 6) == 0) { for (i = 0; i < n; i++) { - njs_string_create(vm, &value, (char *) s.start, s.length); + njs_string_create(vm, &value, s.start, s.length); } } else { _______________________________________________ nginx-devel mailing list nginx-devel@nginx.org https://mailman.nginx.org/mailman/listinfo/nginx-devel