New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

typeof ([].length = '') #27

Closed

drsm opened this issue Jul 1, 2018 · 0 comments

Assignees

Labels

bug

Contributor

drsm commented Jul 1, 2018 •

edited

Loading

njs/njs/njs_array.c

Line 412 in 7cacebe

njs_value_number_set(retval, array->length);

diff -r e210675daceb njs/njs_array.c
--- a/njs/njs_array.c   Sun Jul 01 10:01:53 2018 +0300
+++ b/njs/njs_array.c   Sun Jul 01 13:11:29 2018 +0300
@@ -407,6 +407,9 @@
         }
 
         array->length = length;
+        retval = setval;
+
+        return NJS_OK;
     }
 
     njs_value_number_set(retval, array->length);

xeioex self-assigned this

xeioex added the bug label

drsm mentioned this issue

the Array.length property setter should perform a type conversion #26

Closed

nginx-hg-mirror closed this as completed in

0b4030f

VadimZhestikov mentioned this issue

QuickJS modules. #779

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment