277hz

@drsm Working on it and still confident that the idea of shifting the njs_vmcode_prop_set_t before adding them to the array is a good way to go. To me it looks...

@lexborisov Not yet, still have some work to do. After fixing what @drsm described, I think there are some situations left that do not run smoothly. Edit: As we can...

@lexborisov Final patch https://gist.github.com/277hz/b68936f0b845d96a3dda2bfaf025cb69 Adds support for spreading of arrays, objects and function arguments, solving #78 #79 #92. Passing all spread tests in test262, except those using a generator function:...

@lexborisov Thank you for taking the time. I was busy doing a lot of fixing and refactoring, which should help significantly with merging everything later. What I have been achieving...

@xeioex Don't get me wrong :smile:. I really like how you designed njs, and of course nginx, too... > * object hash is non-compliant and slow (see discussion here #189)...

@xeioex I am trying to obey. At first I thought it is going to be smaller changes only, the biggest part acutally consists of the routine for shifting array indexes....

Patch updated: [njs_spread.diff](https://gist.github.com/277hz/b68936f0b845d96a3dda2bfaf025cb69) Please feel free to play around with: [compare_njs_node.sh](https://gist.github.com/277hz/987a257066b23caeed46ee065f9a28e8) Attached the latest [feature.diff](https://github.com/nginx/njs/files/6716159/feature.diff.txt): ``` === Summary === - Ran 34658 tests - - Passed 13514 tests (39.0%) -...

While providing the latest update for [njs_spread.diff](https://gist.github.com/277hz/b68936f0b845d96a3dda2bfaf025cb69) and hoping to come nearer to an end :smile:, I wanted to shed some insight on what the patch modifies technically. It would...

While I want to endorse @drsm that this is not how hashes are ment to be implemented, I want to point out that this is a significant addition to javascript...