Var: simplified length calculation for $status variable.

andrey-zelenkov · andrey-zelenkov · commit 78c133d0ca3c · 2023-11-08T17:38:07.000Z
diff --git a/src/nxt_http_variables.c b/src/nxt_http_variables.c
@@ -455,7 +455,6 @@ nxt_http_var_body_bytes_sent(nxt_task_t *task, nxt_str_t *str, void *ctx,
 static nxt_int_t
 nxt_http_var_status(nxt_task_t *task, nxt_str_t *str, void *ctx, void *data)
 {
-    u_char              *p;
     nxt_http_request_t  *r;
 
     r = ctx;
@@ -465,9 +464,9 @@ nxt_http_var_status(nxt_task_t *task, nxt_str_t *str, void *ctx, void *data)
         return NXT_ERROR;
     }
 
-    p = nxt_sprintf(str->start, str->start + 3, "%03d", r->status);
+    (void) nxt_sprintf(str->start, str->start + 3, "%03d", r->status);
 
-    str->length = p - str->start;
+    str->length = 3;
 
     return NXT_OK;
 }

Original file line number	Diff line number	Diff line change
`@@ -455,7 +455,6 @@ nxt_http_var_body_bytes_sent(nxt_task_t task, nxt_str_t str, void *ctx,`
`455`	`455`	`static nxt_int_t`
`456`	`456`	`nxt_http_var_status(nxt_task_t task, nxt_str_t str, void ctx, void data)`
`457`	`457`	`{`
`458`		`- u_char *p;`
`459`	`458`	`nxt_http_request_t *r;`
`460`	`459`
`461`	`460`	`r = ctx;`
`@@ -465,9 +464,9 @@ nxt_http_var_status(nxt_task_t task, nxt_str_t str, void ctx, void data)`
`465`	`464`	`return NXT_ERROR;`
`466`	`465`	`}`
`467`	`466`
`468`		`- p = nxt_sprintf(str->start, str->start + 3, "%03d", r->status);`
	`467`	`+ (void) nxt_sprintf(str->start, str->start + 3, "%03d", r->status);`
`469`	`468`
`470`		`- str->length = p - str->start;`
	`469`	`+ str->length = 3;`
`471`	`470`
`472`	`471`	`return NXT_OK;`
`473`	`472`	`}`