diff options
| author | Dmitry Volyntsev <xeioex@nginx.com> | 2023-11-21 08:57:09 -0800 |
|---|---|---|
| committer | Dmitry Volyntsev <xeioex@nginx.com> | 2023-11-21 08:57:09 -0800 |
| commit | 735ec1d89d25d014870452ba1600a8cd9dec5641 (patch) | |
| tree | 300586842972d0d4f0ed2fc96319abf78d8516bf /nginx/ngx_js.c | |
| parent | 2743bab180ab067ea6d146c92a30a7f88bff7b2a (diff) | |
| download | njs-735ec1d89d25d014870452ba1600a8cd9dec5641.tar.gz njs-735ec1d89d25d014870452ba1600a8cd9dec5641.zip | |
Renaming ngx_js_retval() to ngx_js_exception().
After previous commit ngx_js_retval() is only used for getting the
exception value and should be renamed to better reflect the purpose.
Diffstat (limited to 'nginx/ngx_js.c')
| -rw-r--r-- | nginx/ngx_js.c | 14 |
1 files changed, 4 insertions, 10 deletions
diff --git a/nginx/ngx_js.c b/nginx/ngx_js.c index 2a9acc6d..4c0f8c29 100644 --- a/nginx/ngx_js.c +++ b/nginx/ngx_js.c @@ -359,7 +359,7 @@ ngx_js_invoke(njs_vm_t *vm, ngx_str_t *fname, ngx_log_t *log, ret = njs_vm_invoke(vm, func, njs_value_arg(args), nargs, njs_value_arg(retval)); if (ret == NJS_ERROR) { - ngx_js_retval(vm, NULL, &exception); + ngx_js_exception(vm, &exception); ngx_log_error(NGX_LOG_ERR, log, 0, "js exception: %V", &exception); @@ -369,7 +369,7 @@ ngx_js_invoke(njs_vm_t *vm, ngx_str_t *fname, ngx_log_t *log, ret = njs_vm_run(vm); if (ret == NJS_ERROR) { - ngx_js_retval(vm, NULL, &exception); + ngx_js_exception(vm, &exception); ngx_log_error(NGX_LOG_ERR, log, 0, "js exception: %V", &exception); @@ -382,18 +382,12 @@ ngx_js_invoke(njs_vm_t *vm, ngx_str_t *fname, ngx_log_t *log, ngx_int_t -ngx_js_retval(njs_vm_t *vm, njs_opaque_value_t *retval, ngx_str_t *s) +ngx_js_exception(njs_vm_t *vm, ngx_str_t *s) { njs_int_t ret; njs_str_t str; - if (retval != NULL && njs_value_is_valid(njs_value_arg(retval))) { - ret = njs_vm_value_string(vm, &str, njs_value_arg(retval)); - - } else { - ret = njs_vm_exception_string(vm, &str); - } - + ret = njs_vm_exception_string(vm, &str); if (ret != NJS_OK) { return NGX_ERROR; } |