Modules: removed extra copying in shared dictionary. - Githubissues

nginx / njs

A subset of JavaScript language to use in nginx

http://nginx.org/en/docs/njs/

BSD 2-Clause "Simplified" License

1.14k stars 148 forks source link

Modules: removed extra copying in shared dictionary. #736

Closed xeioex closed 3 months ago

xeioex commented 3 months ago

After 5730d5f pool copy is not needed because njs_vm_value_string_create() always create a copy.

Checklist

Before creating a PR, run through this checklist and mark each as complete:

[x] I have read the CONTRIBUTING document
[ ] If applicable, I have added tests that prove my fix is effective or that my feature works
[x] If applicable, I have checked that any relevant tests pass after adding my changes