Welcome! Log In Create A New Profile

Advanced

[njs] Removed obsolete commentaries for njs_value_property() and friends.

Dmitry Volyntsev
August 05, 2019 12:04PM
details: https://hg.nginx.org/njs/rev/439e2f67acf8
branches:
changeset: 1107:439e2f67acf8
user: Dmitry Volyntsev <xeioex@nginx.com>
date: Mon Aug 05 19:02:17 2019 +0300
description:
Removed obsolete commentaries for njs_value_property() and friends.

After NJS_TRAP and NJS_APPLIED was eliminanted they are mostly useless.

diffstat:

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

diffs (30 lines):

diff -r 1acb3acb1059 -r 439e2f67acf8 src/njs_value.c
--- a/src/njs_value.c Sun Aug 04 11:17:48 2019 -0400
+++ b/src/njs_value.c Mon Aug 05 19:02:17 2019 +0300
@@ -911,15 +911,6 @@ njs_external_property_delete(njs_vm_t *v
}


-/*
- * ES5.1, 8.12.3: [[Get]].
- * NJS_OK property has been found in object,
- * retval will contain the property's value
- *
- * NJS_DECLINED property was not found in object
- * NJS_ERROR exception has been thrown.
- * retval will contain undefined
- */
njs_int_t
njs_value_property(njs_vm_t *vm, njs_value_t *value, njs_value_t *key,
njs_value_t *retval)
@@ -1004,10 +995,6 @@ njs_value_property(njs_vm_t *vm, njs_val
}


-/*
- * NJS_OK property has been set successfully
- * NJS_ERROR exception has been thrown.
- */
njs_int_t
njs_value_property_set(njs_vm_t *vm, njs_value_t *value, njs_value_t *key,
njs_value_t *setval)
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

[njs] Removed obsolete commentaries for njs_value_property() and friends.

Dmitry Volyntsev 206 August 05, 2019 12:04PM



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

Online Users

Guests: 305
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