details: https://github.com/nginx/njs/commit/264b1ced4259db93f928f57dfbddcc5e3f93d68b branches: master commit: 264b1ced4259db93f928f57dfbddcc5e3f93d68b user: Zhidao HONG <z.h...@f5.com> date: Mon, 21 Apr 2025 23:37:49 +0800 description: Replaced vm with mp in NJS_CHB_MP_INIT().
--- external/njs_query_string_module.c | 6 +++--- external/njs_xml_module.c | 2 +- external/njs_zlib_module.c | 4 ++-- nginx/ngx_http_js_module.c | 4 ++-- nginx/ngx_js_fetch.c | 8 ++++---- src/njs_array.c | 2 +- src/njs_chb.h | 4 ++-- src/njs_error.c | 2 +- src/njs_function.c | 2 +- src/njs_json.c | 4 ++-- src/njs_regexp.c | 2 +- src/njs_string.c | 8 ++++---- src/njs_typed_array.c | 2 +- src/njs_vm.c | 2 +- src/test/njs_benchmark.c | 2 +- src/test/njs_externals_test.c | 2 +- src/test/njs_unit_test.c | 2 +- 17 files changed, 29 insertions(+), 29 deletions(-) diff --git a/external/njs_query_string_module.c b/external/njs_query_string_module.c index f9b7c552..e2f849d2 100644 --- a/external/njs_query_string_module.c +++ b/external/njs_query_string_module.c @@ -152,7 +152,7 @@ njs_query_string_decode(njs_vm_t *vm, njs_value_t *value, const u_char *start, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, }; - NJS_CHB_MP_INIT(&chain, vm); + NJS_CHB_MP_INIT(&chain, njs_vm_memory_pool(vm)); njs_utf8_decode_init(&ctx); cp = 0; @@ -749,7 +749,7 @@ njs_query_string_stringify(njs_vm_t *vm, njs_value_t *args, njs_uint_t nargs, encode = njs_value_function(val); } - NJS_CHB_MP_INIT(&chain, vm); + NJS_CHB_MP_INIT(&chain, njs_vm_memory_pool(vm)); keys = njs_vm_object_keys(vm, object, njs_value_arg(&value)); if (njs_slow_path(keys == NULL)) { @@ -841,7 +841,7 @@ njs_query_string_escape(njs_vm_t *vm, njs_value_t *args, njs_uint_t nargs, njs_value_string_get(string, &str); - NJS_CHB_MP_INIT(&chain, vm); + NJS_CHB_MP_INIT(&chain, njs_vm_memory_pool(vm)); ret = njs_query_string_encode(&chain, &str); if (njs_slow_path(ret != NJS_OK)) { diff --git a/external/njs_xml_module.c b/external/njs_xml_module.c index 6bdbb639..e524ba5c 100644 --- a/external/njs_xml_module.c +++ b/external/njs_xml_module.c @@ -1785,7 +1785,7 @@ njs_xml_ext_canonicalization(njs_vm_t *vm, njs_value_t *args, njs_uint_t nargs, } } - NJS_CHB_MP_INIT(&chain, vm); + NJS_CHB_MP_INIT(&chain, njs_vm_memory_pool(vm)); buf = xmlOutputBufferCreateIO(njs_xml_buf_write_cb, NULL, &chain, NULL); if (njs_slow_path(buf == NULL)) { diff --git a/external/njs_zlib_module.c b/external/njs_zlib_module.c index 7945e646..d3a667d8 100644 --- a/external/njs_zlib_module.c +++ b/external/njs_zlib_module.c @@ -320,7 +320,7 @@ njs_zlib_ext_deflate(njs_vm_t *vm, njs_value_t *args, njs_uint_t nargs, } } - NJS_CHB_MP_INIT(&chain, vm); + NJS_CHB_MP_INIT(&chain, njs_vm_memory_pool(vm)); do { stream.next_out = njs_chb_reserve(&chain, chunk_size); @@ -461,7 +461,7 @@ njs_zlib_ext_inflate(njs_vm_t *vm, njs_value_t *args, njs_uint_t nargs, } } - NJS_CHB_MP_INIT(&chain, vm); + NJS_CHB_MP_INIT(&chain, njs_vm_memory_pool(vm)); while (rc != Z_STREAM_END) { stream.next_out = njs_chb_reserve(&chain, chunk_size); diff --git a/nginx/ngx_http_js_module.c b/nginx/ngx_http_js_module.c index 5a131bc9..ce4ce365 100644 --- a/nginx/ngx_http_js_module.c +++ b/nginx/ngx_http_js_module.c @@ -3121,7 +3121,7 @@ ngx_http_js_header_in_array(njs_vm_t *vm, ngx_http_request_t *r, (*hh)->value.len); } - NJS_CHB_MP_INIT(&chain, vm); + NJS_CHB_MP_INIT(&chain, njs_vm_memory_pool(vm)); for (i = 0; i < n; i++) { njs_chb_append(&chain, hh[i]->value.data, hh[i]->value.len); @@ -4166,7 +4166,7 @@ ngx_http_js_header_generic(njs_vm_t *vm, ngx_http_request_t *r, (*ph)->value.len); } - NJS_CHB_MP_INIT(&chain, vm); + NJS_CHB_MP_INIT(&chain, njs_vm_memory_pool(vm)); sep = flags & NJS_HEADER_SEMICOLON ? ';' : ','; diff --git a/nginx/ngx_js_fetch.c b/nginx/ngx_js_fetch.c index 9539a687..63e7a298 100644 --- a/nginx/ngx_js_fetch.c +++ b/nginx/ngx_js_fetch.c @@ -732,7 +732,7 @@ ngx_js_ext_fetch(njs_vm_t *vm, njs_value_t *args, njs_uint_t nargs, http->header_only = njs_strstr_eq(&request.method, &njs_str_value("HEAD")); - NJS_CHB_MP_INIT(&http->chain, vm); + NJS_CHB_MP_INIT(&http->chain, njs_vm_memory_pool(vm)); njs_chb_append(&http->chain, request.method.start, request.method.length); njs_chb_append_literal(&http->chain, " "); @@ -1035,7 +1035,7 @@ ngx_js_ext_response_constructor(njs_vm_t *vm, njs_value_t *args, } } - NJS_CHB_MP_INIT(&response->chain, vm); + NJS_CHB_MP_INIT(&response->chain, njs_vm_memory_pool(vm)); body = njs_arg(args, nargs, 1); @@ -2516,7 +2516,7 @@ ngx_js_http_process_headers(ngx_js_http_t *http) njs_chb_destroy(&http->chain); - NJS_CHB_MP_INIT(&http->response.chain, http->vm); + NJS_CHB_MP_INIT(&http->response.chain, njs_vm_memory_pool(http->vm)); http->process = ngx_js_http_process_body; @@ -3296,7 +3296,7 @@ ngx_headers_js_get(njs_vm_t *vm, njs_value_t *value, njs_str_t *name, return NJS_DECLINED; } - NJS_CHB_MP_INIT(&chain, vm); + NJS_CHB_MP_INIT(&chain, njs_vm_memory_pool(vm)); h = ph; diff --git a/src/njs_array.c b/src/njs_array.c index 15a6b6d3..913bc8cf 100644 --- a/src/njs_array.c +++ b/src/njs_array.c @@ -1686,7 +1686,7 @@ njs_array_prototype_join(njs_vm_t *vm, njs_value_t *args, njs_uint_t nargs, value = &entry; - NJS_CHB_MP_INIT(&chain, vm); + NJS_CHB_MP_INIT(&chain, njs_vm_memory_pool(vm)); for (i = 0; i < len; i++) { ret = njs_value_property_i64(vm, this, i, value); diff --git a/src/njs_chb.h b/src/njs_chb.h index 3dbe5ac9..4bd1cb3f 100644 --- a/src/njs_chb.h +++ b/src/njs_chb.h @@ -34,8 +34,8 @@ typedef struct { void njs_chb_init(njs_chb_t *chain, void *pool, njs_chb_alloc_t alloc, njs_chb_free_t free); -#define NJS_CHB_MP_INIT(chain, vm) \ - njs_chb_init(chain, njs_vm_memory_pool(vm), (njs_chb_alloc_t) njs_mp_alloc,\ +#define NJS_CHB_MP_INIT(chain, mp) \ + njs_chb_init(chain, mp, (njs_chb_alloc_t) njs_mp_alloc, \ (njs_chb_free_t) njs_mp_free) #define NJS_CHB_CTX_INIT(chain, ctx) \ njs_chb_init(chain, ctx, (njs_chb_alloc_t) js_malloc, \ diff --git a/src/njs_error.c b/src/njs_error.c index 6a14d767..b9da6597 100644 --- a/src/njs_error.c +++ b/src/njs_error.c @@ -1149,7 +1149,7 @@ njs_backtrace_to_string(njs_vm_t *vm, njs_arr_t *backtrace, njs_str_t *dst) return NJS_OK; } - NJS_CHB_MP_INIT(&chain, vm); + NJS_CHB_MP_INIT(&chain, njs_vm_memory_pool(vm)); njs_chb_append_str(&chain, dst); njs_chb_append(&chain, "\n", 1); diff --git a/src/njs_function.c b/src/njs_function.c index c677be57..326549f5 100644 --- a/src/njs_function.c +++ b/src/njs_function.c @@ -1032,7 +1032,7 @@ njs_function_constructor(njs_vm_t *vm, njs_value_t *args, njs_uint_t nargs, goto fail; } - NJS_CHB_MP_INIT(&chain, vm); + NJS_CHB_MP_INIT(&chain, njs_vm_memory_pool(vm)); if (async) { njs_chb_append_literal(&chain, "(async function("); diff --git a/src/njs_json.c b/src/njs_json.c index 85c5d0e8..8ba13314 100644 --- a/src/njs_json.c +++ b/src/njs_json.c @@ -1093,7 +1093,7 @@ njs_json_stringify_iterator(njs_json_stringify_t *stringify, goto memory_error; } - NJS_CHB_MP_INIT(&chain, stringify->vm); + NJS_CHB_MP_INIT(&chain, njs_vm_memory_pool(stringify->vm)); for ( ;; ) { if (state->index == 0) { @@ -1977,7 +1977,7 @@ njs_vm_value_dump(njs_vm_t *vm, njs_str_t *retval, njs_value_t *value, value = &exception; } - NJS_CHB_MP_INIT(&chain, vm); + NJS_CHB_MP_INIT(&chain, njs_vm_memory_pool(vm)); if (!njs_dump_is_recursive(value)) { ret = njs_dump_terminal(stringify, &chain, value, console); diff --git a/src/njs_regexp.c b/src/njs_regexp.c index 1c7cfe87..479f0b75 100644 --- a/src/njs_regexp.c +++ b/src/njs_regexp.c @@ -1373,7 +1373,7 @@ njs_regexp_prototype_symbol_replace(njs_vm_t *vm, njs_value_t *args, } } - NJS_CHB_MP_INIT(&chain, vm); + NJS_CHB_MP_INIT(&chain, njs_vm_memory_pool(vm)); results.separate = 0; results.pointer = 0; diff --git a/src/njs_string.c b/src/njs_string.c index 9405a8db..6a91cb4c 100644 --- a/src/njs_string.c +++ b/src/njs_string.c @@ -2955,7 +2955,7 @@ njs_string_get_substitution(njs_vm_t *vm, njs_value_t *matched, p = rep.start; end = rep.start + rep.length; - NJS_CHB_MP_INIT(&chain, vm); + NJS_CHB_MP_INIT(&chain, njs_vm_memory_pool(vm)); while (p < end) { r = njs_strlchr(p, end, '$'); @@ -3244,7 +3244,7 @@ njs_string_prototype_replace(njs_vm_t *vm, njs_value_t *args, njs_uint_t nargs, return NJS_OK; } - NJS_CHB_MP_INIT(&chain, vm); + NJS_CHB_MP_INIT(&chain, njs_vm_memory_pool(vm)); start = string.start; @@ -3899,7 +3899,7 @@ njs_string_decode_uri(njs_vm_t *vm, njs_value_t *args, njs_uint_t nargs, src = string.start; end = string.start + string.size; - NJS_CHB_MP_INIT(&chain, vm); + NJS_CHB_MP_INIT(&chain, njs_vm_memory_pool(vm)); njs_utf8_decode_init(&ctx); @@ -4173,7 +4173,7 @@ njs_string_atob(njs_vm_t *vm, njs_value_t *args, njs_uint_t nargs, len = njs_base64_decoded_length(len, pad); - NJS_CHB_MP_INIT(&chain, vm); + NJS_CHB_MP_INIT(&chain, njs_vm_memory_pool(vm)); dst = njs_chb_reserve(&chain, len * 2); if (njs_slow_path(dst == NULL)) { diff --git a/src/njs_typed_array.c b/src/njs_typed_array.c index 2a485a19..5930dcb0 100644 --- a/src/njs_typed_array.c +++ b/src/njs_typed_array.c @@ -2131,7 +2131,7 @@ njs_typed_array_prototype_join(njs_vm_t *vm, njs_value_t *args, return NJS_ERROR; } - NJS_CHB_MP_INIT(&chain, vm); + NJS_CHB_MP_INIT(&chain, njs_vm_memory_pool(vm)); njs_typed_array_to_chain(vm, &chain, array, separator); diff --git a/src/njs_vm.c b/src/njs_vm.c index dbeffa51..43b2ccc2 100644 --- a/src/njs_vm.c +++ b/src/njs_vm.c @@ -233,7 +233,7 @@ njs_vm_compile(njs_vm_t *vm, u_char **start, u_char *end) } if (njs_slow_path(vm->options.ast)) { - NJS_CHB_MP_INIT(&chain, vm); + NJS_CHB_MP_INIT(&chain, njs_vm_memory_pool(vm)); ret = njs_parser_serialize_ast(parser.node, &chain); if (njs_slow_path(ret == NJS_ERROR)) { return ret; diff --git a/src/test/njs_benchmark.c b/src/test/njs_benchmark.c index 98c618c8..66c4e84b 100644 --- a/src/test/njs_benchmark.c +++ b/src/test/njs_benchmark.c @@ -745,7 +745,7 @@ njs_benchmark_string(njs_vm_t *vm, njs_value_t *args, njs_uint_t nargs, } else if (memcmp(mode.start, "chb", 3) == 0) { - NJS_CHB_MP_INIT(&chain, vm); + NJS_CHB_MP_INIT(&chain, njs_vm_memory_pool(vm)); njs_chb_append_literal(&chain, "abc"); njs_chb_append(&chain, s.start, s.length); diff --git a/src/test/njs_externals_test.c b/src/test/njs_externals_test.c index 7d2e82af..33f0020a 100644 --- a/src/test/njs_externals_test.c +++ b/src/test/njs_externals_test.c @@ -323,7 +323,7 @@ njs_unit_test_r_header(njs_vm_t *vm, njs_object_prop_t *prop, ret = njs_vm_prop_name(vm, prop, &h); if (ret == NJS_OK) { - NJS_CHB_MP_INIT(&chain, vm); + NJS_CHB_MP_INIT(&chain, njs_vm_memory_pool(vm)); njs_chb_append(&chain, h.start, h.length); njs_chb_append(&chain, (u_char *) "|АБВ", njs_length("|АБВ")); diff --git a/src/test/njs_unit_test.c b/src/test/njs_unit_test.c index 980fd7fa..9c935e5b 100644 --- a/src/test/njs_unit_test.c +++ b/src/test/njs_unit_test.c @@ -22399,7 +22399,7 @@ njs_chb_test(njs_vm_t *vm, njs_opts_t *opts, njs_stat_t *stat) static const njs_str_t expected = njs_str("arg: \"XYZ\" -5"); - NJS_CHB_MP_INIT(&chain, vm); + NJS_CHB_MP_INIT(&chain, njs_vm_memory_pool(vm)); p = njs_chb_reserve(&chain, 513); if (p == NULL) { _______________________________________________ nginx-devel mailing list nginx-devel@nginx.org https://mailman.nginx.org/mailman/listinfo/nginx-devel