Welcome! Log In Create A New Profile

Advanced

[njs] Removed dead code in njs_value_property_set() after df385232d2af.

Dmitry Volyntsev
August 07, 2019 08:38AM
details: https://hg.nginx.org/njs/rev/90b8adf370d4
branches:
changeset: 1112:90b8adf370d4
user: Dmitry Volyntsev <xeioex@nginx.com>
date: Wed Aug 07 15:17:57 2019 +0300
description:
Removed dead code in njs_value_property_set() after df385232d2af.

Found by Coverity Scan (CID 1452240).

diffstat:

src/njs_value.c | 9 +--------
1 files changed, 1 insertions(+), 8 deletions(-)

diffs (33 lines):

diff -r 30b1199acaaf -r 90b8adf370d4 src/njs_value.c
--- a/src/njs_value.c Tue Aug 06 19:45:23 2019 +0300
+++ b/src/njs_value.c Wed Aug 07 15:17:57 2019 +0300
@@ -1056,7 +1056,7 @@ njs_value_property_set(njs_vm_t *vm, njs
njs_value_t *setval)
{
njs_int_t ret;
- njs_object_prop_t *prop, *shared;
+ njs_object_prop_t *prop;
njs_property_query_t pq;

if (njs_is_primitive(value)) {
@@ -1065,8 +1065,6 @@ njs_value_property_set(njs_vm_t *vm, njs
return NJS_ERROR;
}

- shared = NULL;
-
njs_property_query_init(&pq, NJS_PROPERTY_QUERY_SET, 0);

ret = njs_property_query(vm, &pq, value, key);
@@ -1158,11 +1156,6 @@ njs_value_property_set(njs_vm_t *vm, njs
return NJS_ERROR;
}

- if (njs_slow_path(shared != NULL)) {
- prop->enumerable = shared->enumerable;
- prop->configurable = shared->configurable;
- }
-
pq.lhq.replace = 0;
pq.lhq.value = prop;
pq.lhq.pool = vm->mem_pool;
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

[njs] Removed dead code in njs_value_property_set() after df385232d2af.

Dmitry Volyntsev 256 August 07, 2019 08:38AM



Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 168
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready