From 83b24c1d0ca752f82e24fc261eeb6a71059cdc44 Mon Sep 17 00:00:00 2001
From: Alexander Borisov <alexander.borisov@nginx.com>
Date: Fri, 19 Jun 2020 19:48:13 +0300
Subject: [PATCH] Parser: fixed line counting in template literals.

This closes #321 issue on GitHub.
---
 src/njs_parser.c         | 24 +++++++++++++++---------
 src/test/njs_unit_test.c |  7 +++++++
 2 files changed, 22 insertions(+), 9 deletions(-)

diff --git a/src/njs_parser.c b/src/njs_parser.c
index 9b4632d8..be232f18 100644
--- a/src/njs_parser.c
+++ b/src/njs_parser.c
@@ -1141,7 +1141,7 @@ njs_parser_primary_expression_test(njs_parser_t *parser,
         parser->node = node;
 
         njs_parser_next(parser, njs_parser_template_literal);
-        break;
+        return NJS_OK;
 
     /* CoverParenthesizedExpressionAndArrowParameterList */
     case NJS_TOKEN_OPEN_PARENTHESIS:
@@ -1322,6 +1322,9 @@ njs_parser_template_literal(njs_parser_t *parser, njs_lexer_token_t *token,
 
     parser->target = temp;
 
+    token->text.start++;
+    token->text.length = 0;
+
     njs_parser_next(parser, njs_parser_template_literal_string);
 
     return NJS_OK;
@@ -2199,8 +2202,6 @@ njs_parser_property(njs_parser_t *parser, njs_lexer_token_t *token,
 
         parser->node = node;
 
-        njs_lexer_consume_token(parser->lexer, 1);
-
         njs_parser_next(parser, njs_parser_template_literal);
 
         break;
@@ -7820,28 +7821,33 @@ njs_parser_template_string(njs_parser_t *parser, njs_lexer_token_t *token)
 
         c = *p++;
 
-        if (c == '\\') {
+        switch (c) {
+        case '\\':
             if (p == lexer->end) {
-                break;
+                return NJS_ERROR;
             }
 
             p++;
             escape = 1;
 
             continue;
-        }
 
-        if (c == '`') {
+        case '`':
             text->length = p - text->start - 1;
             goto done;
-        }
 
-        if (c == '$') {
+        case '$':
             if (p < lexer->end && *p == '{') {
                 p++;
                 text->length = p - text->start - 2;
                 goto done;
             }
+
+            break;
+
+        case '\n':
+            parser->lexer->line++;
+            break;
         }
     }
 
diff --git a/src/test/njs_unit_test.c b/src/test/njs_unit_test.c
index b7052caf..c8b8b09e 100644
--- a/src/test/njs_unit_test.c
+++ b/src/test/njs_unit_test.c
@@ -6330,6 +6330,9 @@ static njs_unit_test_t  njs_test[] =
                  "foo`That ${person} is a ${age}`;"),
       njs_str("That  is a Mike21") },
 
+    { njs_str("`\n`.length"),
+      njs_str("1") },
+
     /* Strings. */
 
     { njs_str("var a = '0123456789' + '012345';"
@@ -17889,6 +17892,10 @@ static njs_unit_test_t  njs_shell_test[] =
       njs_str("ReferenceError: \"a\" is not defined in 2\n"
               "    at main (:2)\n") },
 
+    { njs_str("\n`\n${Object}\n${a}`" ENTER),
+      njs_str("ReferenceError: \"a\" is not defined in 4\n"
+              "    at main (:4)\n") },
+
     { njs_str("function log(v) {}\nlog({}\n.a\n.a)" ENTER),
       njs_str("TypeError: cannot get property \"a\" of undefined\n"
               "    at main (:4)\n") },
-- 
2.47.3