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

Dmitry Volyntsev xeioex at nginx.com
Mon Aug 5 16:02:46 UTC 2019


details:   https://hg.nginx.org/njs/rev/439e2f67acf8
branches:  
changeset: 1107:439e2f67acf8
user:      Dmitry Volyntsev <xeioex at 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)


More information about the nginx-devel mailing list