diff options
| author | Dmitry Volyntsev <xeioex@nginx.com> | 2019-08-01 18:49:28 +0300 |
|---|---|---|
| committer | Dmitry Volyntsev <xeioex@nginx.com> | 2019-08-01 18:49:28 +0300 |
| commit | 8e8938f65a901ee6ee25c329c61b20eb25eabd59 (patch) | |
| tree | 61e2007d3d3cd8e46d0448e57900f2984f413f09 /nginx/ngx_http_js_module.c | |
| parent | 5aff5bc3a3b9f7dafba912965ad62f1323b3b1ea (diff) | |
| download | njs-8e8938f65a901ee6ee25c329c61b20eb25eabd59.tar.gz njs-8e8938f65a901ee6ee25c329c61b20eb25eabd59.zip | |
Avoiding excessive (njs_value_t *) casts.
Previously njs_arg() returned expression of type (const njs_value_t *).
This caused a lot of casts when calling functions and forced to declared
const njs_value_t * variables.
NO functional changes.
Diffstat (limited to 'nginx/ngx_http_js_module.c')
| -rw-r--r-- | nginx/ngx_http_js_module.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/nginx/ngx_http_js_module.c b/nginx/ngx_http_js_module.c index 813faf99..9ce334c4 100644 --- a/nginx/ngx_http_js_module.c +++ b/nginx/ngx_http_js_module.c @@ -139,7 +139,7 @@ static void ngx_http_js_clear_timer(njs_external_ptr_t external, static void ngx_http_js_timer_handler(ngx_event_t *ev); static void ngx_http_js_handle_event(ngx_http_request_t *r, njs_vm_event_t vm_event, njs_value_t *args, njs_uint_t nargs); -static njs_int_t ngx_http_js_string(njs_vm_t *vm, const njs_value_t *value, +static njs_int_t ngx_http_js_string(njs_vm_t *vm, njs_value_t *value, njs_str_t *str); static char *ngx_http_js_include(ngx_conf_t *cf, ngx_command_t *cmd, @@ -1246,9 +1246,9 @@ ngx_http_js_ext_return(njs_vm_t *vm, njs_value_t *args, njs_uint_t nargs, { njs_str_t text; ngx_int_t status; + njs_value_t *value; ngx_http_js_ctx_t *ctx; ngx_http_request_t *r; - const njs_value_t *value; ngx_http_complex_value_t cv; r = njs_vm_external(vm, njs_arg(args, nargs, 0)); @@ -1729,10 +1729,9 @@ ngx_http_js_ext_subrequest(njs_vm_t *vm, njs_value_t *args, njs_uint_t nargs, ngx_int_t rc; njs_str_t uri_arg, args_arg, method_name, body_arg; ngx_uint_t method, methods_max, has_body; - njs_value_t *value; + njs_value_t *value, *arg, *options; njs_function_t *callback; ngx_http_js_ctx_t *ctx; - const njs_value_t *arg, *options; ngx_http_request_t *r, *sr; ngx_http_request_body_t *rb; @@ -2187,7 +2186,7 @@ ngx_http_js_handle_event(ngx_http_request_t *r, njs_vm_event_t vm_event, static njs_int_t -ngx_http_js_string(njs_vm_t *vm, const njs_value_t *value, njs_str_t *str) +ngx_http_js_string(njs_vm_t *vm, njs_value_t *value, njs_str_t *str) { if (!njs_value_is_null_or_undefined(value)) { if (njs_vm_value_to_string(vm, str, value) == NJS_ERROR) { |