Welcome! Log In Create A New Profile

Advanced

[njs] Reserverd words can be used as property names.

June 01, 2016 08:52AM
details: http://hg.nginx.org/njs/rev/6fa680a54811
branches:
changeset: 105:6fa680a54811
user: Igor Sysoev <igor@sysoev.ru>
date: Wed Apr 20 18:30:31 2016 +0300
description:
Reserverd words can be used as property names.

diffstat:

njs/njs_lexer.c | 4 ++++
njs/njs_parser.c | 17 ++++++++++++++++-
njs/njs_parser.h | 2 ++
njs/njs_parser_expression.c | 22 +++++++++++++---------
njs/test/njs_unit_test.c | 6 ++++++
5 files changed, 41 insertions(+), 10 deletions(-)

diffs (129 lines):

diff -r 3f2480c38f8e -r 6fa680a54811 njs/njs_lexer.c
--- a/njs/njs_lexer.c Tue Apr 19 16:08:32 2016 +0300
+++ b/njs/njs_lexer.c Wed Apr 20 18:30:31 2016 +0300
@@ -465,6 +465,10 @@ njs_lexer_word(njs_lexer_t *lexer, u_cha
lexer->start = p;
lexer->text.len = p - lexer->text.data;

+ if (lexer->property) {
+ return NJS_TOKEN_NAME;
+ }
+
return njs_lexer_keyword(lexer);
}

diff -r 3f2480c38f8e -r 6fa680a54811 njs/njs_parser.c
--- a/njs/njs_parser.c Tue Apr 19 16:08:32 2016 +0300
+++ b/njs/njs_parser.c Wed Apr 20 18:30:31 2016 +0300
@@ -1381,6 +1381,21 @@ njs_parser_grouping_expression(njs_vm_t


njs_token_t
+njs_parser_property_token(njs_parser_t *parser)
+{
+ njs_token_t token;
+
+ parser->lexer->property = 1;
+
+ token = njs_parser_token(parser);
+
+ parser->lexer->property = 0;
+
+ return token;
+}
+
+
+njs_token_t
njs_parser_token(njs_parser_t *parser)
{
njs_token_t token;
@@ -1711,7 +1726,7 @@ njs_parser_object(njs_vm_t *vm, njs_pars
left = NULL;

for ( ;; ) {
- token = njs_parser_token(parser);
+ token = njs_parser_property_token(parser);

switch (token) {

diff -r 3f2480c38f8e -r 6fa680a54811 njs/njs_parser.h
--- a/njs/njs_parser.h Tue Apr 19 16:08:32 2016 +0300
+++ b/njs/njs_parser.h Wed Apr 20 18:30:31 2016 +0300
@@ -183,6 +183,7 @@ typedef enum {
typedef struct {
njs_token_t token:8;
njs_token_t prev_token:8;
+ uint8_t property; /* 1 bit */
uint32_t key_hash;

nxt_str_t text;
@@ -328,6 +329,7 @@ njs_token_t njs_parser_terminal(njs_vm_t
njs_token_t token);
njs_token_t njs_parser_property_name(njs_vm_t *vm, njs_parser_t *parser,
njs_token_t token);
+njs_token_t njs_parser_property_token(njs_parser_t *parser);
njs_token_t njs_parser_token(njs_parser_t *parser);
nxt_int_t njs_parser_string_create(njs_vm_t *vm, njs_value_t *value);
njs_index_t njs_parser_index(njs_parser_t *parser, uint32_t scope);
diff -r 3f2480c38f8e -r 6fa680a54811 njs/njs_parser_expression.c
--- a/njs/njs_parser_expression.c Tue Apr 19 16:08:32 2016 +0300
+++ b/njs/njs_parser_expression.c Wed Apr 20 18:30:31 2016 +0300
@@ -999,7 +999,6 @@ static njs_token_t
njs_parser_property_expression(njs_vm_t *vm, njs_parser_t *parser,
njs_token_t token)
{
- njs_token_t next;
njs_parser_node_t *node;

for ( ;; ) {
@@ -1019,21 +1018,26 @@ njs_parser_property_expression(njs_vm_t
node->u.operation = njs_vmcode_property_get;
node->left = parser->node;

- next = njs_parser_token(parser);
- if (nxt_slow_path(next <= NJS_TOKEN_ILLEGAL)) {
- return next;
- }
-
if (token == NJS_TOKEN_DOT) {

- if (next != NJS_TOKEN_NAME) {
+ token = njs_parser_property_token(parser);
+ if (nxt_slow_path(token <= NJS_TOKEN_ILLEGAL)) {
+ return token;
+ }
+
+ if (token != NJS_TOKEN_NAME) {
return NJS_TOKEN_ILLEGAL;
}

- token = njs_parser_property_name(vm, parser, next);
+ token = njs_parser_property_name(vm, parser, token);

} else {
- token = njs_parser_property_brackets(vm, parser, next);
+ token = njs_parser_token(parser);
+ if (nxt_slow_path(token <= NJS_TOKEN_ILLEGAL)) {
+ return token;
+ }
+
+ token = njs_parser_property_brackets(vm, parser, token);
}

if (nxt_slow_path(token <= NJS_TOKEN_ILLEGAL)) {
diff -r 3f2480c38f8e -r 6fa680a54811 njs/test/njs_unit_test.c
--- a/njs/test/njs_unit_test.c Tue Apr 19 16:08:32 2016 +0300
+++ b/njs/test/njs_unit_test.c Wed Apr 20 18:30:31 2016 +0300
@@ -1938,6 +1938,12 @@ static njs_unit_test_t njs_test[] =
"var o = { a: 5, '[object Object]': 7 }; o[n]"),
nxt_string("7") },

+ { nxt_string("var o = {}; o.new = 'OK'; o.new"),
+ nxt_string("OK") },
+
+ { nxt_string("var o = { new: 'OK'}; o.new"),
+ nxt_string("OK") },
+
/* Arrays */

/* Empty array to primitive. */

_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

[njs] Reserverd words can be used as property names.

Igor Sysoev 647 June 01, 2016 08:52AM



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

Online Users

Guests: 164
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 500 on July 15, 2024
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready