2012-01-24 16:36:55 +00:00
|
|
|
// Copyright 2012 the V8 project authors. All rights reserved.
|
2009-11-05 10:11:38 +00:00
|
|
|
// Redistribution and use in source and binary forms, with or without
|
|
|
|
// modification, are permitted provided that the following conditions are
|
|
|
|
// met:
|
|
|
|
//
|
|
|
|
// * Redistributions of source code must retain the above copyright
|
|
|
|
// notice, this list of conditions and the following disclaimer.
|
|
|
|
// * Redistributions in binary form must reproduce the above
|
|
|
|
// copyright notice, this list of conditions and the following
|
|
|
|
// disclaimer in the documentation and/or other materials provided
|
|
|
|
// with the distribution.
|
|
|
|
// * Neither the name of Google Inc. nor the names of its
|
|
|
|
// contributors may be used to endorse or promote products derived
|
|
|
|
// from this software without specific prior written permission.
|
|
|
|
//
|
|
|
|
// THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
|
|
|
|
// "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
|
|
|
|
// LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
|
|
|
|
// A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
|
|
|
|
// OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
|
|
|
|
// SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
|
|
|
|
// LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
|
|
|
|
// DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
|
|
|
|
// THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
|
|
|
|
// (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
|
|
|
|
// OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
|
|
|
|
|
2010-11-02 07:21:37 +00:00
|
|
|
#include <stdio.h>
|
2014-06-20 08:40:11 +00:00
|
|
|
#include <stdlib.h>
|
2010-11-11 11:45:25 +00:00
|
|
|
#include <string.h>
|
2009-11-05 10:11:38 +00:00
|
|
|
|
2014-06-03 08:12:43 +00:00
|
|
|
#include "src/v8.h"
|
2009-11-05 10:11:38 +00:00
|
|
|
|
2014-06-24 14:03:24 +00:00
|
|
|
#include "src/ast-value-factory.h"
|
2014-06-03 08:12:43 +00:00
|
|
|
#include "src/compiler.h"
|
|
|
|
#include "src/execution.h"
|
|
|
|
#include "src/isolate.h"
|
|
|
|
#include "src/objects.h"
|
|
|
|
#include "src/parser.h"
|
|
|
|
#include "src/preparser.h"
|
2014-06-26 11:59:42 +00:00
|
|
|
#include "src/rewriter.h"
|
2014-06-03 08:12:43 +00:00
|
|
|
#include "src/scanner-character-streams.h"
|
|
|
|
#include "src/token.h"
|
|
|
|
#include "src/utils.h"
|
|
|
|
#include "test/cctest/cctest.h"
|
2009-11-05 10:11:38 +00:00
|
|
|
|
2011-08-09 13:32:53 +00:00
|
|
|
TEST(ScanKeywords) {
|
2009-11-05 10:11:38 +00:00
|
|
|
struct KeywordToken {
|
|
|
|
const char* keyword;
|
|
|
|
i::Token::Value token;
|
|
|
|
};
|
|
|
|
|
|
|
|
static const KeywordToken keywords[] = {
|
|
|
|
#define KEYWORD(t, s, d) { s, i::Token::t },
|
2011-08-09 13:32:53 +00:00
|
|
|
TOKEN_LIST(IGNORE_TOKEN, KEYWORD)
|
2009-11-05 10:11:38 +00:00
|
|
|
#undef KEYWORD
|
|
|
|
{ NULL, i::Token::IDENTIFIER }
|
|
|
|
};
|
|
|
|
|
|
|
|
KeywordToken key_token;
|
2011-08-09 13:32:53 +00:00
|
|
|
i::UnicodeCache unicode_cache;
|
|
|
|
i::byte buffer[32];
|
2009-11-05 10:11:38 +00:00
|
|
|
for (int i = 0; (key_token = keywords[i]).keyword != NULL; i++) {
|
2011-08-09 13:32:53 +00:00
|
|
|
const i::byte* keyword =
|
|
|
|
reinterpret_cast<const i::byte*>(key_token.keyword);
|
|
|
|
int length = i::StrLength(key_token.keyword);
|
|
|
|
CHECK(static_cast<int>(sizeof(buffer)) >= length);
|
|
|
|
{
|
2012-03-12 12:35:28 +00:00
|
|
|
i::Utf8ToUtf16CharacterStream stream(keyword, length);
|
2011-11-01 07:47:15 +00:00
|
|
|
i::Scanner scanner(&unicode_cache);
|
2012-02-08 10:53:58 +00:00
|
|
|
// The scanner should parse Harmony keywords for this test.
|
2011-10-12 12:23:06 +00:00
|
|
|
scanner.SetHarmonyScoping(true);
|
2012-02-08 10:53:58 +00:00
|
|
|
scanner.SetHarmonyModules(true);
|
2011-08-17 11:45:04 +00:00
|
|
|
scanner.Initialize(&stream);
|
2011-08-09 13:32:53 +00:00
|
|
|
CHECK_EQ(key_token.token, scanner.Next());
|
|
|
|
CHECK_EQ(i::Token::EOS, scanner.Next());
|
2009-11-05 10:11:38 +00:00
|
|
|
}
|
2011-08-09 13:32:53 +00:00
|
|
|
// Removing characters will make keyword matching fail.
|
|
|
|
{
|
2012-03-12 12:35:28 +00:00
|
|
|
i::Utf8ToUtf16CharacterStream stream(keyword, length - 1);
|
2011-11-01 07:47:15 +00:00
|
|
|
i::Scanner scanner(&unicode_cache);
|
2011-08-09 13:32:53 +00:00
|
|
|
scanner.Initialize(&stream);
|
|
|
|
CHECK_EQ(i::Token::IDENTIFIER, scanner.Next());
|
|
|
|
CHECK_EQ(i::Token::EOS, scanner.Next());
|
|
|
|
}
|
|
|
|
// Adding characters will make keyword matching fail.
|
|
|
|
static const char chars_to_append[] = { 'z', '0', '_' };
|
|
|
|
for (int j = 0; j < static_cast<int>(ARRAY_SIZE(chars_to_append)); ++j) {
|
2014-05-27 07:57:22 +00:00
|
|
|
i::MemMove(buffer, keyword, length);
|
2011-08-09 13:32:53 +00:00
|
|
|
buffer[length] = chars_to_append[j];
|
2012-03-12 12:35:28 +00:00
|
|
|
i::Utf8ToUtf16CharacterStream stream(buffer, length + 1);
|
2011-11-01 07:47:15 +00:00
|
|
|
i::Scanner scanner(&unicode_cache);
|
2011-08-09 13:32:53 +00:00
|
|
|
scanner.Initialize(&stream);
|
|
|
|
CHECK_EQ(i::Token::IDENTIFIER, scanner.Next());
|
|
|
|
CHECK_EQ(i::Token::EOS, scanner.Next());
|
|
|
|
}
|
|
|
|
// Replacing characters will make keyword matching fail.
|
|
|
|
{
|
2014-05-27 07:57:22 +00:00
|
|
|
i::MemMove(buffer, keyword, length);
|
2011-08-09 13:32:53 +00:00
|
|
|
buffer[length - 1] = '_';
|
2012-03-12 12:35:28 +00:00
|
|
|
i::Utf8ToUtf16CharacterStream stream(buffer, length);
|
2011-11-01 07:47:15 +00:00
|
|
|
i::Scanner scanner(&unicode_cache);
|
2011-08-09 13:32:53 +00:00
|
|
|
scanner.Initialize(&stream);
|
|
|
|
CHECK_EQ(i::Token::IDENTIFIER, scanner.Next());
|
|
|
|
CHECK_EQ(i::Token::EOS, scanner.Next());
|
2009-11-05 10:11:38 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2010-09-02 07:36:30 +00:00
|
|
|
|
|
|
|
TEST(ScanHTMLEndComments) {
|
2011-08-04 15:18:18 +00:00
|
|
|
v8::V8::Initialize();
|
2013-10-01 09:56:04 +00:00
|
|
|
v8::Isolate* isolate = CcTest::isolate();
|
2014-01-13 11:15:06 +00:00
|
|
|
v8::HandleScope handles(isolate);
|
2011-08-04 15:18:18 +00:00
|
|
|
|
2010-09-02 07:36:30 +00:00
|
|
|
// Regression test. See:
|
|
|
|
// http://code.google.com/p/chromium/issues/detail?id=53548
|
|
|
|
// Tests that --> is correctly interpreted as comment-to-end-of-line if there
|
2011-06-21 13:34:16 +00:00
|
|
|
// is only whitespace before it on the line (with comments considered as
|
|
|
|
// whitespace, even a multiline-comment containing a newline).
|
|
|
|
// This was not the case if it occurred before the first real token
|
2010-09-02 07:36:30 +00:00
|
|
|
// in the input.
|
|
|
|
const char* tests[] = {
|
|
|
|
// Before first real token.
|
|
|
|
"--> is eol-comment\nvar y = 37;\n",
|
|
|
|
"\n --> is eol-comment\nvar y = 37;\n",
|
|
|
|
"/* precomment */ --> is eol-comment\nvar y = 37;\n",
|
|
|
|
"\n/* precomment */ --> is eol-comment\nvar y = 37;\n",
|
|
|
|
// After first real token.
|
|
|
|
"var x = 42;\n--> is eol-comment\nvar y = 37;\n",
|
|
|
|
"var x = 42;\n/* precomment */ --> is eol-comment\nvar y = 37;\n",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
2011-06-21 13:34:16 +00:00
|
|
|
const char* fail_tests[] = {
|
|
|
|
"x --> is eol-comment\nvar y = 37;\n",
|
|
|
|
"\"\\n\" --> is eol-comment\nvar y = 37;\n",
|
|
|
|
"x/* precomment */ --> is eol-comment\nvar y = 37;\n",
|
|
|
|
"x/* precomment\n */ --> is eol-comment\nvar y = 37;\n",
|
|
|
|
"var x = 42; --> is eol-comment\nvar y = 37;\n",
|
|
|
|
"var x = 42; /* precomment\n */ --> is eol-comment\nvar y = 37;\n",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
2010-09-07 12:52:16 +00:00
|
|
|
// Parser/Scanner needs a stack limit.
|
2014-07-03 08:52:28 +00:00
|
|
|
CcTest::i_isolate()->stack_guard()->SetStackLimit(GetCurrentStackPosition() -
|
|
|
|
128 * 1024);
|
2014-04-04 12:36:23 +00:00
|
|
|
uintptr_t stack_limit = CcTest::i_isolate()->stack_guard()->real_climit();
|
2010-09-02 07:36:30 +00:00
|
|
|
for (int i = 0; tests[i]; i++) {
|
2014-04-04 12:36:23 +00:00
|
|
|
const i::byte* source =
|
|
|
|
reinterpret_cast<const i::byte*>(tests[i]);
|
|
|
|
i::Utf8ToUtf16CharacterStream stream(source, i::StrLength(tests[i]));
|
|
|
|
i::CompleteParserRecorder log;
|
|
|
|
i::Scanner scanner(CcTest::i_isolate()->unicode_cache());
|
|
|
|
scanner.Initialize(&stream);
|
|
|
|
i::PreParser preparser(&scanner, &log, stack_limit);
|
|
|
|
preparser.set_allow_lazy(true);
|
|
|
|
i::PreParser::PreParseResult result = preparser.PreParseProgram();
|
|
|
|
CHECK_EQ(i::PreParser::kPreParseSuccess, result);
|
2014-07-10 10:28:05 +00:00
|
|
|
CHECK(!log.HasError());
|
2010-09-02 07:36:30 +00:00
|
|
|
}
|
2011-06-21 13:34:16 +00:00
|
|
|
|
|
|
|
for (int i = 0; fail_tests[i]; i++) {
|
2014-04-04 12:36:23 +00:00
|
|
|
const i::byte* source =
|
|
|
|
reinterpret_cast<const i::byte*>(fail_tests[i]);
|
|
|
|
i::Utf8ToUtf16CharacterStream stream(source, i::StrLength(fail_tests[i]));
|
|
|
|
i::CompleteParserRecorder log;
|
|
|
|
i::Scanner scanner(CcTest::i_isolate()->unicode_cache());
|
|
|
|
scanner.Initialize(&stream);
|
|
|
|
i::PreParser preparser(&scanner, &log, stack_limit);
|
|
|
|
preparser.set_allow_lazy(true);
|
|
|
|
i::PreParser::PreParseResult result = preparser.PreParseProgram();
|
|
|
|
// Even in the case of a syntax error, kPreParseSuccess is returned.
|
|
|
|
CHECK_EQ(i::PreParser::kPreParseSuccess, result);
|
2014-07-10 10:28:05 +00:00
|
|
|
CHECK(log.HasError());
|
2011-06-21 13:34:16 +00:00
|
|
|
}
|
2010-09-02 07:36:30 +00:00
|
|
|
}
|
2010-09-07 12:52:16 +00:00
|
|
|
|
|
|
|
|
|
|
|
class ScriptResource : public v8::String::ExternalAsciiStringResource {
|
|
|
|
public:
|
|
|
|
ScriptResource(const char* data, size_t length)
|
|
|
|
: data_(data), length_(length) { }
|
|
|
|
|
|
|
|
const char* data() const { return data_; }
|
|
|
|
size_t length() const { return length_; }
|
|
|
|
|
|
|
|
private:
|
|
|
|
const char* data_;
|
|
|
|
size_t length_;
|
|
|
|
};
|
|
|
|
|
|
|
|
|
2014-04-04 12:36:23 +00:00
|
|
|
TEST(UsingCachedData) {
|
2013-09-19 08:54:58 +00:00
|
|
|
v8::Isolate* isolate = CcTest::isolate();
|
2013-05-08 07:45:16 +00:00
|
|
|
v8::HandleScope handles(isolate);
|
|
|
|
v8::Local<v8::Context> context = v8::Context::New(isolate);
|
|
|
|
v8::Context::Scope context_scope(context);
|
2014-07-03 08:52:28 +00:00
|
|
|
CcTest::i_isolate()->stack_guard()->SetStackLimit(GetCurrentStackPosition() -
|
|
|
|
128 * 1024);
|
2010-09-07 12:52:16 +00:00
|
|
|
|
|
|
|
// Source containing functions that might be lazily compiled and all types
|
|
|
|
// of symbols (string, propertyName, regexp).
|
|
|
|
const char* source =
|
|
|
|
"var x = 42;"
|
|
|
|
"function foo(a) { return function nolazy(b) { return a + b; } }"
|
|
|
|
"function bar(a) { if (a) return function lazy(b) { return b; } }"
|
|
|
|
"var z = {'string': 'string literal', bareword: 'propertyName', "
|
|
|
|
" 42: 'number literal', for: 'keyword as propertyName', "
|
|
|
|
" f\\u006fr: 'keyword propertyname with escape'};"
|
|
|
|
"var v = /RegExp Literal/;"
|
|
|
|
"var w = /RegExp Literal\\u0020With Escape/gin;"
|
|
|
|
"var y = { get getter() { return 42; }, "
|
2014-07-21 09:58:01 +00:00
|
|
|
" set setter(v) { this.value = v; }};"
|
|
|
|
"var f = a => function (b) { return a + b; };"
|
|
|
|
"var g = a => b => a + b;";
|
2010-09-22 13:24:14 +00:00
|
|
|
int source_length = i::StrLength(source);
|
2010-09-07 12:52:16 +00:00
|
|
|
|
2014-04-04 12:36:23 +00:00
|
|
|
// ScriptResource will be deleted when the corresponding String is GCd.
|
|
|
|
v8::ScriptCompiler::Source script_source(v8::String::NewExternal(
|
|
|
|
isolate, new ScriptResource(source, source_length)));
|
2014-07-21 09:58:01 +00:00
|
|
|
i::FLAG_harmony_arrow_functions = true;
|
2014-04-04 12:36:23 +00:00
|
|
|
i::FLAG_min_preparse_length = 0;
|
|
|
|
v8::ScriptCompiler::Compile(isolate, &script_source,
|
Change ScriptCompiler::CompileOptions to allow for two 'cache' modes
(parser or code) and to be explicit about cache consumption or production
(rather than making presence of cached_data imply one or the other.)
Also add a --cache flag to d8, to allow testing the functionality.
-----------------------------
API change
Reason: Currently, V8 supports a 'parser cache' for repeatedly executing the same script. We'd like to add a 2nd mode that would cache code, and would like to let the embedder decide which mode they chose (if any).
Note: Previously, the 'use cached data' property was implied by the presence of the cached data itself. (That is, kNoCompileOptions and source->cached_data != NULL.) That is no longer sufficient, since the presence of data is no longer sufficient to determine /which kind/ of data is present.
Changes from old behaviour:
- If you previously didn't use caching, nothing changes.
Example:
v8::CompileUnbound(isolate, source, kNoCompileOptions);
- If you previously used caching, it worked like this:
- 1st run:
v8::CompileUnbound(isolate, source, kProduceToCache);
Then, source->cached_data would contain the
data-to-be cached. This remains the same, except you
need to tell V8 which type of data you want.
v8::CompileUnbound(isolate, source, kProduceParserCache);
- 2nd run:
v8::CompileUnbound(isolate, source, kNoCompileOptions);
with source->cached_data set to the data you received in
the first run. This will now ignore the cached data, and
you need to explicitly tell V8 to use it:
v8::CompileUnbound(isolate, source, kConsumeParserCache);
-----------------------------
BUG=
R=marja@chromium.org, yangguo@chromium.org
Review URL: https://codereview.chromium.org/389573006
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22431 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-16 12:18:33 +00:00
|
|
|
v8::ScriptCompiler::kProduceParserCache);
|
2014-04-04 12:36:23 +00:00
|
|
|
CHECK(script_source.GetCachedData());
|
2010-09-07 12:52:16 +00:00
|
|
|
|
2014-04-04 12:36:23 +00:00
|
|
|
// Compile the script again, using the cached data.
|
|
|
|
bool lazy_flag = i::FLAG_lazy;
|
|
|
|
i::FLAG_lazy = true;
|
Change ScriptCompiler::CompileOptions to allow for two 'cache' modes
(parser or code) and to be explicit about cache consumption or production
(rather than making presence of cached_data imply one or the other.)
Also add a --cache flag to d8, to allow testing the functionality.
-----------------------------
API change
Reason: Currently, V8 supports a 'parser cache' for repeatedly executing the same script. We'd like to add a 2nd mode that would cache code, and would like to let the embedder decide which mode they chose (if any).
Note: Previously, the 'use cached data' property was implied by the presence of the cached data itself. (That is, kNoCompileOptions and source->cached_data != NULL.) That is no longer sufficient, since the presence of data is no longer sufficient to determine /which kind/ of data is present.
Changes from old behaviour:
- If you previously didn't use caching, nothing changes.
Example:
v8::CompileUnbound(isolate, source, kNoCompileOptions);
- If you previously used caching, it worked like this:
- 1st run:
v8::CompileUnbound(isolate, source, kProduceToCache);
Then, source->cached_data would contain the
data-to-be cached. This remains the same, except you
need to tell V8 which type of data you want.
v8::CompileUnbound(isolate, source, kProduceParserCache);
- 2nd run:
v8::CompileUnbound(isolate, source, kNoCompileOptions);
with source->cached_data set to the data you received in
the first run. This will now ignore the cached data, and
you need to explicitly tell V8 to use it:
v8::CompileUnbound(isolate, source, kConsumeParserCache);
-----------------------------
BUG=
R=marja@chromium.org, yangguo@chromium.org
Review URL: https://codereview.chromium.org/389573006
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22431 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-16 12:18:33 +00:00
|
|
|
v8::ScriptCompiler::Compile(isolate, &script_source,
|
|
|
|
v8::ScriptCompiler::kConsumeParserCache);
|
2014-04-04 12:36:23 +00:00
|
|
|
i::FLAG_lazy = false;
|
Change ScriptCompiler::CompileOptions to allow for two 'cache' modes
(parser or code) and to be explicit about cache consumption or production
(rather than making presence of cached_data imply one or the other.)
Also add a --cache flag to d8, to allow testing the functionality.
-----------------------------
API change
Reason: Currently, V8 supports a 'parser cache' for repeatedly executing the same script. We'd like to add a 2nd mode that would cache code, and would like to let the embedder decide which mode they chose (if any).
Note: Previously, the 'use cached data' property was implied by the presence of the cached data itself. (That is, kNoCompileOptions and source->cached_data != NULL.) That is no longer sufficient, since the presence of data is no longer sufficient to determine /which kind/ of data is present.
Changes from old behaviour:
- If you previously didn't use caching, nothing changes.
Example:
v8::CompileUnbound(isolate, source, kNoCompileOptions);
- If you previously used caching, it worked like this:
- 1st run:
v8::CompileUnbound(isolate, source, kProduceToCache);
Then, source->cached_data would contain the
data-to-be cached. This remains the same, except you
need to tell V8 which type of data you want.
v8::CompileUnbound(isolate, source, kProduceParserCache);
- 2nd run:
v8::CompileUnbound(isolate, source, kNoCompileOptions);
with source->cached_data set to the data you received in
the first run. This will now ignore the cached data, and
you need to explicitly tell V8 to use it:
v8::CompileUnbound(isolate, source, kConsumeParserCache);
-----------------------------
BUG=
R=marja@chromium.org, yangguo@chromium.org
Review URL: https://codereview.chromium.org/389573006
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22431 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-16 12:18:33 +00:00
|
|
|
v8::ScriptCompiler::CompileUnbound(isolate, &script_source,
|
|
|
|
v8::ScriptCompiler::kConsumeParserCache);
|
2010-09-07 12:52:16 +00:00
|
|
|
i::FLAG_lazy = lazy_flag;
|
|
|
|
}
|
2010-11-02 07:21:37 +00:00
|
|
|
|
|
|
|
|
2014-03-17 15:06:33 +00:00
|
|
|
TEST(PreparseFunctionDataIsUsed) {
|
|
|
|
// This tests that we actually do use the function data generated by the
|
|
|
|
// preparser.
|
2014-03-19 13:24:13 +00:00
|
|
|
|
|
|
|
// Make preparsing work for short scripts.
|
|
|
|
i::FLAG_min_preparse_length = 0;
|
2014-07-21 09:58:01 +00:00
|
|
|
i::FLAG_harmony_arrow_functions = true;
|
2014-03-19 13:24:13 +00:00
|
|
|
|
2014-03-17 15:06:33 +00:00
|
|
|
v8::Isolate* isolate = CcTest::isolate();
|
|
|
|
v8::HandleScope handles(isolate);
|
|
|
|
v8::Local<v8::Context> context = v8::Context::New(isolate);
|
|
|
|
v8::Context::Scope context_scope(context);
|
2014-07-03 08:52:28 +00:00
|
|
|
CcTest::i_isolate()->stack_guard()->SetStackLimit(GetCurrentStackPosition() -
|
|
|
|
128 * 1024);
|
2014-03-17 15:06:33 +00:00
|
|
|
|
2014-07-21 09:58:01 +00:00
|
|
|
const char* good_code[] = {
|
|
|
|
"function this_is_lazy() { var a; } function foo() { return 25; } foo();",
|
|
|
|
"var this_is_lazy = () => { var a; }; var foo = () => 25; foo();",
|
|
|
|
};
|
2014-03-17 15:06:33 +00:00
|
|
|
|
|
|
|
// Insert a syntax error inside the lazy function.
|
2014-07-21 09:58:01 +00:00
|
|
|
const char* bad_code[] = {
|
|
|
|
"function this_is_lazy() { if ( } function foo() { return 25; } foo();",
|
|
|
|
"var this_is_lazy = () => { if ( }; var foo = () => 25; foo();",
|
|
|
|
};
|
2014-03-19 13:24:13 +00:00
|
|
|
|
2014-07-21 09:58:01 +00:00
|
|
|
for (unsigned i = 0; i < ARRAY_SIZE(good_code); i++) {
|
|
|
|
v8::ScriptCompiler::Source good_source(v8_str(good_code[i]));
|
|
|
|
v8::ScriptCompiler::Compile(isolate, &good_source,
|
|
|
|
v8::ScriptCompiler::kProduceDataToCache);
|
|
|
|
|
|
|
|
const v8::ScriptCompiler::CachedData* cached_data =
|
|
|
|
good_source.GetCachedData();
|
|
|
|
CHECK(cached_data->data != NULL);
|
|
|
|
CHECK_GT(cached_data->length, 0);
|
|
|
|
|
|
|
|
// Now compile the erroneous code with the good preparse data. If the
|
|
|
|
// preparse data is used, the lazy function is skipped and it should
|
|
|
|
// compile fine.
|
|
|
|
v8::ScriptCompiler::Source bad_source(
|
|
|
|
v8_str(bad_code[i]), new v8::ScriptCompiler::CachedData(
|
|
|
|
cached_data->data, cached_data->length));
|
|
|
|
v8::Local<v8::Value> result =
|
|
|
|
v8::ScriptCompiler::Compile(isolate, &bad_source)->Run();
|
|
|
|
CHECK(result->IsInt32());
|
|
|
|
CHECK_EQ(25, result->Int32Value());
|
|
|
|
}
|
2014-03-17 15:06:33 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
|
2010-11-02 07:21:37 +00:00
|
|
|
TEST(StandAlonePreParser) {
|
2011-08-04 15:18:18 +00:00
|
|
|
v8::V8::Initialize();
|
|
|
|
|
2014-07-03 08:52:28 +00:00
|
|
|
CcTest::i_isolate()->stack_guard()->SetStackLimit(GetCurrentStackPosition() -
|
|
|
|
128 * 1024);
|
2010-11-02 07:21:37 +00:00
|
|
|
|
|
|
|
const char* programs[] = {
|
2014-07-08 07:48:22 +00:00
|
|
|
"{label: 42}",
|
|
|
|
"var x = 42;",
|
|
|
|
"function foo(x, y) { return x + y; }",
|
|
|
|
"%ArgleBargle(glop);",
|
|
|
|
"var x = new new Function('this.x = 42');",
|
Implement handling of arrow functions in the parser
Arrow functions are parsed from ParseAssignmentExpression(). Handling the
parameter list is done by letting ParseConditionalExpression() parse a comma
separated list of identifiers, and it returns a tree of BinaryOperation nodes
with VariableProxy leaves, or a single VariableProxy if there is only one
parameter. When the arrow token "=>" is found, the VariableProxy nodes are
passed to ParseArrowFunctionLiteral(), which will then skip parsing the
paramaeter list. This avoids having to rewind when the arrow is found and
restart parsing the parameter list.
Note that the empty parameter list "()" is handled directly in
ParsePrimaryExpression(): after is has consumed the opening parenthesis,
if a closing parenthesis follows, then the only valid input is an arrow
function. In this case, ParsePrimaryExpression() directly calls
ParseArrowFunctionLiteral(), to avoid needing to return a sentinel value
to signal the empty parameter list. Because it will consume the body of
the arrow function, ParseAssignmentExpression() will not see the arrow
"=>" token as next, and return the already-parser expression.
The implementation is done in ParserBase, so it was needed to do some
additions to ParserBase, ParserTraits and PreParserTraits. Some of the
glue code can be removed later on when more more functionality is moved
to ParserBase.
Additionally, this adds a runtime flag "harmony_arrow_functions"
(disabled by default); enabling "harmony" will enable it as well.
BUG=v8:2700
LOG=N
R=marja@chromium.org
Review URL: https://codereview.chromium.org/383983002
Patch from Adrián Pérez de Castro <aperez@igalia.com>.
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22366 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-14 07:55:45 +00:00
|
|
|
"var f = (x, y) => x + y;",
|
2014-07-08 07:48:22 +00:00
|
|
|
NULL
|
|
|
|
};
|
2010-11-02 07:21:37 +00:00
|
|
|
|
2013-09-19 09:17:13 +00:00
|
|
|
uintptr_t stack_limit = CcTest::i_isolate()->stack_guard()->real_climit();
|
2010-11-02 07:21:37 +00:00
|
|
|
for (int i = 0; programs[i]; i++) {
|
|
|
|
const char* program = programs[i];
|
2012-03-12 12:35:28 +00:00
|
|
|
i::Utf8ToUtf16CharacterStream stream(
|
2010-12-07 14:03:59 +00:00
|
|
|
reinterpret_cast<const i::byte*>(program),
|
|
|
|
static_cast<unsigned>(strlen(program)));
|
2010-11-02 07:21:37 +00:00
|
|
|
i::CompleteParserRecorder log;
|
2013-09-19 09:17:13 +00:00
|
|
|
i::Scanner scanner(CcTest::i_isolate()->unicode_cache());
|
2010-12-07 14:03:59 +00:00
|
|
|
scanner.Initialize(&stream);
|
2010-11-29 13:24:37 +00:00
|
|
|
|
2013-10-14 13:07:20 +00:00
|
|
|
i::PreParser preparser(&scanner, &log, stack_limit);
|
Refactor parser mode configuration for correctness
This patch refactors the parser and preparser interface to be more
readable and type-safe. It has no behavior changes.
Previously, parsers and preparsers were configured via bitfield called
parser_flags in the Parser constructor, and flags in
PreParser::PreParseProgram, ParserApi::Parse, and ParserApi::PreParse.
This was error-prone in practice: six call sites passed incorrectly
typed values to this interface (a boolean FLAG value, a boolean false
and a boolean true value). None of these errors were caught by the
compiler because it's just an "int".
The parser flags interface was also awkward because it encoded a
language mode, but the language mode was only used to turn on harmony
scoping or not -- it wasn't used to actually set the parser's language
mode.
Fundamentally these errors came in because of the desire for a
procedural parser interface, in ParserApi. Because we need to be able
to configure the parser in various ways, the flags argument got added;
but no one understood how to use the flags properly. Also they were
only used by constructors: callers packed bits, and the constructors
unpacked them into booleans on the parser or preparser.
The solution is to allow parser construction, configuration, and
invocation to be separated. This patch does that.
It passes the existing tests.
BUG=
Review URL: https://codereview.chromium.org/13450007
Patch from Andy Wingo <wingo@igalia.com>.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14151 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2013-04-05 13:01:06 +00:00
|
|
|
preparser.set_allow_lazy(true);
|
|
|
|
preparser.set_allow_natives_syntax(true);
|
Implement handling of arrow functions in the parser
Arrow functions are parsed from ParseAssignmentExpression(). Handling the
parameter list is done by letting ParseConditionalExpression() parse a comma
separated list of identifiers, and it returns a tree of BinaryOperation nodes
with VariableProxy leaves, or a single VariableProxy if there is only one
parameter. When the arrow token "=>" is found, the VariableProxy nodes are
passed to ParseArrowFunctionLiteral(), which will then skip parsing the
paramaeter list. This avoids having to rewind when the arrow is found and
restart parsing the parameter list.
Note that the empty parameter list "()" is handled directly in
ParsePrimaryExpression(): after is has consumed the opening parenthesis,
if a closing parenthesis follows, then the only valid input is an arrow
function. In this case, ParsePrimaryExpression() directly calls
ParseArrowFunctionLiteral(), to avoid needing to return a sentinel value
to signal the empty parameter list. Because it will consume the body of
the arrow function, ParseAssignmentExpression() will not see the arrow
"=>" token as next, and return the already-parser expression.
The implementation is done in ParserBase, so it was needed to do some
additions to ParserBase, ParserTraits and PreParserTraits. Some of the
glue code can be removed later on when more more functionality is moved
to ParserBase.
Additionally, this adds a runtime flag "harmony_arrow_functions"
(disabled by default); enabling "harmony" will enable it as well.
BUG=v8:2700
LOG=N
R=marja@chromium.org
Review URL: https://codereview.chromium.org/383983002
Patch from Adrián Pérez de Castro <aperez@igalia.com>.
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22366 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-14 07:55:45 +00:00
|
|
|
preparser.set_allow_arrow_functions(true);
|
2013-10-14 13:07:20 +00:00
|
|
|
i::PreParser::PreParseResult result = preparser.PreParseProgram();
|
|
|
|
CHECK_EQ(i::PreParser::kPreParseSuccess, result);
|
2014-07-10 10:28:05 +00:00
|
|
|
CHECK(!log.HasError());
|
2010-11-02 07:21:37 +00:00
|
|
|
}
|
|
|
|
}
|
2010-11-11 10:18:48 +00:00
|
|
|
|
|
|
|
|
2011-10-17 12:45:52 +00:00
|
|
|
TEST(StandAlonePreParserNoNatives) {
|
|
|
|
v8::V8::Initialize();
|
|
|
|
|
2014-07-03 08:52:28 +00:00
|
|
|
CcTest::i_isolate()->stack_guard()->SetStackLimit(GetCurrentStackPosition() -
|
|
|
|
128 * 1024);
|
2011-10-17 12:45:52 +00:00
|
|
|
|
|
|
|
const char* programs[] = {
|
|
|
|
"%ArgleBargle(glop);",
|
|
|
|
"var x = %_IsSmi(42);",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
2013-09-19 09:17:13 +00:00
|
|
|
uintptr_t stack_limit = CcTest::i_isolate()->stack_guard()->real_climit();
|
2011-10-17 12:45:52 +00:00
|
|
|
for (int i = 0; programs[i]; i++) {
|
|
|
|
const char* program = programs[i];
|
2012-03-12 12:35:28 +00:00
|
|
|
i::Utf8ToUtf16CharacterStream stream(
|
2011-10-17 12:45:52 +00:00
|
|
|
reinterpret_cast<const i::byte*>(program),
|
|
|
|
static_cast<unsigned>(strlen(program)));
|
|
|
|
i::CompleteParserRecorder log;
|
2013-09-19 09:17:13 +00:00
|
|
|
i::Scanner scanner(CcTest::i_isolate()->unicode_cache());
|
2011-10-17 12:45:52 +00:00
|
|
|
scanner.Initialize(&stream);
|
|
|
|
|
Refactor parser mode configuration for correctness
This patch refactors the parser and preparser interface to be more
readable and type-safe. It has no behavior changes.
Previously, parsers and preparsers were configured via bitfield called
parser_flags in the Parser constructor, and flags in
PreParser::PreParseProgram, ParserApi::Parse, and ParserApi::PreParse.
This was error-prone in practice: six call sites passed incorrectly
typed values to this interface (a boolean FLAG value, a boolean false
and a boolean true value). None of these errors were caught by the
compiler because it's just an "int".
The parser flags interface was also awkward because it encoded a
language mode, but the language mode was only used to turn on harmony
scoping or not -- it wasn't used to actually set the parser's language
mode.
Fundamentally these errors came in because of the desire for a
procedural parser interface, in ParserApi. Because we need to be able
to configure the parser in various ways, the flags argument got added;
but no one understood how to use the flags properly. Also they were
only used by constructors: callers packed bits, and the constructors
unpacked them into booleans on the parser or preparser.
The solution is to allow parser construction, configuration, and
invocation to be separated. This patch does that.
It passes the existing tests.
BUG=
Review URL: https://codereview.chromium.org/13450007
Patch from Andy Wingo <wingo@igalia.com>.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14151 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2013-04-05 13:01:06 +00:00
|
|
|
// Preparser defaults to disallowing natives syntax.
|
2013-10-14 13:07:20 +00:00
|
|
|
i::PreParser preparser(&scanner, &log, stack_limit);
|
Refactor parser mode configuration for correctness
This patch refactors the parser and preparser interface to be more
readable and type-safe. It has no behavior changes.
Previously, parsers and preparsers were configured via bitfield called
parser_flags in the Parser constructor, and flags in
PreParser::PreParseProgram, ParserApi::Parse, and ParserApi::PreParse.
This was error-prone in practice: six call sites passed incorrectly
typed values to this interface (a boolean FLAG value, a boolean false
and a boolean true value). None of these errors were caught by the
compiler because it's just an "int".
The parser flags interface was also awkward because it encoded a
language mode, but the language mode was only used to turn on harmony
scoping or not -- it wasn't used to actually set the parser's language
mode.
Fundamentally these errors came in because of the desire for a
procedural parser interface, in ParserApi. Because we need to be able
to configure the parser in various ways, the flags argument got added;
but no one understood how to use the flags properly. Also they were
only used by constructors: callers packed bits, and the constructors
unpacked them into booleans on the parser or preparser.
The solution is to allow parser construction, configuration, and
invocation to be separated. This patch does that.
It passes the existing tests.
BUG=
Review URL: https://codereview.chromium.org/13450007
Patch from Andy Wingo <wingo@igalia.com>.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14151 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2013-04-05 13:01:06 +00:00
|
|
|
preparser.set_allow_lazy(true);
|
2013-10-14 13:07:20 +00:00
|
|
|
i::PreParser::PreParseResult result = preparser.PreParseProgram();
|
|
|
|
CHECK_EQ(i::PreParser::kPreParseSuccess, result);
|
2014-07-10 10:28:05 +00:00
|
|
|
CHECK(log.HasError());
|
2011-10-17 12:45:52 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
|
2014-02-20 11:35:37 +00:00
|
|
|
TEST(PreparsingObjectLiterals) {
|
|
|
|
// Regression test for a bug where the symbol stream produced by PreParser
|
|
|
|
// didn't match what Parser wanted to consume.
|
|
|
|
v8::Isolate* isolate = CcTest::isolate();
|
|
|
|
v8::HandleScope handles(isolate);
|
|
|
|
v8::Local<v8::Context> context = v8::Context::New(isolate);
|
|
|
|
v8::Context::Scope context_scope(context);
|
2014-07-03 08:52:28 +00:00
|
|
|
CcTest::i_isolate()->stack_guard()->SetStackLimit(GetCurrentStackPosition() -
|
|
|
|
128 * 1024);
|
2014-02-20 11:35:37 +00:00
|
|
|
|
|
|
|
{
|
|
|
|
const char* source = "var myo = {if: \"foo\"}; myo.if;";
|
Change ScriptCompiler::CompileOptions to allow for two 'cache' modes
(parser or code) and to be explicit about cache consumption or production
(rather than making presence of cached_data imply one or the other.)
Also add a --cache flag to d8, to allow testing the functionality.
-----------------------------
API change
Reason: Currently, V8 supports a 'parser cache' for repeatedly executing the same script. We'd like to add a 2nd mode that would cache code, and would like to let the embedder decide which mode they chose (if any).
Note: Previously, the 'use cached data' property was implied by the presence of the cached data itself. (That is, kNoCompileOptions and source->cached_data != NULL.) That is no longer sufficient, since the presence of data is no longer sufficient to determine /which kind/ of data is present.
Changes from old behaviour:
- If you previously didn't use caching, nothing changes.
Example:
v8::CompileUnbound(isolate, source, kNoCompileOptions);
- If you previously used caching, it worked like this:
- 1st run:
v8::CompileUnbound(isolate, source, kProduceToCache);
Then, source->cached_data would contain the
data-to-be cached. This remains the same, except you
need to tell V8 which type of data you want.
v8::CompileUnbound(isolate, source, kProduceParserCache);
- 2nd run:
v8::CompileUnbound(isolate, source, kNoCompileOptions);
with source->cached_data set to the data you received in
the first run. This will now ignore the cached data, and
you need to explicitly tell V8 to use it:
v8::CompileUnbound(isolate, source, kConsumeParserCache);
-----------------------------
BUG=
R=marja@chromium.org, yangguo@chromium.org
Review URL: https://codereview.chromium.org/389573006
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22431 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-16 12:18:33 +00:00
|
|
|
v8::Local<v8::Value> result = ParserCacheCompileRun(source);
|
2014-02-20 11:35:37 +00:00
|
|
|
CHECK(result->IsString());
|
|
|
|
v8::String::Utf8Value utf8(result);
|
|
|
|
CHECK_EQ("foo", *utf8);
|
|
|
|
}
|
|
|
|
|
|
|
|
{
|
|
|
|
const char* source = "var myo = {\"bar\": \"foo\"}; myo[\"bar\"];";
|
Change ScriptCompiler::CompileOptions to allow for two 'cache' modes
(parser or code) and to be explicit about cache consumption or production
(rather than making presence of cached_data imply one or the other.)
Also add a --cache flag to d8, to allow testing the functionality.
-----------------------------
API change
Reason: Currently, V8 supports a 'parser cache' for repeatedly executing the same script. We'd like to add a 2nd mode that would cache code, and would like to let the embedder decide which mode they chose (if any).
Note: Previously, the 'use cached data' property was implied by the presence of the cached data itself. (That is, kNoCompileOptions and source->cached_data != NULL.) That is no longer sufficient, since the presence of data is no longer sufficient to determine /which kind/ of data is present.
Changes from old behaviour:
- If you previously didn't use caching, nothing changes.
Example:
v8::CompileUnbound(isolate, source, kNoCompileOptions);
- If you previously used caching, it worked like this:
- 1st run:
v8::CompileUnbound(isolate, source, kProduceToCache);
Then, source->cached_data would contain the
data-to-be cached. This remains the same, except you
need to tell V8 which type of data you want.
v8::CompileUnbound(isolate, source, kProduceParserCache);
- 2nd run:
v8::CompileUnbound(isolate, source, kNoCompileOptions);
with source->cached_data set to the data you received in
the first run. This will now ignore the cached data, and
you need to explicitly tell V8 to use it:
v8::CompileUnbound(isolate, source, kConsumeParserCache);
-----------------------------
BUG=
R=marja@chromium.org, yangguo@chromium.org
Review URL: https://codereview.chromium.org/389573006
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22431 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-16 12:18:33 +00:00
|
|
|
v8::Local<v8::Value> result = ParserCacheCompileRun(source);
|
2014-02-20 11:35:37 +00:00
|
|
|
CHECK(result->IsString());
|
|
|
|
v8::String::Utf8Value utf8(result);
|
|
|
|
CHECK_EQ("foo", *utf8);
|
|
|
|
}
|
|
|
|
|
|
|
|
{
|
|
|
|
const char* source = "var myo = {1: \"foo\"}; myo[1];";
|
Change ScriptCompiler::CompileOptions to allow for two 'cache' modes
(parser or code) and to be explicit about cache consumption or production
(rather than making presence of cached_data imply one or the other.)
Also add a --cache flag to d8, to allow testing the functionality.
-----------------------------
API change
Reason: Currently, V8 supports a 'parser cache' for repeatedly executing the same script. We'd like to add a 2nd mode that would cache code, and would like to let the embedder decide which mode they chose (if any).
Note: Previously, the 'use cached data' property was implied by the presence of the cached data itself. (That is, kNoCompileOptions and source->cached_data != NULL.) That is no longer sufficient, since the presence of data is no longer sufficient to determine /which kind/ of data is present.
Changes from old behaviour:
- If you previously didn't use caching, nothing changes.
Example:
v8::CompileUnbound(isolate, source, kNoCompileOptions);
- If you previously used caching, it worked like this:
- 1st run:
v8::CompileUnbound(isolate, source, kProduceToCache);
Then, source->cached_data would contain the
data-to-be cached. This remains the same, except you
need to tell V8 which type of data you want.
v8::CompileUnbound(isolate, source, kProduceParserCache);
- 2nd run:
v8::CompileUnbound(isolate, source, kNoCompileOptions);
with source->cached_data set to the data you received in
the first run. This will now ignore the cached data, and
you need to explicitly tell V8 to use it:
v8::CompileUnbound(isolate, source, kConsumeParserCache);
-----------------------------
BUG=
R=marja@chromium.org, yangguo@chromium.org
Review URL: https://codereview.chromium.org/389573006
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22431 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-16 12:18:33 +00:00
|
|
|
v8::Local<v8::Value> result = ParserCacheCompileRun(source);
|
2014-02-20 11:35:37 +00:00
|
|
|
CHECK(result->IsString());
|
|
|
|
v8::String::Utf8Value utf8(result);
|
|
|
|
CHECK_EQ("foo", *utf8);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
|
2010-11-11 10:18:48 +00:00
|
|
|
TEST(RegressChromium62639) {
|
2011-08-04 15:18:18 +00:00
|
|
|
v8::V8::Initialize();
|
2013-09-19 09:17:13 +00:00
|
|
|
i::Isolate* isolate = CcTest::i_isolate();
|
2011-08-04 15:18:18 +00:00
|
|
|
|
2014-07-03 08:52:28 +00:00
|
|
|
isolate->stack_guard()->SetStackLimit(GetCurrentStackPosition() - 128 * 1024);
|
2010-11-11 10:18:48 +00:00
|
|
|
|
2010-11-11 13:47:15 +00:00
|
|
|
const char* program = "var x = 'something';\n"
|
|
|
|
"escape: function() {}";
|
2010-11-11 10:18:48 +00:00
|
|
|
// Fails parsing expecting an identifier after "function".
|
|
|
|
// Before fix, didn't check *ok after Expect(Token::Identifier, ok),
|
|
|
|
// and then used the invalid currently scanned literal. This always
|
|
|
|
// failed in debug mode, and sometimes crashed in release mode.
|
|
|
|
|
2012-03-12 12:35:28 +00:00
|
|
|
i::Utf8ToUtf16CharacterStream stream(
|
|
|
|
reinterpret_cast<const i::byte*>(program),
|
|
|
|
static_cast<unsigned>(strlen(program)));
|
2014-04-04 12:36:23 +00:00
|
|
|
i::CompleteParserRecorder log;
|
|
|
|
i::Scanner scanner(CcTest::i_isolate()->unicode_cache());
|
|
|
|
scanner.Initialize(&stream);
|
|
|
|
i::PreParser preparser(&scanner, &log,
|
|
|
|
CcTest::i_isolate()->stack_guard()->real_climit());
|
|
|
|
preparser.set_allow_lazy(true);
|
|
|
|
i::PreParser::PreParseResult result = preparser.PreParseProgram();
|
|
|
|
// Even in the case of a syntax error, kPreParseSuccess is returned.
|
|
|
|
CHECK_EQ(i::PreParser::kPreParseSuccess, result);
|
2014-07-10 10:28:05 +00:00
|
|
|
CHECK(log.HasError());
|
2010-11-11 10:18:48 +00:00
|
|
|
}
|
2010-11-11 11:45:25 +00:00
|
|
|
|
|
|
|
|
|
|
|
TEST(Regress928) {
|
2011-08-04 15:18:18 +00:00
|
|
|
v8::V8::Initialize();
|
2013-09-19 09:17:13 +00:00
|
|
|
i::Isolate* isolate = CcTest::i_isolate();
|
2013-06-04 10:30:05 +00:00
|
|
|
i::Factory* factory = isolate->factory();
|
2011-08-04 15:18:18 +00:00
|
|
|
|
2010-11-11 11:45:25 +00:00
|
|
|
// Preparsing didn't consider the catch clause of a try statement
|
|
|
|
// as with-content, which made it assume that a function inside
|
|
|
|
// the block could be lazily compiled, and an extra, unexpected,
|
|
|
|
// entry was added to the data.
|
2014-07-03 08:52:28 +00:00
|
|
|
isolate->stack_guard()->SetStackLimit(GetCurrentStackPosition() - 128 * 1024);
|
2010-11-11 11:45:25 +00:00
|
|
|
|
|
|
|
const char* program =
|
|
|
|
"try { } catch (e) { var foo = function () { /* first */ } }"
|
|
|
|
"var bar = function () { /* second */ }";
|
|
|
|
|
2013-09-19 08:54:58 +00:00
|
|
|
v8::HandleScope handles(CcTest::isolate());
|
2014-04-17 13:27:02 +00:00
|
|
|
i::Handle<i::String> source = factory->NewStringFromAsciiChecked(program);
|
2012-07-17 11:31:05 +00:00
|
|
|
i::GenericStringUtf16CharacterStream stream(source, 0, source->length());
|
2014-04-04 12:36:23 +00:00
|
|
|
i::CompleteParserRecorder log;
|
|
|
|
i::Scanner scanner(CcTest::i_isolate()->unicode_cache());
|
|
|
|
scanner.Initialize(&stream);
|
|
|
|
i::PreParser preparser(&scanner, &log,
|
|
|
|
CcTest::i_isolate()->stack_guard()->real_climit());
|
|
|
|
preparser.set_allow_lazy(true);
|
|
|
|
i::PreParser::PreParseResult result = preparser.PreParseProgram();
|
|
|
|
CHECK_EQ(i::PreParser::kPreParseSuccess, result);
|
2014-07-10 10:28:05 +00:00
|
|
|
i::ScriptData* sd = log.GetScriptData();
|
|
|
|
i::ParseData pd(sd);
|
|
|
|
pd.Initialize();
|
2010-11-11 11:45:25 +00:00
|
|
|
|
2011-02-16 13:37:26 +00:00
|
|
|
int first_function =
|
|
|
|
static_cast<int>(strstr(program, "function") - program);
|
2012-03-12 15:26:05 +00:00
|
|
|
int first_lbrace = first_function + i::StrLength("function () ");
|
2010-11-11 11:45:25 +00:00
|
|
|
CHECK_EQ('{', program[first_lbrace]);
|
2014-07-10 10:28:05 +00:00
|
|
|
i::FunctionEntry entry1 = pd.GetFunctionEntry(first_lbrace);
|
2010-11-11 11:45:25 +00:00
|
|
|
CHECK(!entry1.is_valid());
|
|
|
|
|
2011-02-16 13:37:26 +00:00
|
|
|
int second_function =
|
|
|
|
static_cast<int>(strstr(program + first_lbrace, "function") - program);
|
2011-02-16 13:42:26 +00:00
|
|
|
int second_lbrace =
|
2012-03-12 15:26:05 +00:00
|
|
|
second_function + i::StrLength("function () ");
|
2010-11-11 11:45:25 +00:00
|
|
|
CHECK_EQ('{', program[second_lbrace]);
|
2014-07-10 10:28:05 +00:00
|
|
|
i::FunctionEntry entry2 = pd.GetFunctionEntry(second_lbrace);
|
2010-11-11 11:45:25 +00:00
|
|
|
CHECK(entry2.is_valid());
|
|
|
|
CHECK_EQ('}', program[entry2.end_pos() - 1]);
|
2014-07-10 10:28:05 +00:00
|
|
|
delete sd;
|
2010-11-11 11:45:25 +00:00
|
|
|
}
|
2010-11-29 13:24:37 +00:00
|
|
|
|
|
|
|
|
|
|
|
TEST(PreParseOverflow) {
|
2011-08-04 15:18:18 +00:00
|
|
|
v8::V8::Initialize();
|
|
|
|
|
2014-07-03 08:52:28 +00:00
|
|
|
CcTest::i_isolate()->stack_guard()->SetStackLimit(GetCurrentStackPosition() -
|
|
|
|
128 * 1024);
|
2010-11-29 13:24:37 +00:00
|
|
|
|
|
|
|
size_t kProgramSize = 1024 * 1024;
|
2013-05-15 08:59:28 +00:00
|
|
|
i::SmartArrayPointer<char> program(i::NewArray<char>(kProgramSize + 1));
|
2013-12-09 07:41:20 +00:00
|
|
|
memset(program.get(), '(', kProgramSize);
|
2010-11-29 13:24:37 +00:00
|
|
|
program[kProgramSize] = '\0';
|
|
|
|
|
2013-09-19 09:17:13 +00:00
|
|
|
uintptr_t stack_limit = CcTest::i_isolate()->stack_guard()->real_climit();
|
2010-11-29 13:24:37 +00:00
|
|
|
|
2012-03-12 12:35:28 +00:00
|
|
|
i::Utf8ToUtf16CharacterStream stream(
|
2013-12-09 07:41:20 +00:00
|
|
|
reinterpret_cast<const i::byte*>(program.get()),
|
2010-12-07 14:03:59 +00:00
|
|
|
static_cast<unsigned>(kProgramSize));
|
2010-11-29 13:24:37 +00:00
|
|
|
i::CompleteParserRecorder log;
|
2013-09-19 09:17:13 +00:00
|
|
|
i::Scanner scanner(CcTest::i_isolate()->unicode_cache());
|
2010-12-07 14:03:59 +00:00
|
|
|
scanner.Initialize(&stream);
|
2010-11-29 13:24:37 +00:00
|
|
|
|
2013-10-14 13:07:20 +00:00
|
|
|
i::PreParser preparser(&scanner, &log, stack_limit);
|
Refactor parser mode configuration for correctness
This patch refactors the parser and preparser interface to be more
readable and type-safe. It has no behavior changes.
Previously, parsers and preparsers were configured via bitfield called
parser_flags in the Parser constructor, and flags in
PreParser::PreParseProgram, ParserApi::Parse, and ParserApi::PreParse.
This was error-prone in practice: six call sites passed incorrectly
typed values to this interface (a boolean FLAG value, a boolean false
and a boolean true value). None of these errors were caught by the
compiler because it's just an "int".
The parser flags interface was also awkward because it encoded a
language mode, but the language mode was only used to turn on harmony
scoping or not -- it wasn't used to actually set the parser's language
mode.
Fundamentally these errors came in because of the desire for a
procedural parser interface, in ParserApi. Because we need to be able
to configure the parser in various ways, the flags argument got added;
but no one understood how to use the flags properly. Also they were
only used by constructors: callers packed bits, and the constructors
unpacked them into booleans on the parser or preparser.
The solution is to allow parser construction, configuration, and
invocation to be separated. This patch does that.
It passes the existing tests.
BUG=
Review URL: https://codereview.chromium.org/13450007
Patch from Andy Wingo <wingo@igalia.com>.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14151 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2013-04-05 13:01:06 +00:00
|
|
|
preparser.set_allow_lazy(true);
|
2014-07-21 09:58:01 +00:00
|
|
|
preparser.set_allow_arrow_functions(true);
|
2013-10-14 13:07:20 +00:00
|
|
|
i::PreParser::PreParseResult result = preparser.PreParseProgram();
|
|
|
|
CHECK_EQ(i::PreParser::kPreParseStackOverflow, result);
|
2010-11-29 13:24:37 +00:00
|
|
|
}
|
2010-12-07 14:03:59 +00:00
|
|
|
|
|
|
|
|
|
|
|
class TestExternalResource: public v8::String::ExternalStringResource {
|
|
|
|
public:
|
|
|
|
explicit TestExternalResource(uint16_t* data, int length)
|
|
|
|
: data_(data), length_(static_cast<size_t>(length)) { }
|
|
|
|
|
|
|
|
~TestExternalResource() { }
|
|
|
|
|
|
|
|
const uint16_t* data() const {
|
|
|
|
return data_;
|
|
|
|
}
|
|
|
|
|
|
|
|
size_t length() const {
|
|
|
|
return length_;
|
|
|
|
}
|
|
|
|
private:
|
|
|
|
uint16_t* data_;
|
|
|
|
size_t length_;
|
|
|
|
};
|
|
|
|
|
|
|
|
|
|
|
|
#define CHECK_EQU(v1, v2) CHECK_EQ(static_cast<int>(v1), static_cast<int>(v2))
|
|
|
|
|
|
|
|
void TestCharacterStream(const char* ascii_source,
|
|
|
|
unsigned length,
|
|
|
|
unsigned start = 0,
|
|
|
|
unsigned end = 0) {
|
|
|
|
if (end == 0) end = length;
|
|
|
|
unsigned sub_length = end - start;
|
2013-09-19 09:17:13 +00:00
|
|
|
i::Isolate* isolate = CcTest::i_isolate();
|
2013-06-04 10:30:05 +00:00
|
|
|
i::Factory* factory = isolate->factory();
|
|
|
|
i::HandleScope test_scope(isolate);
|
2011-09-09 22:39:47 +00:00
|
|
|
i::SmartArrayPointer<i::uc16> uc16_buffer(new i::uc16[length]);
|
2010-12-07 14:03:59 +00:00
|
|
|
for (unsigned i = 0; i < length; i++) {
|
|
|
|
uc16_buffer[i] = static_cast<i::uc16>(ascii_source[i]);
|
|
|
|
}
|
|
|
|
i::Vector<const char> ascii_vector(ascii_source, static_cast<int>(length));
|
2014-04-17 13:27:02 +00:00
|
|
|
i::Handle<i::String> ascii_string =
|
|
|
|
factory->NewStringFromAscii(ascii_vector).ToHandleChecked();
|
2013-12-09 07:41:20 +00:00
|
|
|
TestExternalResource resource(uc16_buffer.get(), length);
|
2010-12-07 14:03:59 +00:00
|
|
|
i::Handle<i::String> uc16_string(
|
2014-04-03 12:41:37 +00:00
|
|
|
factory->NewExternalStringFromTwoByte(&resource).ToHandleChecked());
|
2010-12-07 14:03:59 +00:00
|
|
|
|
2012-03-12 12:35:28 +00:00
|
|
|
i::ExternalTwoByteStringUtf16CharacterStream uc16_stream(
|
2010-12-07 14:03:59 +00:00
|
|
|
i::Handle<i::ExternalTwoByteString>::cast(uc16_string), start, end);
|
2012-03-12 12:35:28 +00:00
|
|
|
i::GenericStringUtf16CharacterStream string_stream(ascii_string, start, end);
|
|
|
|
i::Utf8ToUtf16CharacterStream utf8_stream(
|
2010-12-07 14:03:59 +00:00
|
|
|
reinterpret_cast<const i::byte*>(ascii_source), end);
|
|
|
|
utf8_stream.SeekForward(start);
|
|
|
|
|
|
|
|
unsigned i = start;
|
|
|
|
while (i < end) {
|
|
|
|
// Read streams one char at a time
|
|
|
|
CHECK_EQU(i, uc16_stream.pos());
|
|
|
|
CHECK_EQU(i, string_stream.pos());
|
|
|
|
CHECK_EQU(i, utf8_stream.pos());
|
|
|
|
int32_t c0 = ascii_source[i];
|
|
|
|
int32_t c1 = uc16_stream.Advance();
|
|
|
|
int32_t c2 = string_stream.Advance();
|
|
|
|
int32_t c3 = utf8_stream.Advance();
|
|
|
|
i++;
|
|
|
|
CHECK_EQ(c0, c1);
|
|
|
|
CHECK_EQ(c0, c2);
|
|
|
|
CHECK_EQ(c0, c3);
|
|
|
|
CHECK_EQU(i, uc16_stream.pos());
|
|
|
|
CHECK_EQU(i, string_stream.pos());
|
|
|
|
CHECK_EQU(i, utf8_stream.pos());
|
|
|
|
}
|
|
|
|
while (i > start + sub_length / 4) {
|
|
|
|
// Pushback, re-read, pushback again.
|
|
|
|
int32_t c0 = ascii_source[i - 1];
|
|
|
|
CHECK_EQU(i, uc16_stream.pos());
|
|
|
|
CHECK_EQU(i, string_stream.pos());
|
|
|
|
CHECK_EQU(i, utf8_stream.pos());
|
|
|
|
uc16_stream.PushBack(c0);
|
|
|
|
string_stream.PushBack(c0);
|
|
|
|
utf8_stream.PushBack(c0);
|
|
|
|
i--;
|
|
|
|
CHECK_EQU(i, uc16_stream.pos());
|
|
|
|
CHECK_EQU(i, string_stream.pos());
|
|
|
|
CHECK_EQU(i, utf8_stream.pos());
|
|
|
|
int32_t c1 = uc16_stream.Advance();
|
|
|
|
int32_t c2 = string_stream.Advance();
|
|
|
|
int32_t c3 = utf8_stream.Advance();
|
|
|
|
i++;
|
|
|
|
CHECK_EQU(i, uc16_stream.pos());
|
|
|
|
CHECK_EQU(i, string_stream.pos());
|
|
|
|
CHECK_EQU(i, utf8_stream.pos());
|
|
|
|
CHECK_EQ(c0, c1);
|
|
|
|
CHECK_EQ(c0, c2);
|
|
|
|
CHECK_EQ(c0, c3);
|
|
|
|
uc16_stream.PushBack(c0);
|
|
|
|
string_stream.PushBack(c0);
|
|
|
|
utf8_stream.PushBack(c0);
|
|
|
|
i--;
|
|
|
|
CHECK_EQU(i, uc16_stream.pos());
|
|
|
|
CHECK_EQU(i, string_stream.pos());
|
|
|
|
CHECK_EQU(i, utf8_stream.pos());
|
|
|
|
}
|
|
|
|
unsigned halfway = start + sub_length / 2;
|
|
|
|
uc16_stream.SeekForward(halfway - i);
|
|
|
|
string_stream.SeekForward(halfway - i);
|
|
|
|
utf8_stream.SeekForward(halfway - i);
|
|
|
|
i = halfway;
|
|
|
|
CHECK_EQU(i, uc16_stream.pos());
|
|
|
|
CHECK_EQU(i, string_stream.pos());
|
|
|
|
CHECK_EQU(i, utf8_stream.pos());
|
|
|
|
|
|
|
|
while (i < end) {
|
|
|
|
// Read streams one char at a time
|
|
|
|
CHECK_EQU(i, uc16_stream.pos());
|
|
|
|
CHECK_EQU(i, string_stream.pos());
|
|
|
|
CHECK_EQU(i, utf8_stream.pos());
|
|
|
|
int32_t c0 = ascii_source[i];
|
|
|
|
int32_t c1 = uc16_stream.Advance();
|
|
|
|
int32_t c2 = string_stream.Advance();
|
|
|
|
int32_t c3 = utf8_stream.Advance();
|
|
|
|
i++;
|
|
|
|
CHECK_EQ(c0, c1);
|
|
|
|
CHECK_EQ(c0, c2);
|
|
|
|
CHECK_EQ(c0, c3);
|
|
|
|
CHECK_EQU(i, uc16_stream.pos());
|
|
|
|
CHECK_EQU(i, string_stream.pos());
|
|
|
|
CHECK_EQU(i, utf8_stream.pos());
|
|
|
|
}
|
|
|
|
|
|
|
|
int32_t c1 = uc16_stream.Advance();
|
|
|
|
int32_t c2 = string_stream.Advance();
|
|
|
|
int32_t c3 = utf8_stream.Advance();
|
|
|
|
CHECK_LT(c1, 0);
|
|
|
|
CHECK_LT(c2, 0);
|
|
|
|
CHECK_LT(c3, 0);
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
TEST(CharacterStreams) {
|
2013-09-19 08:54:58 +00:00
|
|
|
v8::Isolate* isolate = CcTest::isolate();
|
2013-05-08 07:45:16 +00:00
|
|
|
v8::HandleScope handles(isolate);
|
|
|
|
v8::Local<v8::Context> context = v8::Context::New(isolate);
|
|
|
|
v8::Context::Scope context_scope(context);
|
2010-12-07 14:03:59 +00:00
|
|
|
|
|
|
|
TestCharacterStream("abc\0\n\r\x7f", 7);
|
|
|
|
static const unsigned kBigStringSize = 4096;
|
|
|
|
char buffer[kBigStringSize + 1];
|
|
|
|
for (unsigned i = 0; i < kBigStringSize; i++) {
|
|
|
|
buffer[i] = static_cast<char>(i & 0x7f);
|
|
|
|
}
|
|
|
|
TestCharacterStream(buffer, kBigStringSize);
|
|
|
|
|
|
|
|
TestCharacterStream(buffer, kBigStringSize, 576, 3298);
|
|
|
|
|
|
|
|
TestCharacterStream("\0", 1);
|
|
|
|
TestCharacterStream("", 0);
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
TEST(Utf8CharacterStream) {
|
|
|
|
static const unsigned kMaxUC16CharU = unibrow::Utf8::kMaxThreeByteChar;
|
|
|
|
static const int kMaxUC16Char = static_cast<int>(kMaxUC16CharU);
|
|
|
|
|
|
|
|
static const int kAllUtf8CharsSize =
|
|
|
|
(unibrow::Utf8::kMaxOneByteChar + 1) +
|
|
|
|
(unibrow::Utf8::kMaxTwoByteChar - unibrow::Utf8::kMaxOneByteChar) * 2 +
|
|
|
|
(unibrow::Utf8::kMaxThreeByteChar - unibrow::Utf8::kMaxTwoByteChar) * 3;
|
|
|
|
static const unsigned kAllUtf8CharsSizeU =
|
|
|
|
static_cast<unsigned>(kAllUtf8CharsSize);
|
|
|
|
|
|
|
|
char buffer[kAllUtf8CharsSizeU];
|
|
|
|
unsigned cursor = 0;
|
|
|
|
for (int i = 0; i <= kMaxUC16Char; i++) {
|
2012-03-12 12:35:28 +00:00
|
|
|
cursor += unibrow::Utf8::Encode(buffer + cursor,
|
|
|
|
i,
|
|
|
|
unibrow::Utf16::kNoPreviousCharacter);
|
2010-12-07 14:03:59 +00:00
|
|
|
}
|
|
|
|
ASSERT(cursor == kAllUtf8CharsSizeU);
|
|
|
|
|
2012-03-12 12:35:28 +00:00
|
|
|
i::Utf8ToUtf16CharacterStream stream(reinterpret_cast<const i::byte*>(buffer),
|
|
|
|
kAllUtf8CharsSizeU);
|
2010-12-07 14:03:59 +00:00
|
|
|
for (int i = 0; i <= kMaxUC16Char; i++) {
|
|
|
|
CHECK_EQU(i, stream.pos());
|
|
|
|
int32_t c = stream.Advance();
|
|
|
|
CHECK_EQ(i, c);
|
|
|
|
CHECK_EQU(i + 1, stream.pos());
|
|
|
|
}
|
|
|
|
for (int i = kMaxUC16Char; i >= 0; i--) {
|
|
|
|
CHECK_EQU(i + 1, stream.pos());
|
|
|
|
stream.PushBack(i);
|
|
|
|
CHECK_EQU(i, stream.pos());
|
|
|
|
}
|
|
|
|
int i = 0;
|
|
|
|
while (stream.pos() < kMaxUC16CharU) {
|
|
|
|
CHECK_EQU(i, stream.pos());
|
|
|
|
unsigned progress = stream.SeekForward(12);
|
|
|
|
i += progress;
|
|
|
|
int32_t c = stream.Advance();
|
|
|
|
if (i <= kMaxUC16Char) {
|
|
|
|
CHECK_EQ(i, c);
|
|
|
|
} else {
|
|
|
|
CHECK_EQ(-1, c);
|
|
|
|
}
|
|
|
|
i += 1;
|
|
|
|
CHECK_EQU(i, stream.pos());
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
#undef CHECK_EQU
|
|
|
|
|
2012-03-12 12:35:28 +00:00
|
|
|
void TestStreamScanner(i::Utf16CharacterStream* stream,
|
2010-12-07 14:03:59 +00:00
|
|
|
i::Token::Value* expected_tokens,
|
|
|
|
int skip_pos = 0, // Zero means not skipping.
|
|
|
|
int skip_to = 0) {
|
2013-09-19 09:17:13 +00:00
|
|
|
i::Scanner scanner(CcTest::i_isolate()->unicode_cache());
|
2010-12-22 20:14:19 +00:00
|
|
|
scanner.Initialize(stream);
|
2010-12-07 14:03:59 +00:00
|
|
|
|
|
|
|
int i = 0;
|
|
|
|
do {
|
|
|
|
i::Token::Value expected = expected_tokens[i];
|
|
|
|
i::Token::Value actual = scanner.Next();
|
|
|
|
CHECK_EQ(i::Token::String(expected), i::Token::String(actual));
|
|
|
|
if (scanner.location().end_pos == skip_pos) {
|
|
|
|
scanner.SeekForward(skip_to);
|
|
|
|
}
|
|
|
|
i++;
|
|
|
|
} while (expected_tokens[i] != i::Token::ILLEGAL);
|
|
|
|
}
|
|
|
|
|
2013-07-05 09:52:11 +00:00
|
|
|
|
2010-12-07 14:03:59 +00:00
|
|
|
TEST(StreamScanner) {
|
2011-08-04 15:18:18 +00:00
|
|
|
v8::V8::Initialize();
|
|
|
|
|
2010-12-07 14:03:59 +00:00
|
|
|
const char* str1 = "{ foo get for : */ <- \n\n /*foo*/ bib";
|
2012-03-12 12:35:28 +00:00
|
|
|
i::Utf8ToUtf16CharacterStream stream1(reinterpret_cast<const i::byte*>(str1),
|
|
|
|
static_cast<unsigned>(strlen(str1)));
|
2010-12-07 14:03:59 +00:00
|
|
|
i::Token::Value expectations1[] = {
|
|
|
|
i::Token::LBRACE,
|
|
|
|
i::Token::IDENTIFIER,
|
|
|
|
i::Token::IDENTIFIER,
|
|
|
|
i::Token::FOR,
|
|
|
|
i::Token::COLON,
|
|
|
|
i::Token::MUL,
|
|
|
|
i::Token::DIV,
|
|
|
|
i::Token::LT,
|
|
|
|
i::Token::SUB,
|
|
|
|
i::Token::IDENTIFIER,
|
|
|
|
i::Token::EOS,
|
|
|
|
i::Token::ILLEGAL
|
|
|
|
};
|
|
|
|
TestStreamScanner(&stream1, expectations1, 0, 0);
|
|
|
|
|
|
|
|
const char* str2 = "case default const {THIS\nPART\nSKIPPED} do";
|
2012-03-12 12:35:28 +00:00
|
|
|
i::Utf8ToUtf16CharacterStream stream2(reinterpret_cast<const i::byte*>(str2),
|
|
|
|
static_cast<unsigned>(strlen(str2)));
|
2010-12-07 14:03:59 +00:00
|
|
|
i::Token::Value expectations2[] = {
|
|
|
|
i::Token::CASE,
|
|
|
|
i::Token::DEFAULT,
|
|
|
|
i::Token::CONST,
|
|
|
|
i::Token::LBRACE,
|
|
|
|
// Skipped part here
|
|
|
|
i::Token::RBRACE,
|
|
|
|
i::Token::DO,
|
|
|
|
i::Token::EOS,
|
|
|
|
i::Token::ILLEGAL
|
|
|
|
};
|
|
|
|
ASSERT_EQ('{', str2[19]);
|
|
|
|
ASSERT_EQ('}', str2[37]);
|
|
|
|
TestStreamScanner(&stream2, expectations2, 20, 37);
|
|
|
|
|
|
|
|
const char* str3 = "{}}}}";
|
|
|
|
i::Token::Value expectations3[] = {
|
|
|
|
i::Token::LBRACE,
|
|
|
|
i::Token::RBRACE,
|
|
|
|
i::Token::RBRACE,
|
|
|
|
i::Token::RBRACE,
|
|
|
|
i::Token::RBRACE,
|
|
|
|
i::Token::EOS,
|
|
|
|
i::Token::ILLEGAL
|
|
|
|
};
|
|
|
|
// Skip zero-four RBRACEs.
|
|
|
|
for (int i = 0; i <= 4; i++) {
|
|
|
|
expectations3[6 - i] = i::Token::ILLEGAL;
|
|
|
|
expectations3[5 - i] = i::Token::EOS;
|
2012-03-12 12:35:28 +00:00
|
|
|
i::Utf8ToUtf16CharacterStream stream3(
|
2010-12-07 14:03:59 +00:00
|
|
|
reinterpret_cast<const i::byte*>(str3),
|
|
|
|
static_cast<unsigned>(strlen(str3)));
|
|
|
|
TestStreamScanner(&stream3, expectations3, 1, 1 + i);
|
|
|
|
}
|
|
|
|
}
|
2011-01-07 12:35:42 +00:00
|
|
|
|
|
|
|
|
|
|
|
void TestScanRegExp(const char* re_source, const char* expected) {
|
2012-03-12 12:35:28 +00:00
|
|
|
i::Utf8ToUtf16CharacterStream stream(
|
2011-01-07 12:35:42 +00:00
|
|
|
reinterpret_cast<const i::byte*>(re_source),
|
|
|
|
static_cast<unsigned>(strlen(re_source)));
|
2014-03-13 08:29:31 +00:00
|
|
|
i::HandleScope scope(CcTest::i_isolate());
|
2013-09-19 09:17:13 +00:00
|
|
|
i::Scanner scanner(CcTest::i_isolate()->unicode_cache());
|
2011-01-07 12:35:42 +00:00
|
|
|
scanner.Initialize(&stream);
|
|
|
|
|
|
|
|
i::Token::Value start = scanner.peek();
|
|
|
|
CHECK(start == i::Token::DIV || start == i::Token::ASSIGN_DIV);
|
|
|
|
CHECK(scanner.ScanRegExpPattern(start == i::Token::ASSIGN_DIV));
|
|
|
|
scanner.Next(); // Current token is now the regexp literal.
|
2014-06-24 14:03:24 +00:00
|
|
|
i::Zone zone(CcTest::i_isolate());
|
|
|
|
i::AstValueFactory ast_value_factory(&zone,
|
|
|
|
CcTest::i_isolate()->heap()->HashSeed());
|
|
|
|
ast_value_factory.Internalize(CcTest::i_isolate());
|
2014-03-13 08:29:31 +00:00
|
|
|
i::Handle<i::String> val =
|
2014-06-24 14:03:24 +00:00
|
|
|
scanner.CurrentSymbol(&ast_value_factory)->string();
|
2014-03-13 08:29:31 +00:00
|
|
|
i::DisallowHeapAllocation no_alloc;
|
|
|
|
i::String::FlatContent content = val->GetFlatContent();
|
|
|
|
CHECK(content.IsAscii());
|
|
|
|
i::Vector<const uint8_t> actual = content.ToOneByteVector();
|
2011-01-07 12:35:42 +00:00
|
|
|
for (int i = 0; i < actual.length(); i++) {
|
|
|
|
CHECK_NE('\0', expected[i]);
|
|
|
|
CHECK_EQ(expected[i], actual[i]);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
TEST(RegExpScanning) {
|
2011-08-04 15:18:18 +00:00
|
|
|
v8::V8::Initialize();
|
|
|
|
|
2011-01-07 12:35:42 +00:00
|
|
|
// RegExp token with added garbage at the end. The scanner should only
|
|
|
|
// scan the RegExp until the terminating slash just before "flipperwald".
|
|
|
|
TestScanRegExp("/b/flipperwald", "b");
|
|
|
|
// Incomplete escape sequences doesn't hide the terminating slash.
|
|
|
|
TestScanRegExp("/\\x/flipperwald", "\\x");
|
|
|
|
TestScanRegExp("/\\u/flipperwald", "\\u");
|
|
|
|
TestScanRegExp("/\\u1/flipperwald", "\\u1");
|
|
|
|
TestScanRegExp("/\\u12/flipperwald", "\\u12");
|
|
|
|
TestScanRegExp("/\\u123/flipperwald", "\\u123");
|
|
|
|
TestScanRegExp("/\\c/flipperwald", "\\c");
|
|
|
|
TestScanRegExp("/\\c//flipperwald", "\\c");
|
|
|
|
// Slashes inside character classes are not terminating.
|
|
|
|
TestScanRegExp("/[/]/flipperwald", "[/]");
|
|
|
|
TestScanRegExp("/[\\s-/]/flipperwald", "[\\s-/]");
|
|
|
|
// Incomplete escape sequences inside a character class doesn't hide
|
|
|
|
// the end of the character class.
|
|
|
|
TestScanRegExp("/[\\c/]/flipperwald", "[\\c/]");
|
|
|
|
TestScanRegExp("/[\\c]/flipperwald", "[\\c]");
|
|
|
|
TestScanRegExp("/[\\x]/flipperwald", "[\\x]");
|
|
|
|
TestScanRegExp("/[\\x1]/flipperwald", "[\\x1]");
|
|
|
|
TestScanRegExp("/[\\u]/flipperwald", "[\\u]");
|
|
|
|
TestScanRegExp("/[\\u1]/flipperwald", "[\\u1]");
|
|
|
|
TestScanRegExp("/[\\u12]/flipperwald", "[\\u12]");
|
|
|
|
TestScanRegExp("/[\\u123]/flipperwald", "[\\u123]");
|
|
|
|
// Escaped ']'s wont end the character class.
|
|
|
|
TestScanRegExp("/[\\]/]/flipperwald", "[\\]/]");
|
|
|
|
// Escaped slashes are not terminating.
|
|
|
|
TestScanRegExp("/\\//flipperwald", "\\/");
|
|
|
|
// Starting with '=' works too.
|
|
|
|
TestScanRegExp("/=/", "=");
|
|
|
|
TestScanRegExp("/=?/", "=?");
|
|
|
|
}
|
2011-10-21 10:26:59 +00:00
|
|
|
|
|
|
|
|
2012-03-12 12:35:28 +00:00
|
|
|
static int Utf8LengthHelper(const char* s) {
|
2012-03-12 15:26:05 +00:00
|
|
|
int len = i::StrLength(s);
|
2012-03-12 12:35:28 +00:00
|
|
|
int character_length = len;
|
|
|
|
for (int i = 0; i < len; i++) {
|
|
|
|
unsigned char c = s[i];
|
|
|
|
int input_offset = 0;
|
|
|
|
int output_adjust = 0;
|
|
|
|
if (c > 0x7f) {
|
|
|
|
if (c < 0xc0) continue;
|
|
|
|
if (c >= 0xf0) {
|
|
|
|
if (c >= 0xf8) {
|
|
|
|
// 5 and 6 byte UTF-8 sequences turn into a kBadChar for each UTF-8
|
|
|
|
// byte.
|
|
|
|
continue; // Handle first UTF-8 byte.
|
|
|
|
}
|
|
|
|
if ((c & 7) == 0 && ((s[i + 1] & 0x30) == 0)) {
|
|
|
|
// This 4 byte sequence could have been coded as a 3 byte sequence.
|
|
|
|
// Record a single kBadChar for the first byte and continue.
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
input_offset = 3;
|
|
|
|
// 4 bytes of UTF-8 turn into 2 UTF-16 code units.
|
|
|
|
character_length -= 2;
|
|
|
|
} else if (c >= 0xe0) {
|
|
|
|
if ((c & 0xf) == 0 && ((s[i + 1] & 0x20) == 0)) {
|
|
|
|
// This 3 byte sequence could have been coded as a 2 byte sequence.
|
|
|
|
// Record a single kBadChar for the first byte and continue.
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
input_offset = 2;
|
|
|
|
// 3 bytes of UTF-8 turn into 1 UTF-16 code unit.
|
|
|
|
output_adjust = 2;
|
|
|
|
} else {
|
|
|
|
if ((c & 0x1e) == 0) {
|
|
|
|
// This 2 byte sequence could have been coded as a 1 byte sequence.
|
|
|
|
// Record a single kBadChar for the first byte and continue.
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
input_offset = 1;
|
|
|
|
// 2 bytes of UTF-8 turn into 1 UTF-16 code unit.
|
|
|
|
output_adjust = 1;
|
|
|
|
}
|
|
|
|
bool bad = false;
|
|
|
|
for (int j = 1; j <= input_offset; j++) {
|
|
|
|
if ((s[i + j] & 0xc0) != 0x80) {
|
|
|
|
// Bad UTF-8 sequence turns the first in the sequence into kBadChar,
|
|
|
|
// which is a single UTF-16 code unit.
|
|
|
|
bad = true;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
if (!bad) {
|
|
|
|
i += input_offset;
|
|
|
|
character_length -= output_adjust;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
return character_length;
|
|
|
|
}
|
|
|
|
|
|
|
|
|
2011-10-21 10:26:59 +00:00
|
|
|
TEST(ScopePositions) {
|
2014-03-11 14:41:22 +00:00
|
|
|
v8::internal::FLAG_harmony_scoping = true;
|
|
|
|
|
2011-10-21 10:26:59 +00:00
|
|
|
// Test the parser for correctly setting the start and end positions
|
|
|
|
// of a scope. We check the scope positions of exactly one scope
|
|
|
|
// nested in the global scope of a program. 'inner source' is the
|
|
|
|
// source code that determines the part of the source belonging
|
|
|
|
// to the nested scope. 'outer_prefix' and 'outer_suffix' are
|
|
|
|
// parts of the source that belong to the global scope.
|
|
|
|
struct SourceData {
|
|
|
|
const char* outer_prefix;
|
|
|
|
const char* inner_source;
|
|
|
|
const char* outer_suffix;
|
|
|
|
i::ScopeType scope_type;
|
2014-03-11 14:41:22 +00:00
|
|
|
i::StrictMode strict_mode;
|
2011-10-21 10:26:59 +00:00
|
|
|
};
|
|
|
|
|
|
|
|
const SourceData source_data[] = {
|
2014-03-11 14:41:22 +00:00
|
|
|
{ " with ({}) ", "{ block; }", " more;", i::WITH_SCOPE, i::SLOPPY },
|
|
|
|
{ " with ({}) ", "{ block; }", "; more;", i::WITH_SCOPE, i::SLOPPY },
|
2011-10-21 10:26:59 +00:00
|
|
|
{ " with ({}) ", "{\n"
|
|
|
|
" block;\n"
|
|
|
|
" }", "\n"
|
2014-03-11 14:41:22 +00:00
|
|
|
" more;", i::WITH_SCOPE, i::SLOPPY },
|
|
|
|
{ " with ({}) ", "statement;", " more;", i::WITH_SCOPE, i::SLOPPY },
|
2011-10-21 10:26:59 +00:00
|
|
|
{ " with ({}) ", "statement", "\n"
|
2014-03-11 14:41:22 +00:00
|
|
|
" more;", i::WITH_SCOPE, i::SLOPPY },
|
2011-10-21 10:26:59 +00:00
|
|
|
{ " with ({})\n"
|
|
|
|
" ", "statement;", "\n"
|
2014-03-11 14:41:22 +00:00
|
|
|
" more;", i::WITH_SCOPE, i::SLOPPY },
|
2011-11-24 15:17:04 +00:00
|
|
|
{ " try {} catch ", "(e) { block; }", " more;",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::CATCH_SCOPE, i::SLOPPY },
|
2011-11-24 15:17:04 +00:00
|
|
|
{ " try {} catch ", "(e) { block; }", "; more;",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::CATCH_SCOPE, i::SLOPPY },
|
2011-10-21 10:26:59 +00:00
|
|
|
{ " try {} catch ", "(e) {\n"
|
|
|
|
" block;\n"
|
|
|
|
" }", "\n"
|
2014-03-11 14:41:22 +00:00
|
|
|
" more;", i::CATCH_SCOPE, i::SLOPPY },
|
2011-10-21 10:26:59 +00:00
|
|
|
{ " try {} catch ", "(e) { block; }", " finally { block; } more;",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::CATCH_SCOPE, i::SLOPPY },
|
2011-10-21 10:26:59 +00:00
|
|
|
{ " start;\n"
|
2014-03-11 14:41:22 +00:00
|
|
|
" ", "{ let block; }", " more;", i::BLOCK_SCOPE, i::STRICT },
|
2011-10-21 10:26:59 +00:00
|
|
|
{ " start;\n"
|
2014-03-11 14:41:22 +00:00
|
|
|
" ", "{ let block; }", "; more;", i::BLOCK_SCOPE, i::STRICT },
|
2011-10-21 10:26:59 +00:00
|
|
|
{ " start;\n"
|
|
|
|
" ", "{\n"
|
|
|
|
" let block;\n"
|
|
|
|
" }", "\n"
|
2014-03-11 14:41:22 +00:00
|
|
|
" more;", i::BLOCK_SCOPE, i::STRICT },
|
2011-10-21 10:26:59 +00:00
|
|
|
{ " start;\n"
|
|
|
|
" function fun", "(a,b) { infunction; }", " more;",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::FUNCTION_SCOPE, i::SLOPPY },
|
2011-10-21 10:26:59 +00:00
|
|
|
{ " start;\n"
|
|
|
|
" function fun", "(a,b) {\n"
|
|
|
|
" infunction;\n"
|
|
|
|
" }", "\n"
|
2014-03-11 14:41:22 +00:00
|
|
|
" more;", i::FUNCTION_SCOPE, i::SLOPPY },
|
2014-07-21 09:58:01 +00:00
|
|
|
// TODO(aperez): Change to use i::ARROW_SCOPE when implemented
|
|
|
|
{ " start;\n", "(a,b) => a + b", "; more;",
|
|
|
|
i::FUNCTION_SCOPE, i::SLOPPY },
|
|
|
|
{ " start;\n", "(a,b) => { return a+b; }", "\nmore;",
|
|
|
|
i::FUNCTION_SCOPE, i::SLOPPY },
|
|
|
|
{ " start;\n"
|
|
|
|
" (function fun", "(a,b) { infunction; }", ")();",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::FUNCTION_SCOPE, i::SLOPPY },
|
2011-10-21 10:26:59 +00:00
|
|
|
{ " for ", "(let x = 1 ; x < 10; ++ x) { block; }", " more;",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::BLOCK_SCOPE, i::STRICT },
|
2011-10-21 10:26:59 +00:00
|
|
|
{ " for ", "(let x = 1 ; x < 10; ++ x) { block; }", "; more;",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::BLOCK_SCOPE, i::STRICT },
|
2011-10-21 10:26:59 +00:00
|
|
|
{ " for ", "(let x = 1 ; x < 10; ++ x) {\n"
|
|
|
|
" block;\n"
|
|
|
|
" }", "\n"
|
2014-03-11 14:41:22 +00:00
|
|
|
" more;", i::BLOCK_SCOPE, i::STRICT },
|
2011-10-21 10:26:59 +00:00
|
|
|
{ " for ", "(let x = 1 ; x < 10; ++ x) statement;", " more;",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::BLOCK_SCOPE, i::STRICT },
|
2011-10-21 10:26:59 +00:00
|
|
|
{ " for ", "(let x = 1 ; x < 10; ++ x) statement", "\n"
|
2014-03-11 14:41:22 +00:00
|
|
|
" more;", i::BLOCK_SCOPE, i::STRICT },
|
2011-10-21 10:26:59 +00:00
|
|
|
{ " for ", "(let x = 1 ; x < 10; ++ x)\n"
|
|
|
|
" statement;", "\n"
|
2014-03-11 14:41:22 +00:00
|
|
|
" more;", i::BLOCK_SCOPE, i::STRICT },
|
2011-11-24 15:17:04 +00:00
|
|
|
{ " for ", "(let x in {}) { block; }", " more;",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::BLOCK_SCOPE, i::STRICT },
|
2011-11-24 15:17:04 +00:00
|
|
|
{ " for ", "(let x in {}) { block; }", "; more;",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::BLOCK_SCOPE, i::STRICT },
|
2011-10-21 10:26:59 +00:00
|
|
|
{ " for ", "(let x in {}) {\n"
|
|
|
|
" block;\n"
|
|
|
|
" }", "\n"
|
2014-03-11 14:41:22 +00:00
|
|
|
" more;", i::BLOCK_SCOPE, i::STRICT },
|
2011-11-24 15:17:04 +00:00
|
|
|
{ " for ", "(let x in {}) statement;", " more;",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::BLOCK_SCOPE, i::STRICT },
|
2011-10-21 10:26:59 +00:00
|
|
|
{ " for ", "(let x in {}) statement", "\n"
|
2014-03-11 14:41:22 +00:00
|
|
|
" more;", i::BLOCK_SCOPE, i::STRICT },
|
2011-10-21 10:26:59 +00:00
|
|
|
{ " for ", "(let x in {})\n"
|
|
|
|
" statement;", "\n"
|
2014-03-11 14:41:22 +00:00
|
|
|
" more;", i::BLOCK_SCOPE, i::STRICT },
|
2012-03-12 12:35:28 +00:00
|
|
|
// Check that 6-byte and 4-byte encodings of UTF-8 strings do not throw
|
|
|
|
// the preparser off in terms of byte offsets.
|
|
|
|
// 6 byte encoding.
|
|
|
|
{ " 'foo\355\240\201\355\260\211';\n"
|
|
|
|
" (function fun", "(a,b) { infunction; }", ")();",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::FUNCTION_SCOPE, i::SLOPPY },
|
2012-03-12 12:35:28 +00:00
|
|
|
// 4 byte encoding.
|
|
|
|
{ " 'foo\360\220\220\212';\n"
|
|
|
|
" (function fun", "(a,b) { infunction; }", ")();",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::FUNCTION_SCOPE, i::SLOPPY },
|
2012-03-12 12:35:28 +00:00
|
|
|
// 3 byte encoding of \u0fff.
|
|
|
|
{ " 'foo\340\277\277';\n"
|
|
|
|
" (function fun", "(a,b) { infunction; }", ")();",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::FUNCTION_SCOPE, i::SLOPPY },
|
2012-03-12 12:35:28 +00:00
|
|
|
// Broken 6 byte encoding with missing last byte.
|
|
|
|
{ " 'foo\355\240\201\355\211';\n"
|
|
|
|
" (function fun", "(a,b) { infunction; }", ")();",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::FUNCTION_SCOPE, i::SLOPPY },
|
2012-03-12 12:35:28 +00:00
|
|
|
// Broken 3 byte encoding of \u0fff with missing last byte.
|
|
|
|
{ " 'foo\340\277';\n"
|
|
|
|
" (function fun", "(a,b) { infunction; }", ")();",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::FUNCTION_SCOPE, i::SLOPPY },
|
2012-03-12 12:35:28 +00:00
|
|
|
// Broken 3 byte encoding of \u0fff with missing 2 last bytes.
|
|
|
|
{ " 'foo\340';\n"
|
|
|
|
" (function fun", "(a,b) { infunction; }", ")();",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::FUNCTION_SCOPE, i::SLOPPY },
|
2012-03-12 12:35:28 +00:00
|
|
|
// Broken 3 byte encoding of \u00ff should be a 2 byte encoding.
|
|
|
|
{ " 'foo\340\203\277';\n"
|
|
|
|
" (function fun", "(a,b) { infunction; }", ")();",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::FUNCTION_SCOPE, i::SLOPPY },
|
2012-03-12 12:35:28 +00:00
|
|
|
// Broken 3 byte encoding of \u007f should be a 2 byte encoding.
|
|
|
|
{ " 'foo\340\201\277';\n"
|
|
|
|
" (function fun", "(a,b) { infunction; }", ")();",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::FUNCTION_SCOPE, i::SLOPPY },
|
2012-03-12 12:35:28 +00:00
|
|
|
// Unpaired lead surrogate.
|
|
|
|
{ " 'foo\355\240\201';\n"
|
|
|
|
" (function fun", "(a,b) { infunction; }", ")();",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::FUNCTION_SCOPE, i::SLOPPY },
|
2012-03-12 12:35:28 +00:00
|
|
|
// Unpaired lead surrogate where following code point is a 3 byte sequence.
|
|
|
|
{ " 'foo\355\240\201\340\277\277';\n"
|
|
|
|
" (function fun", "(a,b) { infunction; }", ")();",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::FUNCTION_SCOPE, i::SLOPPY },
|
2012-03-12 12:35:28 +00:00
|
|
|
// Unpaired lead surrogate where following code point is a 4 byte encoding
|
|
|
|
// of a trail surrogate.
|
|
|
|
{ " 'foo\355\240\201\360\215\260\211';\n"
|
|
|
|
" (function fun", "(a,b) { infunction; }", ")();",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::FUNCTION_SCOPE, i::SLOPPY },
|
2012-03-12 12:35:28 +00:00
|
|
|
// Unpaired trail surrogate.
|
|
|
|
{ " 'foo\355\260\211';\n"
|
|
|
|
" (function fun", "(a,b) { infunction; }", ")();",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::FUNCTION_SCOPE, i::SLOPPY },
|
2012-03-12 12:35:28 +00:00
|
|
|
// 2 byte encoding of \u00ff.
|
|
|
|
{ " 'foo\303\277';\n"
|
|
|
|
" (function fun", "(a,b) { infunction; }", ")();",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::FUNCTION_SCOPE, i::SLOPPY },
|
2012-03-12 12:35:28 +00:00
|
|
|
// Broken 2 byte encoding of \u00ff with missing last byte.
|
|
|
|
{ " 'foo\303';\n"
|
|
|
|
" (function fun", "(a,b) { infunction; }", ")();",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::FUNCTION_SCOPE, i::SLOPPY },
|
2012-03-12 12:35:28 +00:00
|
|
|
// Broken 2 byte encoding of \u007f should be a 1 byte encoding.
|
|
|
|
{ " 'foo\301\277';\n"
|
|
|
|
" (function fun", "(a,b) { infunction; }", ")();",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::FUNCTION_SCOPE, i::SLOPPY },
|
2012-03-12 12:35:28 +00:00
|
|
|
// Illegal 5 byte encoding.
|
|
|
|
{ " 'foo\370\277\277\277\277';\n"
|
|
|
|
" (function fun", "(a,b) { infunction; }", ")();",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::FUNCTION_SCOPE, i::SLOPPY },
|
2012-03-12 12:35:28 +00:00
|
|
|
// Illegal 6 byte encoding.
|
|
|
|
{ " 'foo\374\277\277\277\277\277';\n"
|
|
|
|
" (function fun", "(a,b) { infunction; }", ")();",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::FUNCTION_SCOPE, i::SLOPPY },
|
2012-03-12 12:35:28 +00:00
|
|
|
// Illegal 0xfe byte
|
|
|
|
{ " 'foo\376\277\277\277\277\277\277';\n"
|
|
|
|
" (function fun", "(a,b) { infunction; }", ")();",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::FUNCTION_SCOPE, i::SLOPPY },
|
2012-03-12 12:35:28 +00:00
|
|
|
// Illegal 0xff byte
|
|
|
|
{ " 'foo\377\277\277\277\277\277\277\277';\n"
|
|
|
|
" (function fun", "(a,b) { infunction; }", ")();",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::FUNCTION_SCOPE, i::SLOPPY },
|
2012-03-12 12:35:28 +00:00
|
|
|
{ " 'foo';\n"
|
|
|
|
" (function fun", "(a,b) { 'bar\355\240\201\355\260\213'; }", ")();",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::FUNCTION_SCOPE, i::SLOPPY },
|
2012-03-12 12:35:28 +00:00
|
|
|
{ " 'foo';\n"
|
|
|
|
" (function fun", "(a,b) { 'bar\360\220\220\214'; }", ")();",
|
2014-03-11 14:41:22 +00:00
|
|
|
i::FUNCTION_SCOPE, i::SLOPPY },
|
|
|
|
{ NULL, NULL, NULL, i::EVAL_SCOPE, i::SLOPPY }
|
2011-10-21 10:26:59 +00:00
|
|
|
};
|
|
|
|
|
2013-09-19 09:17:13 +00:00
|
|
|
i::Isolate* isolate = CcTest::i_isolate();
|
2013-06-04 10:30:05 +00:00
|
|
|
i::Factory* factory = isolate->factory();
|
|
|
|
|
2013-09-19 08:54:58 +00:00
|
|
|
v8::HandleScope handles(CcTest::isolate());
|
|
|
|
v8::Handle<v8::Context> context = v8::Context::New(CcTest::isolate());
|
2011-10-21 10:26:59 +00:00
|
|
|
v8::Context::Scope context_scope(context);
|
|
|
|
|
2014-07-03 08:52:28 +00:00
|
|
|
isolate->stack_guard()->SetStackLimit(GetCurrentStackPosition() - 128 * 1024);
|
2011-10-21 10:26:59 +00:00
|
|
|
|
|
|
|
for (int i = 0; source_data[i].outer_prefix; i++) {
|
2012-03-12 12:35:28 +00:00
|
|
|
int kPrefixLen = Utf8LengthHelper(source_data[i].outer_prefix);
|
|
|
|
int kInnerLen = Utf8LengthHelper(source_data[i].inner_source);
|
|
|
|
int kSuffixLen = Utf8LengthHelper(source_data[i].outer_suffix);
|
|
|
|
int kPrefixByteLen = i::StrLength(source_data[i].outer_prefix);
|
|
|
|
int kInnerByteLen = i::StrLength(source_data[i].inner_source);
|
|
|
|
int kSuffixByteLen = i::StrLength(source_data[i].outer_suffix);
|
2011-10-21 10:26:59 +00:00
|
|
|
int kProgramSize = kPrefixLen + kInnerLen + kSuffixLen;
|
2012-03-12 12:35:28 +00:00
|
|
|
int kProgramByteSize = kPrefixByteLen + kInnerByteLen + kSuffixByteLen;
|
2014-02-06 07:16:41 +00:00
|
|
|
i::ScopedVector<char> program(kProgramByteSize + 1);
|
2014-06-13 16:43:27 +00:00
|
|
|
i::SNPrintF(program, "%s%s%s",
|
|
|
|
source_data[i].outer_prefix,
|
|
|
|
source_data[i].inner_source,
|
|
|
|
source_data[i].outer_suffix);
|
2011-10-21 10:26:59 +00:00
|
|
|
|
|
|
|
// Parse program source.
|
2014-04-17 13:27:02 +00:00
|
|
|
i::Handle<i::String> source = factory->NewStringFromUtf8(
|
|
|
|
i::CStrVector(program.start())).ToHandleChecked();
|
2012-03-12 12:35:28 +00:00
|
|
|
CHECK_EQ(source->length(), kProgramSize);
|
2013-06-04 10:30:05 +00:00
|
|
|
i::Handle<i::Script> script = factory->NewScript(source);
|
2012-06-20 08:58:41 +00:00
|
|
|
i::CompilationInfoWithZone info(script);
|
Refactor parser mode configuration for correctness
This patch refactors the parser and preparser interface to be more
readable and type-safe. It has no behavior changes.
Previously, parsers and preparsers were configured via bitfield called
parser_flags in the Parser constructor, and flags in
PreParser::PreParseProgram, ParserApi::Parse, and ParserApi::PreParse.
This was error-prone in practice: six call sites passed incorrectly
typed values to this interface (a boolean FLAG value, a boolean false
and a boolean true value). None of these errors were caught by the
compiler because it's just an "int".
The parser flags interface was also awkward because it encoded a
language mode, but the language mode was only used to turn on harmony
scoping or not -- it wasn't used to actually set the parser's language
mode.
Fundamentally these errors came in because of the desire for a
procedural parser interface, in ParserApi. Because we need to be able
to configure the parser in various ways, the flags argument got added;
but no one understood how to use the flags properly. Also they were
only used by constructors: callers packed bits, and the constructors
unpacked them into booleans on the parser or preparser.
The solution is to allow parser construction, configuration, and
invocation to be separated. This patch does that.
It passes the existing tests.
BUG=
Review URL: https://codereview.chromium.org/13450007
Patch from Andy Wingo <wingo@igalia.com>.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14151 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2013-04-05 13:01:06 +00:00
|
|
|
i::Parser parser(&info);
|
|
|
|
parser.set_allow_lazy(true);
|
|
|
|
parser.set_allow_harmony_scoping(true);
|
2014-07-21 09:58:01 +00:00
|
|
|
parser.set_allow_arrow_functions(true);
|
2011-11-15 13:48:40 +00:00
|
|
|
info.MarkAsGlobal();
|
2014-03-11 14:41:22 +00:00
|
|
|
info.SetStrictMode(source_data[i].strict_mode);
|
2013-10-01 09:27:03 +00:00
|
|
|
parser.Parse();
|
|
|
|
CHECK(info.function() != NULL);
|
2011-10-21 10:26:59 +00:00
|
|
|
|
|
|
|
// Check scope types and positions.
|
2013-10-01 09:27:03 +00:00
|
|
|
i::Scope* scope = info.function()->scope();
|
2011-10-21 10:26:59 +00:00
|
|
|
CHECK(scope->is_global_scope());
|
|
|
|
CHECK_EQ(scope->start_position(), 0);
|
|
|
|
CHECK_EQ(scope->end_position(), kProgramSize);
|
|
|
|
CHECK_EQ(scope->inner_scopes()->length(), 1);
|
|
|
|
|
|
|
|
i::Scope* inner_scope = scope->inner_scopes()->at(0);
|
2013-06-06 13:28:22 +00:00
|
|
|
CHECK_EQ(inner_scope->scope_type(), source_data[i].scope_type);
|
2011-10-21 10:26:59 +00:00
|
|
|
CHECK_EQ(inner_scope->start_position(), kPrefixLen);
|
|
|
|
// The end position of a token is one position after the last
|
|
|
|
// character belonging to that token.
|
|
|
|
CHECK_EQ(inner_scope->end_position(), kPrefixLen + kInnerLen);
|
|
|
|
}
|
|
|
|
}
|
2011-12-07 16:03:29 +00:00
|
|
|
|
|
|
|
|
2014-07-10 10:28:05 +00:00
|
|
|
const char* ReadString(unsigned* start) {
|
|
|
|
int length = start[0];
|
|
|
|
char* result = i::NewArray<char>(length + 1);
|
|
|
|
for (int i = 0; i < length; i++) {
|
|
|
|
result[i] = start[i + 1];
|
|
|
|
}
|
|
|
|
result[length] = '\0';
|
|
|
|
return result;
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
i::Handle<i::String> FormatMessage(i::Vector<unsigned> data) {
|
2013-09-19 09:17:13 +00:00
|
|
|
i::Isolate* isolate = CcTest::i_isolate();
|
2013-06-04 10:30:05 +00:00
|
|
|
i::Factory* factory = isolate->factory();
|
2014-07-10 10:28:05 +00:00
|
|
|
const char* message =
|
|
|
|
ReadString(&data[i::PreparseDataConstants::kMessageTextPos]);
|
2012-11-14 09:14:47 +00:00
|
|
|
i::Handle<i::String> format = v8::Utils::OpenHandle(
|
2013-11-22 12:43:17 +00:00
|
|
|
*v8::String::NewFromUtf8(CcTest::isolate(), message));
|
2014-07-10 10:28:05 +00:00
|
|
|
int arg_count = data[i::PreparseDataConstants::kMessageArgCountPos];
|
|
|
|
const char* arg = NULL;
|
|
|
|
i::Handle<i::JSArray> args_array;
|
|
|
|
if (arg_count == 1) {
|
|
|
|
// Position after text found by skipping past length field and
|
|
|
|
// length field content words.
|
|
|
|
int pos = i::PreparseDataConstants::kMessageTextPos + 1 +
|
|
|
|
data[i::PreparseDataConstants::kMessageTextPos];
|
|
|
|
arg = ReadString(&data[pos]);
|
|
|
|
args_array = factory->NewJSArray(1);
|
|
|
|
i::JSArray::SetElement(args_array, 0, v8::Utils::OpenHandle(*v8_str(arg)),
|
|
|
|
NONE, i::SLOPPY).Check();
|
|
|
|
} else {
|
|
|
|
CHECK_EQ(0, arg_count);
|
|
|
|
args_array = factory->NewJSArray(0);
|
2012-11-14 09:14:47 +00:00
|
|
|
}
|
2014-07-10 10:28:05 +00:00
|
|
|
|
2013-06-04 10:30:05 +00:00
|
|
|
i::Handle<i::JSObject> builtins(isolate->js_builtins_object());
|
2014-04-16 13:28:11 +00:00
|
|
|
i::Handle<i::Object> format_fun = i::Object::GetProperty(
|
|
|
|
isolate, builtins, "FormatMessage").ToHandleChecked();
|
2012-11-14 09:14:47 +00:00
|
|
|
i::Handle<i::Object> arg_handles[] = { format, args_array };
|
2013-09-05 08:48:34 +00:00
|
|
|
i::Handle<i::Object> result = i::Execution::Call(
|
2014-04-11 10:41:09 +00:00
|
|
|
isolate, format_fun, builtins, 2, arg_handles).ToHandleChecked();
|
2012-11-14 09:14:47 +00:00
|
|
|
CHECK(result->IsString());
|
2013-07-04 15:57:43 +00:00
|
|
|
i::DeleteArray(message);
|
2014-05-15 09:44:57 +00:00
|
|
|
i::DeleteArray(arg);
|
2014-07-10 10:28:05 +00:00
|
|
|
data.Dispose();
|
2012-11-14 09:14:47 +00:00
|
|
|
return i::Handle<i::String>::cast(result);
|
|
|
|
}
|
|
|
|
|
|
|
|
|
Refactor parser mode configuration for correctness
This patch refactors the parser and preparser interface to be more
readable and type-safe. It has no behavior changes.
Previously, parsers and preparsers were configured via bitfield called
parser_flags in the Parser constructor, and flags in
PreParser::PreParseProgram, ParserApi::Parse, and ParserApi::PreParse.
This was error-prone in practice: six call sites passed incorrectly
typed values to this interface (a boolean FLAG value, a boolean false
and a boolean true value). None of these errors were caught by the
compiler because it's just an "int".
The parser flags interface was also awkward because it encoded a
language mode, but the language mode was only used to turn on harmony
scoping or not -- it wasn't used to actually set the parser's language
mode.
Fundamentally these errors came in because of the desire for a
procedural parser interface, in ParserApi. Because we need to be able
to configure the parser in various ways, the flags argument got added;
but no one understood how to use the flags properly. Also they were
only used by constructors: callers packed bits, and the constructors
unpacked them into booleans on the parser or preparser.
The solution is to allow parser construction, configuration, and
invocation to be separated. This patch does that.
It passes the existing tests.
BUG=
Review URL: https://codereview.chromium.org/13450007
Patch from Andy Wingo <wingo@igalia.com>.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14151 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2013-04-05 13:01:06 +00:00
|
|
|
enum ParserFlag {
|
|
|
|
kAllowLazy,
|
|
|
|
kAllowNativesSyntax,
|
|
|
|
kAllowHarmonyScoping,
|
|
|
|
kAllowModules,
|
|
|
|
kAllowGenerators,
|
2013-06-06 14:38:26 +00:00
|
|
|
kAllowForOf,
|
Implement handling of arrow functions in the parser
Arrow functions are parsed from ParseAssignmentExpression(). Handling the
parameter list is done by letting ParseConditionalExpression() parse a comma
separated list of identifiers, and it returns a tree of BinaryOperation nodes
with VariableProxy leaves, or a single VariableProxy if there is only one
parameter. When the arrow token "=>" is found, the VariableProxy nodes are
passed to ParseArrowFunctionLiteral(), which will then skip parsing the
paramaeter list. This avoids having to rewind when the arrow is found and
restart parsing the parameter list.
Note that the empty parameter list "()" is handled directly in
ParsePrimaryExpression(): after is has consumed the opening parenthesis,
if a closing parenthesis follows, then the only valid input is an arrow
function. In this case, ParsePrimaryExpression() directly calls
ParseArrowFunctionLiteral(), to avoid needing to return a sentinel value
to signal the empty parameter list. Because it will consume the body of
the arrow function, ParseAssignmentExpression() will not see the arrow
"=>" token as next, and return the already-parser expression.
The implementation is done in ParserBase, so it was needed to do some
additions to ParserBase, ParserTraits and PreParserTraits. Some of the
glue code can be removed later on when more more functionality is moved
to ParserBase.
Additionally, this adds a runtime flag "harmony_arrow_functions"
(disabled by default); enabling "harmony" will enable it as well.
BUG=v8:2700
LOG=N
R=marja@chromium.org
Review URL: https://codereview.chromium.org/383983002
Patch from Adrián Pérez de Castro <aperez@igalia.com>.
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22366 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-14 07:55:45 +00:00
|
|
|
kAllowHarmonyNumericLiterals,
|
|
|
|
kAllowArrowFunctions
|
Refactor parser mode configuration for correctness
This patch refactors the parser and preparser interface to be more
readable and type-safe. It has no behavior changes.
Previously, parsers and preparsers were configured via bitfield called
parser_flags in the Parser constructor, and flags in
PreParser::PreParseProgram, ParserApi::Parse, and ParserApi::PreParse.
This was error-prone in practice: six call sites passed incorrectly
typed values to this interface (a boolean FLAG value, a boolean false
and a boolean true value). None of these errors were caught by the
compiler because it's just an "int".
The parser flags interface was also awkward because it encoded a
language mode, but the language mode was only used to turn on harmony
scoping or not -- it wasn't used to actually set the parser's language
mode.
Fundamentally these errors came in because of the desire for a
procedural parser interface, in ParserApi. Because we need to be able
to configure the parser in various ways, the flags argument got added;
but no one understood how to use the flags properly. Also they were
only used by constructors: callers packed bits, and the constructors
unpacked them into booleans on the parser or preparser.
The solution is to allow parser construction, configuration, and
invocation to be separated. This patch does that.
It passes the existing tests.
BUG=
Review URL: https://codereview.chromium.org/13450007
Patch from Andy Wingo <wingo@igalia.com>.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14151 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2013-04-05 13:01:06 +00:00
|
|
|
};
|
|
|
|
|
|
|
|
|
2014-02-05 15:30:01 +00:00
|
|
|
enum ParserSyncTestResult {
|
|
|
|
kSuccessOrError,
|
|
|
|
kSuccess,
|
|
|
|
kError
|
|
|
|
};
|
|
|
|
|
2014-02-11 09:35:32 +00:00
|
|
|
template <typename Traits>
|
|
|
|
void SetParserFlags(i::ParserBase<Traits>* parser,
|
|
|
|
i::EnumSet<ParserFlag> flags) {
|
2013-10-25 07:24:08 +00:00
|
|
|
parser->set_allow_lazy(flags.Contains(kAllowLazy));
|
|
|
|
parser->set_allow_natives_syntax(flags.Contains(kAllowNativesSyntax));
|
|
|
|
parser->set_allow_harmony_scoping(flags.Contains(kAllowHarmonyScoping));
|
|
|
|
parser->set_allow_modules(flags.Contains(kAllowModules));
|
|
|
|
parser->set_allow_generators(flags.Contains(kAllowGenerators));
|
|
|
|
parser->set_allow_for_of(flags.Contains(kAllowForOf));
|
|
|
|
parser->set_allow_harmony_numeric_literals(
|
|
|
|
flags.Contains(kAllowHarmonyNumericLiterals));
|
Implement handling of arrow functions in the parser
Arrow functions are parsed from ParseAssignmentExpression(). Handling the
parameter list is done by letting ParseConditionalExpression() parse a comma
separated list of identifiers, and it returns a tree of BinaryOperation nodes
with VariableProxy leaves, or a single VariableProxy if there is only one
parameter. When the arrow token "=>" is found, the VariableProxy nodes are
passed to ParseArrowFunctionLiteral(), which will then skip parsing the
paramaeter list. This avoids having to rewind when the arrow is found and
restart parsing the parameter list.
Note that the empty parameter list "()" is handled directly in
ParsePrimaryExpression(): after is has consumed the opening parenthesis,
if a closing parenthesis follows, then the only valid input is an arrow
function. In this case, ParsePrimaryExpression() directly calls
ParseArrowFunctionLiteral(), to avoid needing to return a sentinel value
to signal the empty parameter list. Because it will consume the body of
the arrow function, ParseAssignmentExpression() will not see the arrow
"=>" token as next, and return the already-parser expression.
The implementation is done in ParserBase, so it was needed to do some
additions to ParserBase, ParserTraits and PreParserTraits. Some of the
glue code can be removed later on when more more functionality is moved
to ParserBase.
Additionally, this adds a runtime flag "harmony_arrow_functions"
(disabled by default); enabling "harmony" will enable it as well.
BUG=v8:2700
LOG=N
R=marja@chromium.org
Review URL: https://codereview.chromium.org/383983002
Patch from Adrián Pérez de Castro <aperez@igalia.com>.
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22366 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-14 07:55:45 +00:00
|
|
|
parser->set_allow_arrow_functions(flags.Contains(kAllowArrowFunctions));
|
Refactor parser mode configuration for correctness
This patch refactors the parser and preparser interface to be more
readable and type-safe. It has no behavior changes.
Previously, parsers and preparsers were configured via bitfield called
parser_flags in the Parser constructor, and flags in
PreParser::PreParseProgram, ParserApi::Parse, and ParserApi::PreParse.
This was error-prone in practice: six call sites passed incorrectly
typed values to this interface (a boolean FLAG value, a boolean false
and a boolean true value). None of these errors were caught by the
compiler because it's just an "int".
The parser flags interface was also awkward because it encoded a
language mode, but the language mode was only used to turn on harmony
scoping or not -- it wasn't used to actually set the parser's language
mode.
Fundamentally these errors came in because of the desire for a
procedural parser interface, in ParserApi. Because we need to be able
to configure the parser in various ways, the flags argument got added;
but no one understood how to use the flags properly. Also they were
only used by constructors: callers packed bits, and the constructors
unpacked them into booleans on the parser or preparser.
The solution is to allow parser construction, configuration, and
invocation to be separated. This patch does that.
It passes the existing tests.
BUG=
Review URL: https://codereview.chromium.org/13450007
Patch from Andy Wingo <wingo@igalia.com>.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14151 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2013-04-05 13:01:06 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
|
2013-10-25 07:24:08 +00:00
|
|
|
void TestParserSyncWithFlags(i::Handle<i::String> source,
|
2014-02-05 15:30:01 +00:00
|
|
|
i::EnumSet<ParserFlag> flags,
|
|
|
|
ParserSyncTestResult result) {
|
2013-09-19 09:17:13 +00:00
|
|
|
i::Isolate* isolate = CcTest::i_isolate();
|
2013-06-04 10:30:05 +00:00
|
|
|
i::Factory* factory = isolate->factory();
|
|
|
|
|
|
|
|
uintptr_t stack_limit = isolate->stack_guard()->real_climit();
|
2011-12-07 16:03:29 +00:00
|
|
|
|
|
|
|
// Preparse the data.
|
|
|
|
i::CompleteParserRecorder log;
|
Refactor parser mode configuration for correctness
This patch refactors the parser and preparser interface to be more
readable and type-safe. It has no behavior changes.
Previously, parsers and preparsers were configured via bitfield called
parser_flags in the Parser constructor, and flags in
PreParser::PreParseProgram, ParserApi::Parse, and ParserApi::PreParse.
This was error-prone in practice: six call sites passed incorrectly
typed values to this interface (a boolean FLAG value, a boolean false
and a boolean true value). None of these errors were caught by the
compiler because it's just an "int".
The parser flags interface was also awkward because it encoded a
language mode, but the language mode was only used to turn on harmony
scoping or not -- it wasn't used to actually set the parser's language
mode.
Fundamentally these errors came in because of the desire for a
procedural parser interface, in ParserApi. Because we need to be able
to configure the parser in various ways, the flags argument got added;
but no one understood how to use the flags properly. Also they were
only used by constructors: callers packed bits, and the constructors
unpacked them into booleans on the parser or preparser.
The solution is to allow parser construction, configuration, and
invocation to be separated. This patch does that.
It passes the existing tests.
BUG=
Review URL: https://codereview.chromium.org/13450007
Patch from Andy Wingo <wingo@igalia.com>.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14151 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2013-04-05 13:01:06 +00:00
|
|
|
{
|
2013-06-04 10:30:05 +00:00
|
|
|
i::Scanner scanner(isolate->unicode_cache());
|
Refactor parser mode configuration for correctness
This patch refactors the parser and preparser interface to be more
readable and type-safe. It has no behavior changes.
Previously, parsers and preparsers were configured via bitfield called
parser_flags in the Parser constructor, and flags in
PreParser::PreParseProgram, ParserApi::Parse, and ParserApi::PreParse.
This was error-prone in practice: six call sites passed incorrectly
typed values to this interface (a boolean FLAG value, a boolean false
and a boolean true value). None of these errors were caught by the
compiler because it's just an "int".
The parser flags interface was also awkward because it encoded a
language mode, but the language mode was only used to turn on harmony
scoping or not -- it wasn't used to actually set the parser's language
mode.
Fundamentally these errors came in because of the desire for a
procedural parser interface, in ParserApi. Because we need to be able
to configure the parser in various ways, the flags argument got added;
but no one understood how to use the flags properly. Also they were
only used by constructors: callers packed bits, and the constructors
unpacked them into booleans on the parser or preparser.
The solution is to allow parser construction, configuration, and
invocation to be separated. This patch does that.
It passes the existing tests.
BUG=
Review URL: https://codereview.chromium.org/13450007
Patch from Andy Wingo <wingo@igalia.com>.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14151 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2013-04-05 13:01:06 +00:00
|
|
|
i::GenericStringUtf16CharacterStream stream(source, 0, source->length());
|
2013-10-14 13:07:20 +00:00
|
|
|
i::PreParser preparser(&scanner, &log, stack_limit);
|
2013-10-25 07:24:08 +00:00
|
|
|
SetParserFlags(&preparser, flags);
|
Refactor parser mode configuration for correctness
This patch refactors the parser and preparser interface to be more
readable and type-safe. It has no behavior changes.
Previously, parsers and preparsers were configured via bitfield called
parser_flags in the Parser constructor, and flags in
PreParser::PreParseProgram, ParserApi::Parse, and ParserApi::PreParse.
This was error-prone in practice: six call sites passed incorrectly
typed values to this interface (a boolean FLAG value, a boolean false
and a boolean true value). None of these errors were caught by the
compiler because it's just an "int".
The parser flags interface was also awkward because it encoded a
language mode, but the language mode was only used to turn on harmony
scoping or not -- it wasn't used to actually set the parser's language
mode.
Fundamentally these errors came in because of the desire for a
procedural parser interface, in ParserApi. Because we need to be able
to configure the parser in various ways, the flags argument got added;
but no one understood how to use the flags properly. Also they were
only used by constructors: callers packed bits, and the constructors
unpacked them into booleans on the parser or preparser.
The solution is to allow parser construction, configuration, and
invocation to be separated. This patch does that.
It passes the existing tests.
BUG=
Review URL: https://codereview.chromium.org/13450007
Patch from Andy Wingo <wingo@igalia.com>.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14151 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2013-04-05 13:01:06 +00:00
|
|
|
scanner.Initialize(&stream);
|
2013-10-14 13:07:20 +00:00
|
|
|
i::PreParser::PreParseResult result = preparser.PreParseProgram();
|
|
|
|
CHECK_EQ(i::PreParser::kPreParseSuccess, result);
|
Refactor parser mode configuration for correctness
This patch refactors the parser and preparser interface to be more
readable and type-safe. It has no behavior changes.
Previously, parsers and preparsers were configured via bitfield called
parser_flags in the Parser constructor, and flags in
PreParser::PreParseProgram, ParserApi::Parse, and ParserApi::PreParse.
This was error-prone in practice: six call sites passed incorrectly
typed values to this interface (a boolean FLAG value, a boolean false
and a boolean true value). None of these errors were caught by the
compiler because it's just an "int".
The parser flags interface was also awkward because it encoded a
language mode, but the language mode was only used to turn on harmony
scoping or not -- it wasn't used to actually set the parser's language
mode.
Fundamentally these errors came in because of the desire for a
procedural parser interface, in ParserApi. Because we need to be able
to configure the parser in various ways, the flags argument got added;
but no one understood how to use the flags properly. Also they were
only used by constructors: callers packed bits, and the constructors
unpacked them into booleans on the parser or preparser.
The solution is to allow parser construction, configuration, and
invocation to be separated. This patch does that.
It passes the existing tests.
BUG=
Review URL: https://codereview.chromium.org/13450007
Patch from Andy Wingo <wingo@igalia.com>.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14151 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2013-04-05 13:01:06 +00:00
|
|
|
}
|
2014-07-10 10:28:05 +00:00
|
|
|
|
|
|
|
bool preparse_error = log.HasError();
|
2011-12-07 16:03:29 +00:00
|
|
|
|
|
|
|
// Parse the data
|
Refactor parser mode configuration for correctness
This patch refactors the parser and preparser interface to be more
readable and type-safe. It has no behavior changes.
Previously, parsers and preparsers were configured via bitfield called
parser_flags in the Parser constructor, and flags in
PreParser::PreParseProgram, ParserApi::Parse, and ParserApi::PreParse.
This was error-prone in practice: six call sites passed incorrectly
typed values to this interface (a boolean FLAG value, a boolean false
and a boolean true value). None of these errors were caught by the
compiler because it's just an "int".
The parser flags interface was also awkward because it encoded a
language mode, but the language mode was only used to turn on harmony
scoping or not -- it wasn't used to actually set the parser's language
mode.
Fundamentally these errors came in because of the desire for a
procedural parser interface, in ParserApi. Because we need to be able
to configure the parser in various ways, the flags argument got added;
but no one understood how to use the flags properly. Also they were
only used by constructors: callers packed bits, and the constructors
unpacked them into booleans on the parser or preparser.
The solution is to allow parser construction, configuration, and
invocation to be separated. This patch does that.
It passes the existing tests.
BUG=
Review URL: https://codereview.chromium.org/13450007
Patch from Andy Wingo <wingo@igalia.com>.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14151 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2013-04-05 13:01:06 +00:00
|
|
|
i::FunctionLiteral* function;
|
|
|
|
{
|
2013-06-04 10:30:05 +00:00
|
|
|
i::Handle<i::Script> script = factory->NewScript(source);
|
Refactor parser mode configuration for correctness
This patch refactors the parser and preparser interface to be more
readable and type-safe. It has no behavior changes.
Previously, parsers and preparsers were configured via bitfield called
parser_flags in the Parser constructor, and flags in
PreParser::PreParseProgram, ParserApi::Parse, and ParserApi::PreParse.
This was error-prone in practice: six call sites passed incorrectly
typed values to this interface (a boolean FLAG value, a boolean false
and a boolean true value). None of these errors were caught by the
compiler because it's just an "int".
The parser flags interface was also awkward because it encoded a
language mode, but the language mode was only used to turn on harmony
scoping or not -- it wasn't used to actually set the parser's language
mode.
Fundamentally these errors came in because of the desire for a
procedural parser interface, in ParserApi. Because we need to be able
to configure the parser in various ways, the flags argument got added;
but no one understood how to use the flags properly. Also they were
only used by constructors: callers packed bits, and the constructors
unpacked them into booleans on the parser or preparser.
The solution is to allow parser construction, configuration, and
invocation to be separated. This patch does that.
It passes the existing tests.
BUG=
Review URL: https://codereview.chromium.org/13450007
Patch from Andy Wingo <wingo@igalia.com>.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14151 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2013-04-05 13:01:06 +00:00
|
|
|
i::CompilationInfoWithZone info(script);
|
|
|
|
i::Parser parser(&info);
|
2013-10-25 07:24:08 +00:00
|
|
|
SetParserFlags(&parser, flags);
|
Refactor parser mode configuration for correctness
This patch refactors the parser and preparser interface to be more
readable and type-safe. It has no behavior changes.
Previously, parsers and preparsers were configured via bitfield called
parser_flags in the Parser constructor, and flags in
PreParser::PreParseProgram, ParserApi::Parse, and ParserApi::PreParse.
This was error-prone in practice: six call sites passed incorrectly
typed values to this interface (a boolean FLAG value, a boolean false
and a boolean true value). None of these errors were caught by the
compiler because it's just an "int".
The parser flags interface was also awkward because it encoded a
language mode, but the language mode was only used to turn on harmony
scoping or not -- it wasn't used to actually set the parser's language
mode.
Fundamentally these errors came in because of the desire for a
procedural parser interface, in ParserApi. Because we need to be able
to configure the parser in various ways, the flags argument got added;
but no one understood how to use the flags properly. Also they were
only used by constructors: callers packed bits, and the constructors
unpacked them into booleans on the parser or preparser.
The solution is to allow parser construction, configuration, and
invocation to be separated. This patch does that.
It passes the existing tests.
BUG=
Review URL: https://codereview.chromium.org/13450007
Patch from Andy Wingo <wingo@igalia.com>.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14151 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2013-04-05 13:01:06 +00:00
|
|
|
info.MarkAsGlobal();
|
2013-10-01 09:27:03 +00:00
|
|
|
parser.Parse();
|
|
|
|
function = info.function();
|
Refactor parser mode configuration for correctness
This patch refactors the parser and preparser interface to be more
readable and type-safe. It has no behavior changes.
Previously, parsers and preparsers were configured via bitfield called
parser_flags in the Parser constructor, and flags in
PreParser::PreParseProgram, ParserApi::Parse, and ParserApi::PreParse.
This was error-prone in practice: six call sites passed incorrectly
typed values to this interface (a boolean FLAG value, a boolean false
and a boolean true value). None of these errors were caught by the
compiler because it's just an "int".
The parser flags interface was also awkward because it encoded a
language mode, but the language mode was only used to turn on harmony
scoping or not -- it wasn't used to actually set the parser's language
mode.
Fundamentally these errors came in because of the desire for a
procedural parser interface, in ParserApi. Because we need to be able
to configure the parser in various ways, the flags argument got added;
but no one understood how to use the flags properly. Also they were
only used by constructors: callers packed bits, and the constructors
unpacked them into booleans on the parser or preparser.
The solution is to allow parser construction, configuration, and
invocation to be separated. This patch does that.
It passes the existing tests.
BUG=
Review URL: https://codereview.chromium.org/13450007
Patch from Andy Wingo <wingo@igalia.com>.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14151 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2013-04-05 13:01:06 +00:00
|
|
|
}
|
2011-12-07 16:03:29 +00:00
|
|
|
|
2012-11-14 09:14:47 +00:00
|
|
|
// Check that preparsing fails iff parsing fails.
|
2011-12-07 16:03:29 +00:00
|
|
|
if (function == NULL) {
|
|
|
|
// Extract exception from the parser.
|
2013-06-04 10:30:05 +00:00
|
|
|
CHECK(isolate->has_pending_exception());
|
2014-04-08 09:44:24 +00:00
|
|
|
i::Handle<i::JSObject> exception_handle(
|
|
|
|
i::JSObject::cast(isolate->pending_exception()));
|
2012-11-14 09:14:47 +00:00
|
|
|
i::Handle<i::String> message_string =
|
2014-04-16 13:28:11 +00:00
|
|
|
i::Handle<i::String>::cast(i::Object::GetProperty(
|
|
|
|
isolate, exception_handle, "message").ToHandleChecked());
|
2011-12-07 16:03:29 +00:00
|
|
|
|
2014-02-05 15:30:01 +00:00
|
|
|
if (result == kSuccess) {
|
2014-06-30 13:25:46 +00:00
|
|
|
v8::base::OS::Print(
|
2014-02-05 15:30:01 +00:00
|
|
|
"Parser failed on:\n"
|
|
|
|
"\t%s\n"
|
|
|
|
"with error:\n"
|
|
|
|
"\t%s\n"
|
|
|
|
"However, we expected no error.",
|
|
|
|
source->ToCString().get(), message_string->ToCString().get());
|
|
|
|
CHECK(false);
|
|
|
|
}
|
|
|
|
|
2014-07-10 10:28:05 +00:00
|
|
|
if (!preparse_error) {
|
2014-06-30 13:25:46 +00:00
|
|
|
v8::base::OS::Print(
|
2012-11-14 09:14:47 +00:00
|
|
|
"Parser failed on:\n"
|
|
|
|
"\t%s\n"
|
|
|
|
"with error:\n"
|
|
|
|
"\t%s\n"
|
|
|
|
"However, the preparser succeeded",
|
2013-12-09 07:41:20 +00:00
|
|
|
source->ToCString().get(), message_string->ToCString().get());
|
2012-11-14 09:14:47 +00:00
|
|
|
CHECK(false);
|
|
|
|
}
|
|
|
|
// Check that preparser and parser produce the same error.
|
2014-07-10 10:28:05 +00:00
|
|
|
i::Handle<i::String> preparser_message =
|
|
|
|
FormatMessage(log.ErrorMessageData());
|
2014-04-11 07:27:25 +00:00
|
|
|
if (!i::String::Equals(message_string, preparser_message)) {
|
2014-06-30 13:25:46 +00:00
|
|
|
v8::base::OS::Print(
|
2011-12-07 16:03:29 +00:00
|
|
|
"Expected parser and preparser to produce the same error on:\n"
|
|
|
|
"\t%s\n"
|
|
|
|
"However, found the following error messages\n"
|
|
|
|
"\tparser: %s\n"
|
|
|
|
"\tpreparser: %s\n",
|
2013-12-09 07:41:20 +00:00
|
|
|
source->ToCString().get(),
|
|
|
|
message_string->ToCString().get(),
|
|
|
|
preparser_message->ToCString().get());
|
2011-12-07 16:03:29 +00:00
|
|
|
CHECK(false);
|
|
|
|
}
|
2014-07-10 10:28:05 +00:00
|
|
|
} else if (preparse_error) {
|
2014-06-30 13:25:46 +00:00
|
|
|
v8::base::OS::Print(
|
2012-11-14 09:14:47 +00:00
|
|
|
"Preparser failed on:\n"
|
|
|
|
"\t%s\n"
|
|
|
|
"with error:\n"
|
|
|
|
"\t%s\n"
|
|
|
|
"However, the parser succeeded",
|
2014-07-10 10:48:28 +00:00
|
|
|
source->ToCString().get(),
|
|
|
|
FormatMessage(log.ErrorMessageData())->ToCString().get());
|
2012-11-14 09:14:47 +00:00
|
|
|
CHECK(false);
|
2014-02-05 15:30:01 +00:00
|
|
|
} else if (result == kError) {
|
2014-06-30 13:25:46 +00:00
|
|
|
v8::base::OS::Print(
|
2014-02-05 15:30:01 +00:00
|
|
|
"Expected error on:\n"
|
|
|
|
"\t%s\n"
|
|
|
|
"However, parser and preparser succeeded",
|
|
|
|
source->ToCString().get());
|
|
|
|
CHECK(false);
|
2011-12-07 16:03:29 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
|
2013-10-25 07:24:08 +00:00
|
|
|
void TestParserSync(const char* source,
|
2014-06-20 12:28:13 +00:00
|
|
|
const ParserFlag* varying_flags,
|
|
|
|
size_t varying_flags_length,
|
|
|
|
ParserSyncTestResult result = kSuccessOrError,
|
|
|
|
const ParserFlag* always_true_flags = NULL,
|
|
|
|
size_t always_true_flags_length = 0) {
|
2013-10-25 07:24:08 +00:00
|
|
|
i::Handle<i::String> str =
|
2014-04-17 13:27:02 +00:00
|
|
|
CcTest::i_isolate()->factory()->NewStringFromAsciiChecked(source);
|
2014-06-20 12:28:13 +00:00
|
|
|
for (int bits = 0; bits < (1 << varying_flags_length); bits++) {
|
2013-10-25 07:24:08 +00:00
|
|
|
i::EnumSet<ParserFlag> flags;
|
2014-06-20 12:28:13 +00:00
|
|
|
for (size_t flag_index = 0; flag_index < varying_flags_length;
|
|
|
|
++flag_index) {
|
|
|
|
if ((bits & (1 << flag_index)) != 0) flags.Add(varying_flags[flag_index]);
|
|
|
|
}
|
|
|
|
for (size_t flag_index = 0; flag_index < always_true_flags_length;
|
|
|
|
++flag_index) {
|
|
|
|
flags.Add(always_true_flags[flag_index]);
|
2013-10-25 07:24:08 +00:00
|
|
|
}
|
2014-02-05 15:30:01 +00:00
|
|
|
TestParserSyncWithFlags(str, flags, result);
|
2011-12-07 16:03:29 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
TEST(ParserSync) {
|
|
|
|
const char* context_data[][2] = {
|
|
|
|
{ "", "" },
|
|
|
|
{ "{", "}" },
|
|
|
|
{ "if (true) ", " else {}" },
|
|
|
|
{ "if (true) {} else ", "" },
|
|
|
|
{ "if (true) ", "" },
|
|
|
|
{ "do ", " while (false)" },
|
|
|
|
{ "while (false) ", "" },
|
|
|
|
{ "for (;;) ", "" },
|
|
|
|
{ "with ({})", "" },
|
|
|
|
{ "switch (12) { case 12: ", "}" },
|
|
|
|
{ "switch (12) { default: ", "}" },
|
2012-07-11 07:47:29 +00:00
|
|
|
{ "switch (12) { ", "case 12: }" },
|
2011-12-07 16:03:29 +00:00
|
|
|
{ "label2: ", "" },
|
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
|
|
|
|
|
|
|
const char* statement_data[] = {
|
|
|
|
"{}",
|
|
|
|
"var x",
|
|
|
|
"var x = 1",
|
|
|
|
"const x",
|
|
|
|
"const x = 1",
|
|
|
|
";",
|
|
|
|
"12",
|
|
|
|
"if (false) {} else ;",
|
|
|
|
"if (false) {} else {}",
|
|
|
|
"if (false) {} else 12",
|
|
|
|
"if (false) ;"
|
|
|
|
"if (false) {}",
|
|
|
|
"if (false) 12",
|
|
|
|
"do {} while (false)",
|
|
|
|
"for (;;) ;",
|
|
|
|
"for (;;) {}",
|
|
|
|
"for (;;) 12",
|
|
|
|
"continue",
|
|
|
|
"continue label",
|
|
|
|
"continue\nlabel",
|
|
|
|
"break",
|
|
|
|
"break label",
|
|
|
|
"break\nlabel",
|
2014-04-02 12:38:01 +00:00
|
|
|
// TODO(marja): activate once parsing 'return' is merged into ParserBase.
|
|
|
|
// "return",
|
|
|
|
// "return 12",
|
|
|
|
// "return\n12",
|
2011-12-07 16:03:29 +00:00
|
|
|
"with ({}) ;",
|
|
|
|
"with ({}) {}",
|
|
|
|
"with ({}) 12",
|
|
|
|
"switch ({}) { default: }"
|
|
|
|
"label3: "
|
|
|
|
"throw",
|
|
|
|
"throw 12",
|
|
|
|
"throw\n12",
|
|
|
|
"try {} catch(e) {}",
|
|
|
|
"try {} finally {}",
|
|
|
|
"try {} catch(e) {} finally {}",
|
|
|
|
"debugger",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
const char* termination_data[] = {
|
|
|
|
"",
|
|
|
|
";",
|
|
|
|
"\n",
|
|
|
|
";\n",
|
|
|
|
"\n;",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
2013-09-19 08:54:58 +00:00
|
|
|
v8::HandleScope handles(CcTest::isolate());
|
|
|
|
v8::Handle<v8::Context> context = v8::Context::New(CcTest::isolate());
|
2011-12-07 16:03:29 +00:00
|
|
|
v8::Context::Scope context_scope(context);
|
|
|
|
|
2014-07-03 08:52:28 +00:00
|
|
|
CcTest::i_isolate()->stack_guard()->SetStackLimit(GetCurrentStackPosition() -
|
|
|
|
128 * 1024);
|
2011-12-07 16:03:29 +00:00
|
|
|
|
2014-07-08 07:48:22 +00:00
|
|
|
static const ParserFlag flags1[] = {
|
|
|
|
kAllowLazy, kAllowHarmonyScoping, kAllowModules, kAllowGenerators,
|
Implement handling of arrow functions in the parser
Arrow functions are parsed from ParseAssignmentExpression(). Handling the
parameter list is done by letting ParseConditionalExpression() parse a comma
separated list of identifiers, and it returns a tree of BinaryOperation nodes
with VariableProxy leaves, or a single VariableProxy if there is only one
parameter. When the arrow token "=>" is found, the VariableProxy nodes are
passed to ParseArrowFunctionLiteral(), which will then skip parsing the
paramaeter list. This avoids having to rewind when the arrow is found and
restart parsing the parameter list.
Note that the empty parameter list "()" is handled directly in
ParsePrimaryExpression(): after is has consumed the opening parenthesis,
if a closing parenthesis follows, then the only valid input is an arrow
function. In this case, ParsePrimaryExpression() directly calls
ParseArrowFunctionLiteral(), to avoid needing to return a sentinel value
to signal the empty parameter list. Because it will consume the body of
the arrow function, ParseAssignmentExpression() will not see the arrow
"=>" token as next, and return the already-parser expression.
The implementation is done in ParserBase, so it was needed to do some
additions to ParserBase, ParserTraits and PreParserTraits. Some of the
glue code can be removed later on when more more functionality is moved
to ParserBase.
Additionally, this adds a runtime flag "harmony_arrow_functions"
(disabled by default); enabling "harmony" will enable it as well.
BUG=v8:2700
LOG=N
R=marja@chromium.org
Review URL: https://codereview.chromium.org/383983002
Patch from Adrián Pérez de Castro <aperez@igalia.com>.
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22366 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-14 07:55:45 +00:00
|
|
|
kAllowForOf, kAllowArrowFunctions
|
2014-07-08 07:48:22 +00:00
|
|
|
};
|
2011-12-07 16:03:29 +00:00
|
|
|
for (int i = 0; context_data[i][0] != NULL; ++i) {
|
|
|
|
for (int j = 0; statement_data[j] != NULL; ++j) {
|
|
|
|
for (int k = 0; termination_data[k] != NULL; ++k) {
|
|
|
|
int kPrefixLen = i::StrLength(context_data[i][0]);
|
|
|
|
int kStatementLen = i::StrLength(statement_data[j]);
|
|
|
|
int kTerminationLen = i::StrLength(termination_data[k]);
|
|
|
|
int kSuffixLen = i::StrLength(context_data[i][1]);
|
|
|
|
int kProgramSize = kPrefixLen + kStatementLen + kTerminationLen
|
|
|
|
+ kSuffixLen + i::StrLength("label: for (;;) { }");
|
|
|
|
|
|
|
|
// Plug the source code pieces together.
|
2013-07-04 15:57:43 +00:00
|
|
|
i::ScopedVector<char> program(kProgramSize + 1);
|
2014-06-13 16:43:27 +00:00
|
|
|
int length = i::SNPrintF(program,
|
2011-12-07 16:03:29 +00:00
|
|
|
"label: for (;;) { %s%s%s%s }",
|
|
|
|
context_data[i][0],
|
|
|
|
statement_data[j],
|
|
|
|
termination_data[k],
|
|
|
|
context_data[i][1]);
|
|
|
|
CHECK(length == kProgramSize);
|
2013-10-25 07:24:08 +00:00
|
|
|
TestParserSync(program.start(), flags1, ARRAY_SIZE(flags1));
|
2011-12-07 16:03:29 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
2013-10-25 07:24:08 +00:00
|
|
|
|
|
|
|
// Neither Harmony numeric literals nor our natives syntax have any
|
|
|
|
// interaction with the flags above, so test these separately to reduce
|
|
|
|
// the combinatorial explosion.
|
|
|
|
static const ParserFlag flags2[] = { kAllowHarmonyNumericLiterals };
|
|
|
|
TestParserSync("0o1234", flags2, ARRAY_SIZE(flags2));
|
|
|
|
TestParserSync("0b1011", flags2, ARRAY_SIZE(flags2));
|
|
|
|
|
|
|
|
static const ParserFlag flags3[] = { kAllowNativesSyntax };
|
|
|
|
TestParserSync("%DebugPrint(123)", flags3, ARRAY_SIZE(flags3));
|
2011-12-07 16:03:29 +00:00
|
|
|
}
|
2012-07-10 12:24:17 +00:00
|
|
|
|
|
|
|
|
2014-04-04 12:36:23 +00:00
|
|
|
TEST(StrictOctal) {
|
2012-07-10 12:24:17 +00:00
|
|
|
// Test that syntax error caused by octal literal is reported correctly as
|
|
|
|
// such (issue 2220).
|
|
|
|
v8::V8::Initialize();
|
2013-09-19 08:54:58 +00:00
|
|
|
v8::HandleScope scope(CcTest::isolate());
|
2013-05-02 20:18:42 +00:00
|
|
|
v8::Context::Scope context_scope(
|
2013-09-19 08:54:58 +00:00
|
|
|
v8::Context::New(CcTest::isolate()));
|
2012-07-10 12:24:17 +00:00
|
|
|
v8::TryCatch try_catch;
|
|
|
|
const char* script =
|
|
|
|
"\"use strict\"; \n"
|
|
|
|
"a = function() { \n"
|
|
|
|
" b = function() { \n"
|
|
|
|
" 01; \n"
|
|
|
|
" }; \n"
|
|
|
|
"}; \n";
|
2013-11-22 12:43:17 +00:00
|
|
|
v8::Script::Compile(v8::String::NewFromUtf8(CcTest::isolate(), script));
|
2012-07-10 12:24:17 +00:00
|
|
|
CHECK(try_catch.HasCaught());
|
|
|
|
v8::String::Utf8Value exception(try_catch.Exception());
|
|
|
|
CHECK_EQ("SyntaxError: Octal literals are not allowed in strict mode.",
|
|
|
|
*exception);
|
|
|
|
}
|
2014-02-04 11:26:19 +00:00
|
|
|
|
|
|
|
|
2014-02-05 15:30:01 +00:00
|
|
|
void RunParserSyncTest(const char* context_data[][2],
|
|
|
|
const char* statement_data[],
|
2014-03-11 16:30:47 +00:00
|
|
|
ParserSyncTestResult result,
|
|
|
|
const ParserFlag* flags = NULL,
|
2014-06-20 12:28:13 +00:00
|
|
|
int flags_len = 0,
|
|
|
|
const ParserFlag* always_true_flags = NULL,
|
|
|
|
int always_true_flags_len = 0) {
|
2014-02-05 15:30:01 +00:00
|
|
|
v8::HandleScope handles(CcTest::isolate());
|
|
|
|
v8::Handle<v8::Context> context = v8::Context::New(CcTest::isolate());
|
|
|
|
v8::Context::Scope context_scope(context);
|
|
|
|
|
2014-07-03 08:52:28 +00:00
|
|
|
CcTest::i_isolate()->stack_guard()->SetStackLimit(GetCurrentStackPosition() -
|
|
|
|
128 * 1024);
|
2014-02-05 15:30:01 +00:00
|
|
|
|
2014-03-11 16:30:47 +00:00
|
|
|
static const ParserFlag default_flags[] = {
|
2014-07-08 07:48:22 +00:00
|
|
|
kAllowLazy, kAllowHarmonyScoping, kAllowModules, kAllowGenerators,
|
Implement handling of arrow functions in the parser
Arrow functions are parsed from ParseAssignmentExpression(). Handling the
parameter list is done by letting ParseConditionalExpression() parse a comma
separated list of identifiers, and it returns a tree of BinaryOperation nodes
with VariableProxy leaves, or a single VariableProxy if there is only one
parameter. When the arrow token "=>" is found, the VariableProxy nodes are
passed to ParseArrowFunctionLiteral(), which will then skip parsing the
paramaeter list. This avoids having to rewind when the arrow is found and
restart parsing the parameter list.
Note that the empty parameter list "()" is handled directly in
ParsePrimaryExpression(): after is has consumed the opening parenthesis,
if a closing parenthesis follows, then the only valid input is an arrow
function. In this case, ParsePrimaryExpression() directly calls
ParseArrowFunctionLiteral(), to avoid needing to return a sentinel value
to signal the empty parameter list. Because it will consume the body of
the arrow function, ParseAssignmentExpression() will not see the arrow
"=>" token as next, and return the already-parser expression.
The implementation is done in ParserBase, so it was needed to do some
additions to ParserBase, ParserTraits and PreParserTraits. Some of the
glue code can be removed later on when more more functionality is moved
to ParserBase.
Additionally, this adds a runtime flag "harmony_arrow_functions"
(disabled by default); enabling "harmony" will enable it as well.
BUG=v8:2700
LOG=N
R=marja@chromium.org
Review URL: https://codereview.chromium.org/383983002
Patch from Adrián Pérez de Castro <aperez@igalia.com>.
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22366 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-14 07:55:45 +00:00
|
|
|
kAllowForOf, kAllowNativesSyntax, kAllowArrowFunctions
|
2014-07-08 07:48:22 +00:00
|
|
|
};
|
2014-06-20 12:28:13 +00:00
|
|
|
ParserFlag* generated_flags = NULL;
|
|
|
|
if (flags == NULL) {
|
2014-03-11 16:30:47 +00:00
|
|
|
flags = default_flags;
|
|
|
|
flags_len = ARRAY_SIZE(default_flags);
|
2014-06-20 12:28:13 +00:00
|
|
|
if (always_true_flags != NULL) {
|
|
|
|
// Remove always_true_flags from default_flags.
|
|
|
|
CHECK(always_true_flags_len < flags_len);
|
|
|
|
generated_flags = new ParserFlag[flags_len - always_true_flags_len];
|
|
|
|
int flag_index = 0;
|
|
|
|
for (int i = 0; i < flags_len; ++i) {
|
|
|
|
bool use_flag = true;
|
|
|
|
for (int j = 0; j < always_true_flags_len; ++j) {
|
|
|
|
if (flags[i] == always_true_flags[j]) {
|
|
|
|
use_flag = false;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
if (use_flag) generated_flags[flag_index++] = flags[i];
|
|
|
|
}
|
|
|
|
CHECK(flag_index == flags_len - always_true_flags_len);
|
|
|
|
flags_len = flag_index;
|
|
|
|
flags = generated_flags;
|
|
|
|
}
|
2014-03-11 16:30:47 +00:00
|
|
|
}
|
2014-02-05 15:30:01 +00:00
|
|
|
for (int i = 0; context_data[i][0] != NULL; ++i) {
|
|
|
|
for (int j = 0; statement_data[j] != NULL; ++j) {
|
|
|
|
int kPrefixLen = i::StrLength(context_data[i][0]);
|
|
|
|
int kStatementLen = i::StrLength(statement_data[j]);
|
|
|
|
int kSuffixLen = i::StrLength(context_data[i][1]);
|
|
|
|
int kProgramSize = kPrefixLen + kStatementLen + kSuffixLen;
|
|
|
|
|
|
|
|
// Plug the source code pieces together.
|
|
|
|
i::ScopedVector<char> program(kProgramSize + 1);
|
2014-06-13 16:43:27 +00:00
|
|
|
int length = i::SNPrintF(program,
|
|
|
|
"%s%s%s",
|
|
|
|
context_data[i][0],
|
|
|
|
statement_data[j],
|
|
|
|
context_data[i][1]);
|
2014-02-05 15:30:01 +00:00
|
|
|
CHECK(length == kProgramSize);
|
|
|
|
TestParserSync(program.start(),
|
|
|
|
flags,
|
2014-03-11 16:30:47 +00:00
|
|
|
flags_len,
|
2014-06-20 12:28:13 +00:00
|
|
|
result,
|
|
|
|
always_true_flags,
|
|
|
|
always_true_flags_len);
|
2014-02-05 15:30:01 +00:00
|
|
|
}
|
|
|
|
}
|
2014-06-20 12:28:13 +00:00
|
|
|
delete[] generated_flags;
|
2014-02-05 15:30:01 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
|
2014-02-04 11:26:19 +00:00
|
|
|
TEST(ErrorsEvalAndArguments) {
|
|
|
|
// Tests that both preparsing and parsing produce the right kind of errors for
|
|
|
|
// using "eval" and "arguments" as identifiers. Without the strict mode, it's
|
|
|
|
// ok to use "eval" or "arguments" as identifiers. With the strict mode, it
|
|
|
|
// isn't.
|
2014-02-04 16:38:47 +00:00
|
|
|
const char* context_data[][2] = {
|
2014-02-05 15:30:01 +00:00
|
|
|
{ "\"use strict\";", "" },
|
2014-02-04 16:38:47 +00:00
|
|
|
{ "var eval; function test_func() {\"use strict\"; ", "}"},
|
|
|
|
{ NULL, NULL }
|
2014-02-04 11:26:19 +00:00
|
|
|
};
|
|
|
|
|
2014-02-04 16:38:47 +00:00
|
|
|
const char* statement_data[] = {
|
2014-07-08 07:48:22 +00:00
|
|
|
"var eval;",
|
|
|
|
"var arguments",
|
|
|
|
"var foo, eval;",
|
|
|
|
"var foo, arguments;",
|
|
|
|
"try { } catch (eval) { }",
|
|
|
|
"try { } catch (arguments) { }",
|
|
|
|
"function eval() { }",
|
|
|
|
"function arguments() { }",
|
|
|
|
"function foo(eval) { }",
|
|
|
|
"function foo(arguments) { }",
|
|
|
|
"function foo(bar, eval) { }",
|
|
|
|
"function foo(bar, arguments) { }",
|
Implement handling of arrow functions in the parser
Arrow functions are parsed from ParseAssignmentExpression(). Handling the
parameter list is done by letting ParseConditionalExpression() parse a comma
separated list of identifiers, and it returns a tree of BinaryOperation nodes
with VariableProxy leaves, or a single VariableProxy if there is only one
parameter. When the arrow token "=>" is found, the VariableProxy nodes are
passed to ParseArrowFunctionLiteral(), which will then skip parsing the
paramaeter list. This avoids having to rewind when the arrow is found and
restart parsing the parameter list.
Note that the empty parameter list "()" is handled directly in
ParsePrimaryExpression(): after is has consumed the opening parenthesis,
if a closing parenthesis follows, then the only valid input is an arrow
function. In this case, ParsePrimaryExpression() directly calls
ParseArrowFunctionLiteral(), to avoid needing to return a sentinel value
to signal the empty parameter list. Because it will consume the body of
the arrow function, ParseAssignmentExpression() will not see the arrow
"=>" token as next, and return the already-parser expression.
The implementation is done in ParserBase, so it was needed to do some
additions to ParserBase, ParserTraits and PreParserTraits. Some of the
glue code can be removed later on when more more functionality is moved
to ParserBase.
Additionally, this adds a runtime flag "harmony_arrow_functions"
(disabled by default); enabling "harmony" will enable it as well.
BUG=v8:2700
LOG=N
R=marja@chromium.org
Review URL: https://codereview.chromium.org/383983002
Patch from Adrián Pérez de Castro <aperez@igalia.com>.
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22366 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-14 07:55:45 +00:00
|
|
|
"(eval) => { }",
|
|
|
|
"(arguments) => { }",
|
|
|
|
"(foo, eval) => { }",
|
|
|
|
"(foo, arguments) => { }",
|
2014-07-08 07:48:22 +00:00
|
|
|
"eval = 1;",
|
|
|
|
"arguments = 1;",
|
|
|
|
"var foo = eval = 1;",
|
|
|
|
"var foo = arguments = 1;",
|
|
|
|
"++eval;",
|
|
|
|
"++arguments;",
|
|
|
|
"eval++;",
|
|
|
|
"arguments++;",
|
|
|
|
NULL
|
|
|
|
};
|
2014-02-04 11:26:19 +00:00
|
|
|
|
2014-02-05 15:30:01 +00:00
|
|
|
RunParserSyncTest(context_data, statement_data, kError);
|
|
|
|
}
|
2014-02-04 12:19:53 +00:00
|
|
|
|
|
|
|
|
2014-03-11 14:39:08 +00:00
|
|
|
TEST(NoErrorsEvalAndArgumentsSloppy) {
|
2014-02-05 15:30:01 +00:00
|
|
|
// Tests that both preparsing and parsing accept "eval" and "arguments" as
|
|
|
|
// identifiers when needed.
|
|
|
|
const char* context_data[][2] = {
|
|
|
|
{ "", "" },
|
|
|
|
{ "function test_func() {", "}"},
|
|
|
|
{ NULL, NULL }
|
2014-02-04 16:38:47 +00:00
|
|
|
};
|
|
|
|
|
2014-02-05 15:30:01 +00:00
|
|
|
const char* statement_data[] = {
|
|
|
|
"var eval;",
|
|
|
|
"var arguments",
|
|
|
|
"var foo, eval;",
|
|
|
|
"var foo, arguments;",
|
|
|
|
"try { } catch (eval) { }",
|
|
|
|
"try { } catch (arguments) { }",
|
|
|
|
"function eval() { }",
|
|
|
|
"function arguments() { }",
|
|
|
|
"function foo(eval) { }",
|
|
|
|
"function foo(arguments) { }",
|
|
|
|
"function foo(bar, eval) { }",
|
|
|
|
"function foo(bar, arguments) { }",
|
|
|
|
"eval = 1;",
|
|
|
|
"arguments = 1;",
|
|
|
|
"var foo = eval = 1;",
|
|
|
|
"var foo = arguments = 1;",
|
|
|
|
"++eval;",
|
|
|
|
"++arguments;",
|
|
|
|
"eval++;",
|
|
|
|
"arguments++;",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
RunParserSyncTest(context_data, statement_data, kSuccess);
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
TEST(NoErrorsEvalAndArgumentsStrict) {
|
|
|
|
const char* context_data[][2] = {
|
2014-07-08 07:48:22 +00:00
|
|
|
{ "\"use strict\";", "" },
|
|
|
|
{ "function test_func() { \"use strict\";", "}" },
|
Implement handling of arrow functions in the parser
Arrow functions are parsed from ParseAssignmentExpression(). Handling the
parameter list is done by letting ParseConditionalExpression() parse a comma
separated list of identifiers, and it returns a tree of BinaryOperation nodes
with VariableProxy leaves, or a single VariableProxy if there is only one
parameter. When the arrow token "=>" is found, the VariableProxy nodes are
passed to ParseArrowFunctionLiteral(), which will then skip parsing the
paramaeter list. This avoids having to rewind when the arrow is found and
restart parsing the parameter list.
Note that the empty parameter list "()" is handled directly in
ParsePrimaryExpression(): after is has consumed the opening parenthesis,
if a closing parenthesis follows, then the only valid input is an arrow
function. In this case, ParsePrimaryExpression() directly calls
ParseArrowFunctionLiteral(), to avoid needing to return a sentinel value
to signal the empty parameter list. Because it will consume the body of
the arrow function, ParseAssignmentExpression() will not see the arrow
"=>" token as next, and return the already-parser expression.
The implementation is done in ParserBase, so it was needed to do some
additions to ParserBase, ParserTraits and PreParserTraits. Some of the
glue code can be removed later on when more more functionality is moved
to ParserBase.
Additionally, this adds a runtime flag "harmony_arrow_functions"
(disabled by default); enabling "harmony" will enable it as well.
BUG=v8:2700
LOG=N
R=marja@chromium.org
Review URL: https://codereview.chromium.org/383983002
Patch from Adrián Pérez de Castro <aperez@igalia.com>.
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22366 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-14 07:55:45 +00:00
|
|
|
{ "() => { \"use strict\"; ", "}" },
|
2014-07-08 07:48:22 +00:00
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
2014-02-05 15:30:01 +00:00
|
|
|
|
|
|
|
const char* statement_data[] = {
|
|
|
|
"eval;",
|
|
|
|
"arguments;",
|
|
|
|
"var foo = eval;",
|
|
|
|
"var foo = arguments;",
|
|
|
|
"var foo = { eval: 1 };",
|
|
|
|
"var foo = { arguments: 1 };",
|
|
|
|
"var foo = { }; foo.eval = {};",
|
|
|
|
"var foo = { }; foo.arguments = {};",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
Implement handling of arrow functions in the parser
Arrow functions are parsed from ParseAssignmentExpression(). Handling the
parameter list is done by letting ParseConditionalExpression() parse a comma
separated list of identifiers, and it returns a tree of BinaryOperation nodes
with VariableProxy leaves, or a single VariableProxy if there is only one
parameter. When the arrow token "=>" is found, the VariableProxy nodes are
passed to ParseArrowFunctionLiteral(), which will then skip parsing the
paramaeter list. This avoids having to rewind when the arrow is found and
restart parsing the parameter list.
Note that the empty parameter list "()" is handled directly in
ParsePrimaryExpression(): after is has consumed the opening parenthesis,
if a closing parenthesis follows, then the only valid input is an arrow
function. In this case, ParsePrimaryExpression() directly calls
ParseArrowFunctionLiteral(), to avoid needing to return a sentinel value
to signal the empty parameter list. Because it will consume the body of
the arrow function, ParseAssignmentExpression() will not see the arrow
"=>" token as next, and return the already-parser expression.
The implementation is done in ParserBase, so it was needed to do some
additions to ParserBase, ParserTraits and PreParserTraits. Some of the
glue code can be removed later on when more more functionality is moved
to ParserBase.
Additionally, this adds a runtime flag "harmony_arrow_functions"
(disabled by default); enabling "harmony" will enable it as well.
BUG=v8:2700
LOG=N
R=marja@chromium.org
Review URL: https://codereview.chromium.org/383983002
Patch from Adrián Pérez de Castro <aperez@igalia.com>.
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22366 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-14 07:55:45 +00:00
|
|
|
static const ParserFlag always_flags[] = {kAllowArrowFunctions};
|
|
|
|
RunParserSyncTest(context_data, statement_data, kSuccess, NULL, 0,
|
|
|
|
always_flags, ARRAY_SIZE(always_flags));
|
2014-02-04 12:19:53 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
TEST(ErrorsFutureStrictReservedWords) {
|
|
|
|
// Tests that both preparsing and parsing produce the right kind of errors for
|
|
|
|
// using future strict reserved words as identifiers. Without the strict mode,
|
|
|
|
// it's ok to use future strict reserved words as identifiers. With the strict
|
|
|
|
// mode, it isn't.
|
2014-02-04 16:38:47 +00:00
|
|
|
const char* context_data[][2] = {
|
2014-07-08 07:48:22 +00:00
|
|
|
{ "\"use strict\";", "" },
|
|
|
|
{ "function test_func() {\"use strict\"; ", "}"},
|
Implement handling of arrow functions in the parser
Arrow functions are parsed from ParseAssignmentExpression(). Handling the
parameter list is done by letting ParseConditionalExpression() parse a comma
separated list of identifiers, and it returns a tree of BinaryOperation nodes
with VariableProxy leaves, or a single VariableProxy if there is only one
parameter. When the arrow token "=>" is found, the VariableProxy nodes are
passed to ParseArrowFunctionLiteral(), which will then skip parsing the
paramaeter list. This avoids having to rewind when the arrow is found and
restart parsing the parameter list.
Note that the empty parameter list "()" is handled directly in
ParsePrimaryExpression(): after is has consumed the opening parenthesis,
if a closing parenthesis follows, then the only valid input is an arrow
function. In this case, ParsePrimaryExpression() directly calls
ParseArrowFunctionLiteral(), to avoid needing to return a sentinel value
to signal the empty parameter list. Because it will consume the body of
the arrow function, ParseAssignmentExpression() will not see the arrow
"=>" token as next, and return the already-parser expression.
The implementation is done in ParserBase, so it was needed to do some
additions to ParserBase, ParserTraits and PreParserTraits. Some of the
glue code can be removed later on when more more functionality is moved
to ParserBase.
Additionally, this adds a runtime flag "harmony_arrow_functions"
(disabled by default); enabling "harmony" will enable it as well.
BUG=v8:2700
LOG=N
R=marja@chromium.org
Review URL: https://codereview.chromium.org/383983002
Patch from Adrián Pérez de Castro <aperez@igalia.com>.
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22366 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-14 07:55:45 +00:00
|
|
|
{ "() => { \"use strict\"; ", "}" },
|
2014-07-08 07:48:22 +00:00
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
2014-02-04 12:19:53 +00:00
|
|
|
|
2014-02-04 16:38:47 +00:00
|
|
|
const char* statement_data[] = {
|
2014-02-05 08:38:26 +00:00
|
|
|
"var interface;",
|
2014-02-04 16:38:47 +00:00
|
|
|
"var foo, interface;",
|
|
|
|
"try { } catch (interface) { }",
|
|
|
|
"function interface() { }",
|
|
|
|
"function foo(interface) { }",
|
|
|
|
"function foo(bar, interface) { }",
|
|
|
|
"interface = 1;",
|
2014-02-05 08:38:26 +00:00
|
|
|
"var foo = interface = 1;",
|
2014-02-04 16:38:47 +00:00
|
|
|
"++interface;",
|
|
|
|
"interface++;",
|
2014-07-02 12:27:34 +00:00
|
|
|
"var yield = 13;",
|
2014-02-04 16:38:47 +00:00
|
|
|
NULL
|
2014-02-04 12:19:53 +00:00
|
|
|
};
|
|
|
|
|
Implement handling of arrow functions in the parser
Arrow functions are parsed from ParseAssignmentExpression(). Handling the
parameter list is done by letting ParseConditionalExpression() parse a comma
separated list of identifiers, and it returns a tree of BinaryOperation nodes
with VariableProxy leaves, or a single VariableProxy if there is only one
parameter. When the arrow token "=>" is found, the VariableProxy nodes are
passed to ParseArrowFunctionLiteral(), which will then skip parsing the
paramaeter list. This avoids having to rewind when the arrow is found and
restart parsing the parameter list.
Note that the empty parameter list "()" is handled directly in
ParsePrimaryExpression(): after is has consumed the opening parenthesis,
if a closing parenthesis follows, then the only valid input is an arrow
function. In this case, ParsePrimaryExpression() directly calls
ParseArrowFunctionLiteral(), to avoid needing to return a sentinel value
to signal the empty parameter list. Because it will consume the body of
the arrow function, ParseAssignmentExpression() will not see the arrow
"=>" token as next, and return the already-parser expression.
The implementation is done in ParserBase, so it was needed to do some
additions to ParserBase, ParserTraits and PreParserTraits. Some of the
glue code can be removed later on when more more functionality is moved
to ParserBase.
Additionally, this adds a runtime flag "harmony_arrow_functions"
(disabled by default); enabling "harmony" will enable it as well.
BUG=v8:2700
LOG=N
R=marja@chromium.org
Review URL: https://codereview.chromium.org/383983002
Patch from Adrián Pérez de Castro <aperez@igalia.com>.
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22366 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-14 07:55:45 +00:00
|
|
|
static const ParserFlag always_flags[] = {kAllowArrowFunctions};
|
|
|
|
RunParserSyncTest(context_data, statement_data, kError, NULL, 0, always_flags,
|
|
|
|
ARRAY_SIZE(always_flags));
|
2014-02-05 15:30:01 +00:00
|
|
|
}
|
2014-02-04 12:19:53 +00:00
|
|
|
|
|
|
|
|
2014-02-05 15:30:01 +00:00
|
|
|
TEST(NoErrorsFutureStrictReservedWords) {
|
2014-07-08 07:48:22 +00:00
|
|
|
const char* context_data[][2] = {
|
|
|
|
{ "", "" },
|
|
|
|
{ "function test_func() {", "}"},
|
Implement handling of arrow functions in the parser
Arrow functions are parsed from ParseAssignmentExpression(). Handling the
parameter list is done by letting ParseConditionalExpression() parse a comma
separated list of identifiers, and it returns a tree of BinaryOperation nodes
with VariableProxy leaves, or a single VariableProxy if there is only one
parameter. When the arrow token "=>" is found, the VariableProxy nodes are
passed to ParseArrowFunctionLiteral(), which will then skip parsing the
paramaeter list. This avoids having to rewind when the arrow is found and
restart parsing the parameter list.
Note that the empty parameter list "()" is handled directly in
ParsePrimaryExpression(): after is has consumed the opening parenthesis,
if a closing parenthesis follows, then the only valid input is an arrow
function. In this case, ParsePrimaryExpression() directly calls
ParseArrowFunctionLiteral(), to avoid needing to return a sentinel value
to signal the empty parameter list. Because it will consume the body of
the arrow function, ParseAssignmentExpression() will not see the arrow
"=>" token as next, and return the already-parser expression.
The implementation is done in ParserBase, so it was needed to do some
additions to ParserBase, ParserTraits and PreParserTraits. Some of the
glue code can be removed later on when more more functionality is moved
to ParserBase.
Additionally, this adds a runtime flag "harmony_arrow_functions"
(disabled by default); enabling "harmony" will enable it as well.
BUG=v8:2700
LOG=N
R=marja@chromium.org
Review URL: https://codereview.chromium.org/383983002
Patch from Adrián Pérez de Castro <aperez@igalia.com>.
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22366 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-14 07:55:45 +00:00
|
|
|
{ "() => {", "}" },
|
2014-07-08 07:48:22 +00:00
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
2014-02-04 16:38:47 +00:00
|
|
|
|
2014-02-05 15:30:01 +00:00
|
|
|
const char* statement_data[] = {
|
|
|
|
"var interface;",
|
|
|
|
"var foo, interface;",
|
|
|
|
"try { } catch (interface) { }",
|
|
|
|
"function interface() { }",
|
|
|
|
"function foo(interface) { }",
|
|
|
|
"function foo(bar, interface) { }",
|
|
|
|
"interface = 1;",
|
|
|
|
"var foo = interface = 1;",
|
|
|
|
"++interface;",
|
|
|
|
"interface++;",
|
2014-07-02 12:27:34 +00:00
|
|
|
"var yield = 13;",
|
2014-02-05 15:30:01 +00:00
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
Implement handling of arrow functions in the parser
Arrow functions are parsed from ParseAssignmentExpression(). Handling the
parameter list is done by letting ParseConditionalExpression() parse a comma
separated list of identifiers, and it returns a tree of BinaryOperation nodes
with VariableProxy leaves, or a single VariableProxy if there is only one
parameter. When the arrow token "=>" is found, the VariableProxy nodes are
passed to ParseArrowFunctionLiteral(), which will then skip parsing the
paramaeter list. This avoids having to rewind when the arrow is found and
restart parsing the parameter list.
Note that the empty parameter list "()" is handled directly in
ParsePrimaryExpression(): after is has consumed the opening parenthesis,
if a closing parenthesis follows, then the only valid input is an arrow
function. In this case, ParsePrimaryExpression() directly calls
ParseArrowFunctionLiteral(), to avoid needing to return a sentinel value
to signal the empty parameter list. Because it will consume the body of
the arrow function, ParseAssignmentExpression() will not see the arrow
"=>" token as next, and return the already-parser expression.
The implementation is done in ParserBase, so it was needed to do some
additions to ParserBase, ParserTraits and PreParserTraits. Some of the
glue code can be removed later on when more more functionality is moved
to ParserBase.
Additionally, this adds a runtime flag "harmony_arrow_functions"
(disabled by default); enabling "harmony" will enable it as well.
BUG=v8:2700
LOG=N
R=marja@chromium.org
Review URL: https://codereview.chromium.org/383983002
Patch from Adrián Pérez de Castro <aperez@igalia.com>.
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22366 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-14 07:55:45 +00:00
|
|
|
static const ParserFlag always_flags[] = {kAllowArrowFunctions};
|
|
|
|
RunParserSyncTest(context_data, statement_data, kSuccess, NULL, 0,
|
|
|
|
always_flags, ARRAY_SIZE(always_flags));
|
2014-02-04 16:38:47 +00:00
|
|
|
}
|
2014-02-04 12:19:53 +00:00
|
|
|
|
2014-02-04 16:38:47 +00:00
|
|
|
|
|
|
|
TEST(ErrorsReservedWords) {
|
|
|
|
// Tests that both preparsing and parsing produce the right kind of errors for
|
|
|
|
// using future reserved words as identifiers. These tests don't depend on the
|
|
|
|
// strict mode.
|
|
|
|
const char* context_data[][2] = {
|
2014-07-08 07:48:22 +00:00
|
|
|
{ "", "" },
|
|
|
|
{ "\"use strict\";", "" },
|
|
|
|
{ "var eval; function test_func() {", "}"},
|
|
|
|
{ "var eval; function test_func() {\"use strict\"; ", "}"},
|
Implement handling of arrow functions in the parser
Arrow functions are parsed from ParseAssignmentExpression(). Handling the
parameter list is done by letting ParseConditionalExpression() parse a comma
separated list of identifiers, and it returns a tree of BinaryOperation nodes
with VariableProxy leaves, or a single VariableProxy if there is only one
parameter. When the arrow token "=>" is found, the VariableProxy nodes are
passed to ParseArrowFunctionLiteral(), which will then skip parsing the
paramaeter list. This avoids having to rewind when the arrow is found and
restart parsing the parameter list.
Note that the empty parameter list "()" is handled directly in
ParsePrimaryExpression(): after is has consumed the opening parenthesis,
if a closing parenthesis follows, then the only valid input is an arrow
function. In this case, ParsePrimaryExpression() directly calls
ParseArrowFunctionLiteral(), to avoid needing to return a sentinel value
to signal the empty parameter list. Because it will consume the body of
the arrow function, ParseAssignmentExpression() will not see the arrow
"=>" token as next, and return the already-parser expression.
The implementation is done in ParserBase, so it was needed to do some
additions to ParserBase, ParserTraits and PreParserTraits. Some of the
glue code can be removed later on when more more functionality is moved
to ParserBase.
Additionally, this adds a runtime flag "harmony_arrow_functions"
(disabled by default); enabling "harmony" will enable it as well.
BUG=v8:2700
LOG=N
R=marja@chromium.org
Review URL: https://codereview.chromium.org/383983002
Patch from Adrián Pérez de Castro <aperez@igalia.com>.
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22366 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-14 07:55:45 +00:00
|
|
|
{ "var eval; () => {", "}"},
|
|
|
|
{ "var eval; () => {\"use strict\"; ", "}"},
|
2014-07-08 07:48:22 +00:00
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
2014-02-04 12:19:53 +00:00
|
|
|
|
2014-02-04 16:38:47 +00:00
|
|
|
const char* statement_data[] = {
|
2014-07-08 07:48:22 +00:00
|
|
|
"var super;",
|
|
|
|
"var foo, super;",
|
|
|
|
"try { } catch (super) { }",
|
|
|
|
"function super() { }",
|
|
|
|
"function foo(super) { }",
|
|
|
|
"function foo(bar, super) { }",
|
Implement handling of arrow functions in the parser
Arrow functions are parsed from ParseAssignmentExpression(). Handling the
parameter list is done by letting ParseConditionalExpression() parse a comma
separated list of identifiers, and it returns a tree of BinaryOperation nodes
with VariableProxy leaves, or a single VariableProxy if there is only one
parameter. When the arrow token "=>" is found, the VariableProxy nodes are
passed to ParseArrowFunctionLiteral(), which will then skip parsing the
paramaeter list. This avoids having to rewind when the arrow is found and
restart parsing the parameter list.
Note that the empty parameter list "()" is handled directly in
ParsePrimaryExpression(): after is has consumed the opening parenthesis,
if a closing parenthesis follows, then the only valid input is an arrow
function. In this case, ParsePrimaryExpression() directly calls
ParseArrowFunctionLiteral(), to avoid needing to return a sentinel value
to signal the empty parameter list. Because it will consume the body of
the arrow function, ParseAssignmentExpression() will not see the arrow
"=>" token as next, and return the already-parser expression.
The implementation is done in ParserBase, so it was needed to do some
additions to ParserBase, ParserTraits and PreParserTraits. Some of the
glue code can be removed later on when more more functionality is moved
to ParserBase.
Additionally, this adds a runtime flag "harmony_arrow_functions"
(disabled by default); enabling "harmony" will enable it as well.
BUG=v8:2700
LOG=N
R=marja@chromium.org
Review URL: https://codereview.chromium.org/383983002
Patch from Adrián Pérez de Castro <aperez@igalia.com>.
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22366 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-14 07:55:45 +00:00
|
|
|
"(super) => { }",
|
|
|
|
"(bar, super) => { }",
|
2014-07-08 07:48:22 +00:00
|
|
|
"super = 1;",
|
|
|
|
"var foo = super = 1;",
|
|
|
|
"++super;",
|
|
|
|
"super++;",
|
|
|
|
"function foo super",
|
|
|
|
NULL
|
|
|
|
};
|
2014-02-04 16:38:47 +00:00
|
|
|
|
2014-02-05 15:30:01 +00:00
|
|
|
RunParserSyncTest(context_data, statement_data, kError);
|
|
|
|
}
|
2014-02-04 16:38:47 +00:00
|
|
|
|
2014-02-04 18:16:45 +00:00
|
|
|
|
2014-07-10 14:39:35 +00:00
|
|
|
TEST(NoErrorsLetSloppyAllModes) {
|
|
|
|
// In sloppy mode, it's okay to use "let" as identifier.
|
|
|
|
const char* context_data[][2] = {
|
|
|
|
{ "", "" },
|
|
|
|
{ "function f() {", "}" },
|
|
|
|
{ "(function f() {", "})" },
|
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
|
|
|
|
|
|
|
const char* statement_data[] = {
|
|
|
|
"var let;",
|
|
|
|
"var foo, let;",
|
|
|
|
"try { } catch (let) { }",
|
|
|
|
"function let() { }",
|
|
|
|
"(function let() { })",
|
|
|
|
"function foo(let) { }",
|
|
|
|
"function foo(bar, let) { }",
|
|
|
|
"let = 1;",
|
|
|
|
"var foo = let = 1;",
|
|
|
|
"let * 2;",
|
|
|
|
"++let;",
|
|
|
|
"let++;",
|
|
|
|
"let: 34",
|
|
|
|
"function let(let) { let: let(let + let(0)); }",
|
|
|
|
"({ let: 1 })",
|
|
|
|
"({ get let() { 1 } })",
|
|
|
|
"let(100)",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
RunParserSyncTest(context_data, statement_data, kSuccess);
|
|
|
|
}
|
|
|
|
|
|
|
|
|
2014-07-02 12:27:34 +00:00
|
|
|
TEST(NoErrorsYieldSloppyAllModes) {
|
2014-03-11 14:39:08 +00:00
|
|
|
// In sloppy mode, it's okay to use "yield" as identifier, *except* inside a
|
2014-07-10 14:39:35 +00:00
|
|
|
// generator (see other test).
|
2014-02-05 15:30:01 +00:00
|
|
|
const char* context_data[][2] = {
|
|
|
|
{ "", "" },
|
2014-07-02 12:27:34 +00:00
|
|
|
{ "function not_gen() {", "}" },
|
|
|
|
{ "(function not_gen() {", "})" },
|
2014-02-05 15:30:01 +00:00
|
|
|
{ NULL, NULL }
|
2014-02-04 18:16:45 +00:00
|
|
|
};
|
|
|
|
|
2014-02-05 15:30:01 +00:00
|
|
|
const char* statement_data[] = {
|
|
|
|
"var yield;",
|
|
|
|
"var foo, yield;",
|
|
|
|
"try { } catch (yield) { }",
|
|
|
|
"function yield() { }",
|
2014-07-02 12:27:34 +00:00
|
|
|
"(function yield() { })",
|
2014-02-05 15:30:01 +00:00
|
|
|
"function foo(yield) { }",
|
|
|
|
"function foo(bar, yield) { }",
|
|
|
|
"yield = 1;",
|
|
|
|
"var foo = yield = 1;",
|
2014-07-10 14:39:35 +00:00
|
|
|
"yield * 2;",
|
2014-02-05 15:30:01 +00:00
|
|
|
"++yield;",
|
|
|
|
"yield++;",
|
2014-07-02 12:27:34 +00:00
|
|
|
"yield: 34",
|
2014-07-10 14:39:35 +00:00
|
|
|
"function yield(yield) { yield: yield (yield + yield(0)); }",
|
2014-07-02 12:27:34 +00:00
|
|
|
"({ yield: 1 })",
|
|
|
|
"({ get yield() { 1 } })",
|
2014-07-10 14:39:35 +00:00
|
|
|
"yield(100)",
|
|
|
|
"yield[100]",
|
2014-02-05 15:30:01 +00:00
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
2014-07-15 08:10:26 +00:00
|
|
|
RunParserSyncTest(context_data, statement_data, kSuccess);
|
2014-02-04 18:16:45 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
|
2014-07-02 12:27:34 +00:00
|
|
|
TEST(NoErrorsYieldSloppyGeneratorsEnabled) {
|
|
|
|
// In sloppy mode, it's okay to use "yield" as identifier, *except* inside a
|
|
|
|
// generator (see next test).
|
2014-02-04 18:16:45 +00:00
|
|
|
const char* context_data[][2] = {
|
2014-07-02 12:27:34 +00:00
|
|
|
{ "", "" },
|
|
|
|
{ "function not_gen() {", "}" },
|
|
|
|
{ "function * gen() { function not_gen() {", "} }" },
|
|
|
|
{ "(function not_gen() {", "})" },
|
|
|
|
{ "(function * gen() { (function not_gen() {", "}) })" },
|
2014-02-05 15:30:01 +00:00
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
|
|
|
|
|
|
|
const char* statement_data[] = {
|
|
|
|
"var yield;",
|
|
|
|
"var foo, yield;",
|
|
|
|
"try { } catch (yield) { }",
|
|
|
|
"function yield() { }",
|
2014-07-02 12:27:34 +00:00
|
|
|
"(function yield() { })",
|
|
|
|
"function foo(yield) { }",
|
|
|
|
"function foo(bar, yield) { }",
|
|
|
|
"function * yield() { }",
|
|
|
|
"(function * yield() { })",
|
2014-02-05 15:30:01 +00:00
|
|
|
"yield = 1;",
|
|
|
|
"var foo = yield = 1;",
|
2014-07-10 14:39:35 +00:00
|
|
|
"yield * 2;",
|
2014-02-05 15:30:01 +00:00
|
|
|
"++yield;",
|
|
|
|
"yield++;",
|
2014-07-02 12:27:34 +00:00
|
|
|
"yield: 34",
|
2014-07-10 14:39:35 +00:00
|
|
|
"function yield(yield) { yield: yield (yield + yield(0)); }",
|
2014-07-02 12:27:34 +00:00
|
|
|
"({ yield: 1 })",
|
|
|
|
"({ get yield() { 1 } })",
|
2014-07-10 14:39:35 +00:00
|
|
|
"yield(100)",
|
|
|
|
"yield[100]",
|
2014-02-05 15:30:01 +00:00
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
2014-07-02 12:27:34 +00:00
|
|
|
// This test requires kAllowGenerators to succeed.
|
2014-07-10 14:39:35 +00:00
|
|
|
static const ParserFlag always_true_flags[] = { kAllowGenerators };
|
2014-07-02 12:27:34 +00:00
|
|
|
RunParserSyncTest(context_data, statement_data, kSuccess, NULL, 0,
|
|
|
|
always_true_flags, 1);
|
2014-02-05 15:30:01 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
TEST(ErrorsYieldStrict) {
|
|
|
|
const char* context_data[][2] = {
|
2014-07-08 07:48:22 +00:00
|
|
|
{ "\"use strict\";", "" },
|
|
|
|
{ "\"use strict\"; function not_gen() {", "}" },
|
|
|
|
{ "function test_func() {\"use strict\"; ", "}"},
|
|
|
|
{ "\"use strict\"; function * gen() { function not_gen() {", "} }" },
|
|
|
|
{ "\"use strict\"; (function not_gen() {", "})" },
|
|
|
|
{ "\"use strict\"; (function * gen() { (function not_gen() {", "}) })" },
|
Implement handling of arrow functions in the parser
Arrow functions are parsed from ParseAssignmentExpression(). Handling the
parameter list is done by letting ParseConditionalExpression() parse a comma
separated list of identifiers, and it returns a tree of BinaryOperation nodes
with VariableProxy leaves, or a single VariableProxy if there is only one
parameter. When the arrow token "=>" is found, the VariableProxy nodes are
passed to ParseArrowFunctionLiteral(), which will then skip parsing the
paramaeter list. This avoids having to rewind when the arrow is found and
restart parsing the parameter list.
Note that the empty parameter list "()" is handled directly in
ParsePrimaryExpression(): after is has consumed the opening parenthesis,
if a closing parenthesis follows, then the only valid input is an arrow
function. In this case, ParsePrimaryExpression() directly calls
ParseArrowFunctionLiteral(), to avoid needing to return a sentinel value
to signal the empty parameter list. Because it will consume the body of
the arrow function, ParseAssignmentExpression() will not see the arrow
"=>" token as next, and return the already-parser expression.
The implementation is done in ParserBase, so it was needed to do some
additions to ParserBase, ParserTraits and PreParserTraits. Some of the
glue code can be removed later on when more more functionality is moved
to ParserBase.
Additionally, this adds a runtime flag "harmony_arrow_functions"
(disabled by default); enabling "harmony" will enable it as well.
BUG=v8:2700
LOG=N
R=marja@chromium.org
Review URL: https://codereview.chromium.org/383983002
Patch from Adrián Pérez de Castro <aperez@igalia.com>.
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22366 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-14 07:55:45 +00:00
|
|
|
{ "() => {\"use strict\"; ", "}" },
|
2014-07-08 07:48:22 +00:00
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
2014-02-04 18:16:45 +00:00
|
|
|
|
|
|
|
const char* statement_data[] = {
|
|
|
|
"var yield;",
|
|
|
|
"var foo, yield;",
|
|
|
|
"try { } catch (yield) { }",
|
|
|
|
"function yield() { }",
|
2014-07-02 12:27:34 +00:00
|
|
|
"(function yield() { })",
|
2014-02-04 18:16:45 +00:00
|
|
|
"function foo(yield) { }",
|
|
|
|
"function foo(bar, yield) { }",
|
2014-07-02 12:27:34 +00:00
|
|
|
"function * yield() { }",
|
|
|
|
"(function * yield() { })",
|
2014-02-04 18:16:45 +00:00
|
|
|
"yield = 1;",
|
2014-02-05 08:38:26 +00:00
|
|
|
"var foo = yield = 1;",
|
2014-02-04 18:16:45 +00:00
|
|
|
"++yield;",
|
|
|
|
"yield++;",
|
2014-07-02 12:27:34 +00:00
|
|
|
"yield: 34;",
|
2014-02-04 18:16:45 +00:00
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
2014-02-05 15:30:01 +00:00
|
|
|
RunParserSyncTest(context_data, statement_data, kError);
|
|
|
|
}
|
2014-02-04 18:16:45 +00:00
|
|
|
|
2014-02-05 08:38:26 +00:00
|
|
|
|
2014-07-02 12:27:34 +00:00
|
|
|
TEST(NoErrorsGenerator) {
|
2014-02-05 15:30:01 +00:00
|
|
|
const char* context_data[][2] = {
|
2014-07-02 12:27:34 +00:00
|
|
|
{ "function * gen() {", "}" },
|
|
|
|
{ "(function * gen() {", "})" },
|
|
|
|
{ "(function * () {", "})" },
|
2014-02-05 15:30:01 +00:00
|
|
|
{ NULL, NULL }
|
2014-02-05 08:38:26 +00:00
|
|
|
};
|
|
|
|
|
2014-02-05 15:30:01 +00:00
|
|
|
const char* statement_data[] = {
|
2014-07-02 12:27:34 +00:00
|
|
|
// A generator without a body is valid.
|
|
|
|
""
|
|
|
|
// Valid yield expressions inside generators.
|
|
|
|
"yield 2;",
|
|
|
|
"yield * 2;",
|
|
|
|
"yield * \n 2;",
|
|
|
|
"yield yield 1;",
|
|
|
|
"yield * yield * 1;",
|
|
|
|
"yield 3 + (yield 4);",
|
|
|
|
"yield * 3 + (yield * 4);",
|
|
|
|
"(yield * 3) + (yield * 4);",
|
|
|
|
"yield 3; yield 4;",
|
|
|
|
"yield * 3; yield * 4;",
|
|
|
|
"(function (yield) { })",
|
2014-07-02 13:48:28 +00:00
|
|
|
"yield { yield: 12 }",
|
|
|
|
"yield /* comment */ { yield: 12 }",
|
|
|
|
"yield * \n { yield: 12 }",
|
|
|
|
"yield /* comment */ * \n { yield: 12 }",
|
2014-07-02 12:27:34 +00:00
|
|
|
// You can return in a generator.
|
|
|
|
"yield 1; return",
|
|
|
|
"yield * 1; return",
|
|
|
|
"yield 1; return 37",
|
|
|
|
"yield * 1; return 37",
|
|
|
|
"yield 1; return 37; yield 'dead';",
|
|
|
|
"yield * 1; return 37; yield * 'dead';",
|
|
|
|
// Yield is still a valid key in object literals.
|
|
|
|
"({ yield: 1 })",
|
|
|
|
"({ get yield() { } })",
|
2014-07-02 13:48:28 +00:00
|
|
|
// Yield without RHS.
|
|
|
|
"yield;",
|
|
|
|
"yield",
|
|
|
|
"yield\n",
|
|
|
|
"yield /* comment */"
|
|
|
|
"yield // comment\n"
|
|
|
|
"(yield)",
|
|
|
|
"[yield]",
|
|
|
|
"{yield}",
|
|
|
|
"yield, yield",
|
|
|
|
"yield; yield",
|
|
|
|
"(yield) ? yield : yield",
|
|
|
|
"(yield) \n ? yield : yield",
|
|
|
|
// If there is a newline before the next token, we don't look for RHS.
|
|
|
|
"yield\nfor (;;) {}",
|
2014-02-05 15:30:01 +00:00
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
2014-06-20 12:28:13 +00:00
|
|
|
// This test requires kAllowGenerators to succeed.
|
|
|
|
static const ParserFlag always_true_flags[] = {
|
|
|
|
kAllowGenerators
|
|
|
|
};
|
|
|
|
RunParserSyncTest(context_data, statement_data, kSuccess, NULL, 0,
|
|
|
|
always_true_flags, 1);
|
2014-02-05 08:38:26 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
|
2014-07-02 12:27:34 +00:00
|
|
|
TEST(ErrorsYieldGenerator) {
|
|
|
|
const char* context_data[][2] = {
|
|
|
|
{ "function * gen() {", "}" },
|
|
|
|
{ "\"use strict\"; function * gen() {", "}" },
|
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
|
|
|
|
|
|
|
const char* statement_data[] = {
|
|
|
|
// Invalid yield expressions inside generators.
|
|
|
|
"var yield;",
|
|
|
|
"var foo, yield;",
|
|
|
|
"try { } catch (yield) { }",
|
|
|
|
"function yield() { }",
|
|
|
|
// The name of the NFE is let-bound in the generator, which does not permit
|
|
|
|
// yield to be an identifier.
|
|
|
|
"(function yield() { })",
|
|
|
|
"(function * yield() { })",
|
|
|
|
// Yield isn't valid as a formal parameter for generators.
|
|
|
|
"function * foo(yield) { }",
|
|
|
|
"(function * foo(yield) { })",
|
|
|
|
"yield = 1;",
|
|
|
|
"var foo = yield = 1;",
|
|
|
|
"++yield;",
|
|
|
|
"yield++;",
|
|
|
|
"yield *",
|
|
|
|
"(yield *)",
|
|
|
|
// Yield binds very loosely, so this parses as "yield (3 + yield 4)", which
|
|
|
|
// is invalid.
|
|
|
|
"yield 3 + yield 4;",
|
|
|
|
"yield: 34",
|
|
|
|
"yield ? 1 : 2",
|
|
|
|
// Parses as yield (/ yield): invalid.
|
|
|
|
"yield / yield",
|
2014-07-02 13:48:28 +00:00
|
|
|
"+ yield",
|
|
|
|
"+ yield 3",
|
|
|
|
// Invalid (no newline allowed between yield and *).
|
|
|
|
"yield\n*3",
|
|
|
|
// Invalid (we see a newline, so we parse {yield:42} as a statement, not an
|
|
|
|
// object literal, and yield is not a valid label).
|
|
|
|
"yield\n{yield: 42}",
|
|
|
|
"yield /* comment */\n {yield: 42}",
|
|
|
|
"yield //comment\n {yield: 42}",
|
2014-07-02 12:27:34 +00:00
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
RunParserSyncTest(context_data, statement_data, kError);
|
|
|
|
}
|
|
|
|
|
|
|
|
|
2014-02-05 08:38:26 +00:00
|
|
|
TEST(ErrorsNameOfStrictFunction) {
|
|
|
|
// Tests that illegal tokens as names of a strict function produce the correct
|
|
|
|
// errors.
|
2014-02-05 15:30:01 +00:00
|
|
|
const char* context_data[][2] = {
|
2014-07-02 12:27:34 +00:00
|
|
|
{ "function ", ""},
|
|
|
|
{ "\"use strict\"; function", ""},
|
|
|
|
{ "function * ", ""},
|
|
|
|
{ "\"use strict\"; function * ", ""},
|
2014-02-05 15:30:01 +00:00
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
|
|
|
|
2014-02-05 08:38:26 +00:00
|
|
|
const char* statement_data[] = {
|
2014-07-02 12:27:34 +00:00
|
|
|
"eval() {\"use strict\";}",
|
|
|
|
"arguments() {\"use strict\";}",
|
|
|
|
"interface() {\"use strict\";}",
|
|
|
|
"yield() {\"use strict\";}",
|
2014-02-05 08:38:26 +00:00
|
|
|
// Future reserved words are always illegal
|
2014-02-05 15:30:01 +00:00
|
|
|
"function super() { }",
|
|
|
|
"function super() {\"use strict\";}",
|
2014-02-05 08:38:26 +00:00
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
2014-02-05 15:30:01 +00:00
|
|
|
RunParserSyncTest(context_data, statement_data, kError);
|
|
|
|
}
|
2014-02-05 08:38:26 +00:00
|
|
|
|
|
|
|
|
2014-02-05 15:30:01 +00:00
|
|
|
TEST(NoErrorsNameOfStrictFunction) {
|
|
|
|
const char* context_data[][2] = {
|
2014-07-02 12:27:34 +00:00
|
|
|
{ "function ", ""},
|
2014-02-05 15:30:01 +00:00
|
|
|
{ NULL, NULL }
|
2014-02-05 08:38:26 +00:00
|
|
|
};
|
2014-02-05 15:30:01 +00:00
|
|
|
|
|
|
|
const char* statement_data[] = {
|
2014-07-02 12:27:34 +00:00
|
|
|
"eval() { }",
|
|
|
|
"arguments() { }",
|
|
|
|
"interface() { }",
|
|
|
|
"yield() { }",
|
2014-02-05 15:30:01 +00:00
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
RunParserSyncTest(context_data, statement_data, kSuccess);
|
|
|
|
}
|
|
|
|
|
|
|
|
|
2014-07-02 12:27:34 +00:00
|
|
|
TEST(NoErrorsNameOfStrictGenerator) {
|
|
|
|
const char* context_data[][2] = {
|
|
|
|
{ "function * ", ""},
|
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
|
|
|
|
|
|
|
const char* statement_data[] = {
|
|
|
|
"eval() { }",
|
|
|
|
"arguments() { }",
|
|
|
|
"interface() { }",
|
|
|
|
"yield() { }",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
// This test requires kAllowGenerators to succeed.
|
|
|
|
static const ParserFlag always_true_flags[] = {
|
|
|
|
kAllowGenerators
|
|
|
|
};
|
|
|
|
RunParserSyncTest(context_data, statement_data, kSuccess, NULL, 0,
|
|
|
|
always_true_flags, 1);
|
|
|
|
}
|
|
|
|
|
2014-02-05 15:30:01 +00:00
|
|
|
|
2014-03-11 14:39:08 +00:00
|
|
|
TEST(ErrorsIllegalWordsAsLabelsSloppy) {
|
2014-02-05 15:30:01 +00:00
|
|
|
// Using future reserved words as labels is always an error.
|
2014-07-08 07:48:22 +00:00
|
|
|
const char* context_data[][2] = {
|
|
|
|
{ "", ""},
|
|
|
|
{ "function test_func() {", "}" },
|
Implement handling of arrow functions in the parser
Arrow functions are parsed from ParseAssignmentExpression(). Handling the
parameter list is done by letting ParseConditionalExpression() parse a comma
separated list of identifiers, and it returns a tree of BinaryOperation nodes
with VariableProxy leaves, or a single VariableProxy if there is only one
parameter. When the arrow token "=>" is found, the VariableProxy nodes are
passed to ParseArrowFunctionLiteral(), which will then skip parsing the
paramaeter list. This avoids having to rewind when the arrow is found and
restart parsing the parameter list.
Note that the empty parameter list "()" is handled directly in
ParsePrimaryExpression(): after is has consumed the opening parenthesis,
if a closing parenthesis follows, then the only valid input is an arrow
function. In this case, ParsePrimaryExpression() directly calls
ParseArrowFunctionLiteral(), to avoid needing to return a sentinel value
to signal the empty parameter list. Because it will consume the body of
the arrow function, ParseAssignmentExpression() will not see the arrow
"=>" token as next, and return the already-parser expression.
The implementation is done in ParserBase, so it was needed to do some
additions to ParserBase, ParserTraits and PreParserTraits. Some of the
glue code can be removed later on when more more functionality is moved
to ParserBase.
Additionally, this adds a runtime flag "harmony_arrow_functions"
(disabled by default); enabling "harmony" will enable it as well.
BUG=v8:2700
LOG=N
R=marja@chromium.org
Review URL: https://codereview.chromium.org/383983002
Patch from Adrián Pérez de Castro <aperez@igalia.com>.
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22366 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-14 07:55:45 +00:00
|
|
|
{ "() => {", "}" },
|
2014-07-08 07:48:22 +00:00
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
2014-02-05 15:30:01 +00:00
|
|
|
|
|
|
|
const char* statement_data[] = {
|
|
|
|
"super: while(true) { break super; }",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
RunParserSyncTest(context_data, statement_data, kError);
|
2014-02-05 08:38:26 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
|
2014-02-05 15:30:01 +00:00
|
|
|
TEST(ErrorsIllegalWordsAsLabelsStrict) {
|
2014-02-05 08:38:26 +00:00
|
|
|
// Tests that illegal tokens as labels produce the correct errors.
|
|
|
|
const char* context_data[][2] = {
|
2014-07-08 07:48:22 +00:00
|
|
|
{ "\"use strict\";", "" },
|
|
|
|
{ "function test_func() {\"use strict\"; ", "}"},
|
Implement handling of arrow functions in the parser
Arrow functions are parsed from ParseAssignmentExpression(). Handling the
parameter list is done by letting ParseConditionalExpression() parse a comma
separated list of identifiers, and it returns a tree of BinaryOperation nodes
with VariableProxy leaves, or a single VariableProxy if there is only one
parameter. When the arrow token "=>" is found, the VariableProxy nodes are
passed to ParseArrowFunctionLiteral(), which will then skip parsing the
paramaeter list. This avoids having to rewind when the arrow is found and
restart parsing the parameter list.
Note that the empty parameter list "()" is handled directly in
ParsePrimaryExpression(): after is has consumed the opening parenthesis,
if a closing parenthesis follows, then the only valid input is an arrow
function. In this case, ParsePrimaryExpression() directly calls
ParseArrowFunctionLiteral(), to avoid needing to return a sentinel value
to signal the empty parameter list. Because it will consume the body of
the arrow function, ParseAssignmentExpression() will not see the arrow
"=>" token as next, and return the already-parser expression.
The implementation is done in ParserBase, so it was needed to do some
additions to ParserBase, ParserTraits and PreParserTraits. Some of the
glue code can be removed later on when more more functionality is moved
to ParserBase.
Additionally, this adds a runtime flag "harmony_arrow_functions"
(disabled by default); enabling "harmony" will enable it as well.
BUG=v8:2700
LOG=N
R=marja@chromium.org
Review URL: https://codereview.chromium.org/383983002
Patch from Adrián Pérez de Castro <aperez@igalia.com>.
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22366 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-14 07:55:45 +00:00
|
|
|
{ "() => {\"use strict\"; ", "}" },
|
2014-07-08 07:48:22 +00:00
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
2014-02-05 08:38:26 +00:00
|
|
|
|
|
|
|
const char* statement_data[] = {
|
2014-02-05 15:30:01 +00:00
|
|
|
"super: while(true) { break super; }",
|
2014-02-05 08:38:26 +00:00
|
|
|
"interface: while(true) { break interface; }",
|
|
|
|
"yield: while(true) { break yield; }",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
2014-02-05 15:30:01 +00:00
|
|
|
RunParserSyncTest(context_data, statement_data, kError);
|
|
|
|
}
|
2014-02-05 08:38:26 +00:00
|
|
|
|
2014-02-04 16:38:47 +00:00
|
|
|
|
2014-02-05 15:30:01 +00:00
|
|
|
TEST(NoErrorsIllegalWordsAsLabels) {
|
|
|
|
// Using eval and arguments as labels is legal even in strict mode.
|
|
|
|
const char* context_data[][2] = {
|
2014-07-08 07:48:22 +00:00
|
|
|
{ "", ""},
|
|
|
|
{ "function test_func() {", "}" },
|
Implement handling of arrow functions in the parser
Arrow functions are parsed from ParseAssignmentExpression(). Handling the
parameter list is done by letting ParseConditionalExpression() parse a comma
separated list of identifiers, and it returns a tree of BinaryOperation nodes
with VariableProxy leaves, or a single VariableProxy if there is only one
parameter. When the arrow token "=>" is found, the VariableProxy nodes are
passed to ParseArrowFunctionLiteral(), which will then skip parsing the
paramaeter list. This avoids having to rewind when the arrow is found and
restart parsing the parameter list.
Note that the empty parameter list "()" is handled directly in
ParsePrimaryExpression(): after is has consumed the opening parenthesis,
if a closing parenthesis follows, then the only valid input is an arrow
function. In this case, ParsePrimaryExpression() directly calls
ParseArrowFunctionLiteral(), to avoid needing to return a sentinel value
to signal the empty parameter list. Because it will consume the body of
the arrow function, ParseAssignmentExpression() will not see the arrow
"=>" token as next, and return the already-parser expression.
The implementation is done in ParserBase, so it was needed to do some
additions to ParserBase, ParserTraits and PreParserTraits. Some of the
glue code can be removed later on when more more functionality is moved
to ParserBase.
Additionally, this adds a runtime flag "harmony_arrow_functions"
(disabled by default); enabling "harmony" will enable it as well.
BUG=v8:2700
LOG=N
R=marja@chromium.org
Review URL: https://codereview.chromium.org/383983002
Patch from Adrián Pérez de Castro <aperez@igalia.com>.
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22366 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-14 07:55:45 +00:00
|
|
|
{ "() => {", "}" },
|
2014-07-08 07:48:22 +00:00
|
|
|
{ "\"use strict\";", "" },
|
|
|
|
{ "\"use strict\"; function test_func() {", "}" },
|
Implement handling of arrow functions in the parser
Arrow functions are parsed from ParseAssignmentExpression(). Handling the
parameter list is done by letting ParseConditionalExpression() parse a comma
separated list of identifiers, and it returns a tree of BinaryOperation nodes
with VariableProxy leaves, or a single VariableProxy if there is only one
parameter. When the arrow token "=>" is found, the VariableProxy nodes are
passed to ParseArrowFunctionLiteral(), which will then skip parsing the
paramaeter list. This avoids having to rewind when the arrow is found and
restart parsing the parameter list.
Note that the empty parameter list "()" is handled directly in
ParsePrimaryExpression(): after is has consumed the opening parenthesis,
if a closing parenthesis follows, then the only valid input is an arrow
function. In this case, ParsePrimaryExpression() directly calls
ParseArrowFunctionLiteral(), to avoid needing to return a sentinel value
to signal the empty parameter list. Because it will consume the body of
the arrow function, ParseAssignmentExpression() will not see the arrow
"=>" token as next, and return the already-parser expression.
The implementation is done in ParserBase, so it was needed to do some
additions to ParserBase, ParserTraits and PreParserTraits. Some of the
glue code can be removed later on when more more functionality is moved
to ParserBase.
Additionally, this adds a runtime flag "harmony_arrow_functions"
(disabled by default); enabling "harmony" will enable it as well.
BUG=v8:2700
LOG=N
R=marja@chromium.org
Review URL: https://codereview.chromium.org/383983002
Patch from Adrián Pérez de Castro <aperez@igalia.com>.
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22366 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-14 07:55:45 +00:00
|
|
|
{ "\"use strict\"; () => {", "}" },
|
2014-07-08 07:48:22 +00:00
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
2014-02-04 16:38:47 +00:00
|
|
|
|
2014-02-05 15:30:01 +00:00
|
|
|
const char* statement_data[] = {
|
|
|
|
"mylabel: while(true) { break mylabel; }",
|
|
|
|
"eval: while(true) { break eval; }",
|
|
|
|
"arguments: while(true) { break arguments; }",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
Implement handling of arrow functions in the parser
Arrow functions are parsed from ParseAssignmentExpression(). Handling the
parameter list is done by letting ParseConditionalExpression() parse a comma
separated list of identifiers, and it returns a tree of BinaryOperation nodes
with VariableProxy leaves, or a single VariableProxy if there is only one
parameter. When the arrow token "=>" is found, the VariableProxy nodes are
passed to ParseArrowFunctionLiteral(), which will then skip parsing the
paramaeter list. This avoids having to rewind when the arrow is found and
restart parsing the parameter list.
Note that the empty parameter list "()" is handled directly in
ParsePrimaryExpression(): after is has consumed the opening parenthesis,
if a closing parenthesis follows, then the only valid input is an arrow
function. In this case, ParsePrimaryExpression() directly calls
ParseArrowFunctionLiteral(), to avoid needing to return a sentinel value
to signal the empty parameter list. Because it will consume the body of
the arrow function, ParseAssignmentExpression() will not see the arrow
"=>" token as next, and return the already-parser expression.
The implementation is done in ParserBase, so it was needed to do some
additions to ParserBase, ParserTraits and PreParserTraits. Some of the
glue code can be removed later on when more more functionality is moved
to ParserBase.
Additionally, this adds a runtime flag "harmony_arrow_functions"
(disabled by default); enabling "harmony" will enable it as well.
BUG=v8:2700
LOG=N
R=marja@chromium.org
Review URL: https://codereview.chromium.org/383983002
Patch from Adrián Pérez de Castro <aperez@igalia.com>.
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22366 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-14 07:55:45 +00:00
|
|
|
static const ParserFlag always_flags[] = {kAllowArrowFunctions};
|
|
|
|
RunParserSyncTest(context_data, statement_data, kSuccess, NULL, 0,
|
|
|
|
always_flags, ARRAY_SIZE(always_flags));
|
2014-02-04 11:26:19 +00:00
|
|
|
}
|
2014-02-06 13:12:10 +00:00
|
|
|
|
|
|
|
|
|
|
|
TEST(ErrorsParenthesizedLabels) {
|
|
|
|
// Parenthesized identifiers shouldn't be recognized as labels.
|
2014-07-08 07:48:22 +00:00
|
|
|
const char* context_data[][2] = {
|
|
|
|
{ "", ""},
|
|
|
|
{ "function test_func() {", "}" },
|
Implement handling of arrow functions in the parser
Arrow functions are parsed from ParseAssignmentExpression(). Handling the
parameter list is done by letting ParseConditionalExpression() parse a comma
separated list of identifiers, and it returns a tree of BinaryOperation nodes
with VariableProxy leaves, or a single VariableProxy if there is only one
parameter. When the arrow token "=>" is found, the VariableProxy nodes are
passed to ParseArrowFunctionLiteral(), which will then skip parsing the
paramaeter list. This avoids having to rewind when the arrow is found and
restart parsing the parameter list.
Note that the empty parameter list "()" is handled directly in
ParsePrimaryExpression(): after is has consumed the opening parenthesis,
if a closing parenthesis follows, then the only valid input is an arrow
function. In this case, ParsePrimaryExpression() directly calls
ParseArrowFunctionLiteral(), to avoid needing to return a sentinel value
to signal the empty parameter list. Because it will consume the body of
the arrow function, ParseAssignmentExpression() will not see the arrow
"=>" token as next, and return the already-parser expression.
The implementation is done in ParserBase, so it was needed to do some
additions to ParserBase, ParserTraits and PreParserTraits. Some of the
glue code can be removed later on when more more functionality is moved
to ParserBase.
Additionally, this adds a runtime flag "harmony_arrow_functions"
(disabled by default); enabling "harmony" will enable it as well.
BUG=v8:2700
LOG=N
R=marja@chromium.org
Review URL: https://codereview.chromium.org/383983002
Patch from Adrián Pérez de Castro <aperez@igalia.com>.
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22366 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-14 07:55:45 +00:00
|
|
|
{ "() => {", "}" },
|
2014-07-08 07:48:22 +00:00
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
2014-02-06 13:12:10 +00:00
|
|
|
|
|
|
|
const char* statement_data[] = {
|
|
|
|
"(mylabel): while(true) { break mylabel; }",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
RunParserSyncTest(context_data, statement_data, kError);
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
TEST(NoErrorsParenthesizedDirectivePrologue) {
|
|
|
|
// Parenthesized directive prologue shouldn't be recognized.
|
|
|
|
const char* context_data[][2] = {
|
|
|
|
{ "", ""},
|
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
|
|
|
|
|
|
|
const char* statement_data[] = {
|
|
|
|
"(\"use strict\"); var eval;",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
RunParserSyncTest(context_data, statement_data, kSuccess);
|
|
|
|
}
|
2014-02-07 08:45:28 +00:00
|
|
|
|
|
|
|
|
|
|
|
TEST(ErrorsNotAnIdentifierName) {
|
|
|
|
const char* context_data[][2] = {
|
|
|
|
{ "", ""},
|
|
|
|
{ "\"use strict\";", ""},
|
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
|
|
|
|
|
|
|
const char* statement_data[] = {
|
|
|
|
"var foo = {}; foo.{;",
|
|
|
|
"var foo = {}; foo.};",
|
|
|
|
"var foo = {}; foo.=;",
|
|
|
|
"var foo = {}; foo.888;",
|
|
|
|
"var foo = {}; foo.-;",
|
|
|
|
"var foo = {}; foo.--;",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
RunParserSyncTest(context_data, statement_data, kError);
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
TEST(NoErrorsIdentifierNames) {
|
|
|
|
// Keywords etc. are valid as property names.
|
|
|
|
const char* context_data[][2] = {
|
|
|
|
{ "", ""},
|
|
|
|
{ "\"use strict\";", ""},
|
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
|
|
|
|
|
|
|
const char* statement_data[] = {
|
|
|
|
"var foo = {}; foo.if;",
|
|
|
|
"var foo = {}; foo.yield;",
|
|
|
|
"var foo = {}; foo.super;",
|
|
|
|
"var foo = {}; foo.interface;",
|
|
|
|
"var foo = {}; foo.eval;",
|
|
|
|
"var foo = {}; foo.arguments;",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
RunParserSyncTest(context_data, statement_data, kSuccess);
|
|
|
|
}
|
2014-02-07 10:28:00 +00:00
|
|
|
|
|
|
|
|
|
|
|
TEST(DontRegressPreParserDataSizes) {
|
2014-04-04 12:36:23 +00:00
|
|
|
// These tests make sure that Parser doesn't start producing less "preparse
|
|
|
|
// data" (data which the embedder can cache).
|
2014-02-07 10:28:00 +00:00
|
|
|
v8::V8::Initialize();
|
2014-04-04 12:36:23 +00:00
|
|
|
v8::Isolate* isolate = CcTest::isolate();
|
|
|
|
v8::HandleScope handles(isolate);
|
|
|
|
|
2014-07-03 08:52:28 +00:00
|
|
|
CcTest::i_isolate()->stack_guard()->SetStackLimit(GetCurrentStackPosition() -
|
|
|
|
128 * 1024);
|
2014-02-07 10:28:00 +00:00
|
|
|
|
|
|
|
struct TestCase {
|
|
|
|
const char* program;
|
|
|
|
int functions;
|
|
|
|
} test_cases[] = {
|
2014-05-05 14:55:13 +00:00
|
|
|
// No functions.
|
|
|
|
{"var x = 42;", 0},
|
|
|
|
// Functions.
|
|
|
|
{"function foo() {}", 1}, {"function foo() {} function bar() {}", 2},
|
|
|
|
// Getter / setter functions are recorded as functions if they're on the top
|
|
|
|
// level.
|
|
|
|
{"var x = {get foo(){} };", 1},
|
|
|
|
// Functions insize lazy functions are not recorded.
|
|
|
|
{"function lazy() { function a() {} function b() {} function c() {} }", 1},
|
|
|
|
{"function lazy() { var x = {get foo(){} } }", 1},
|
|
|
|
{NULL, 0}
|
2014-02-07 10:28:00 +00:00
|
|
|
};
|
|
|
|
|
|
|
|
for (int i = 0; test_cases[i].program; i++) {
|
|
|
|
const char* program = test_cases[i].program;
|
2014-04-04 12:36:23 +00:00
|
|
|
i::Factory* factory = CcTest::i_isolate()->factory();
|
2014-04-17 13:27:02 +00:00
|
|
|
i::Handle<i::String> source =
|
|
|
|
factory->NewStringFromUtf8(i::CStrVector(program)).ToHandleChecked();
|
2014-04-04 12:36:23 +00:00
|
|
|
i::Handle<i::Script> script = factory->NewScript(source);
|
|
|
|
i::CompilationInfoWithZone info(script);
|
2014-07-10 10:28:05 +00:00
|
|
|
i::ScriptData* sd = NULL;
|
Change ScriptCompiler::CompileOptions to allow for two 'cache' modes
(parser or code) and to be explicit about cache consumption or production
(rather than making presence of cached_data imply one or the other.)
Also add a --cache flag to d8, to allow testing the functionality.
-----------------------------
API change
Reason: Currently, V8 supports a 'parser cache' for repeatedly executing the same script. We'd like to add a 2nd mode that would cache code, and would like to let the embedder decide which mode they chose (if any).
Note: Previously, the 'use cached data' property was implied by the presence of the cached data itself. (That is, kNoCompileOptions and source->cached_data != NULL.) That is no longer sufficient, since the presence of data is no longer sufficient to determine /which kind/ of data is present.
Changes from old behaviour:
- If you previously didn't use caching, nothing changes.
Example:
v8::CompileUnbound(isolate, source, kNoCompileOptions);
- If you previously used caching, it worked like this:
- 1st run:
v8::CompileUnbound(isolate, source, kProduceToCache);
Then, source->cached_data would contain the
data-to-be cached. This remains the same, except you
need to tell V8 which type of data you want.
v8::CompileUnbound(isolate, source, kProduceParserCache);
- 2nd run:
v8::CompileUnbound(isolate, source, kNoCompileOptions);
with source->cached_data set to the data you received in
the first run. This will now ignore the cached data, and
you need to explicitly tell V8 to use it:
v8::CompileUnbound(isolate, source, kConsumeParserCache);
-----------------------------
BUG=
R=marja@chromium.org, yangguo@chromium.org
Review URL: https://codereview.chromium.org/389573006
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22431 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-16 12:18:33 +00:00
|
|
|
info.SetCachedData(&sd, v8::ScriptCompiler::kProduceParserCache);
|
2014-04-04 12:36:23 +00:00
|
|
|
i::Parser::Parse(&info, true);
|
2014-07-10 10:28:05 +00:00
|
|
|
i::ParseData pd(sd);
|
2014-02-07 10:28:00 +00:00
|
|
|
|
2014-07-10 10:28:05 +00:00
|
|
|
if (pd.FunctionCount() != test_cases[i].functions) {
|
2014-06-30 13:25:46 +00:00
|
|
|
v8::base::OS::Print(
|
2014-02-07 10:28:00 +00:00
|
|
|
"Expected preparse data for program:\n"
|
|
|
|
"\t%s\n"
|
|
|
|
"to contain %d functions, however, received %d functions.\n",
|
2014-07-10 10:28:05 +00:00
|
|
|
program, test_cases[i].functions, pd.FunctionCount());
|
2014-02-07 10:28:00 +00:00
|
|
|
CHECK(false);
|
|
|
|
}
|
2014-07-10 10:28:05 +00:00
|
|
|
delete sd;
|
2014-02-07 10:28:00 +00:00
|
|
|
}
|
|
|
|
}
|
2014-02-07 12:44:45 +00:00
|
|
|
|
|
|
|
|
|
|
|
TEST(FunctionDeclaresItselfStrict) {
|
|
|
|
// Tests that we produce the right kinds of errors when a function declares
|
|
|
|
// itself strict (we cannot produce there errors as soon as we see the
|
|
|
|
// offending identifiers, because we don't know at that point whether the
|
|
|
|
// function is strict or not).
|
|
|
|
const char* context_data[][2] = {
|
|
|
|
{"function eval() {", "}"},
|
|
|
|
{"function arguments() {", "}"},
|
|
|
|
{"function yield() {", "}"},
|
|
|
|
{"function interface() {", "}"},
|
|
|
|
{"function foo(eval) {", "}"},
|
|
|
|
{"function foo(arguments) {", "}"},
|
|
|
|
{"function foo(yield) {", "}"},
|
|
|
|
{"function foo(interface) {", "}"},
|
|
|
|
{"function foo(bar, eval) {", "}"},
|
|
|
|
{"function foo(bar, arguments) {", "}"},
|
|
|
|
{"function foo(bar, yield) {", "}"},
|
|
|
|
{"function foo(bar, interface) {", "}"},
|
|
|
|
{"function foo(bar, bar) {", "}"},
|
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
|
|
|
|
|
|
|
const char* strict_statement_data[] = {
|
|
|
|
"\"use strict\";",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
const char* non_strict_statement_data[] = {
|
|
|
|
";",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
RunParserSyncTest(context_data, strict_statement_data, kError);
|
|
|
|
RunParserSyncTest(context_data, non_strict_statement_data, kSuccess);
|
|
|
|
}
|
2014-02-10 08:45:13 +00:00
|
|
|
|
|
|
|
|
|
|
|
TEST(ErrorsTryWithoutCatchOrFinally) {
|
|
|
|
const char* context_data[][2] = {
|
|
|
|
{"", ""},
|
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
|
|
|
|
|
|
|
const char* statement_data[] = {
|
|
|
|
"try { }",
|
|
|
|
"try { } foo();",
|
|
|
|
"try { } catch (e) foo();",
|
|
|
|
"try { } catch { }",
|
|
|
|
"try { } finally foo();",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
RunParserSyncTest(context_data, statement_data, kError);
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
TEST(NoErrorsTryCatchFinally) {
|
|
|
|
const char* context_data[][2] = {
|
|
|
|
{"", ""},
|
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
|
|
|
|
|
|
|
const char* statement_data[] = {
|
|
|
|
"try { } catch (e) { }",
|
|
|
|
"try { } catch (e) { } finally { }",
|
|
|
|
"try { } finally { }",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
RunParserSyncTest(context_data, statement_data, kSuccess);
|
|
|
|
}
|
2014-02-11 11:51:01 +00:00
|
|
|
|
|
|
|
|
|
|
|
TEST(ErrorsRegexpLiteral) {
|
|
|
|
const char* context_data[][2] = {
|
|
|
|
{"var r = ", ""},
|
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
|
|
|
|
|
|
|
const char* statement_data[] = {
|
|
|
|
"/unterminated",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
RunParserSyncTest(context_data, statement_data, kError);
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
TEST(NoErrorsRegexpLiteral) {
|
|
|
|
const char* context_data[][2] = {
|
|
|
|
{"var r = ", ""},
|
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
|
|
|
|
|
|
|
const char* statement_data[] = {
|
|
|
|
"/foo/",
|
|
|
|
"/foo/g",
|
|
|
|
"/foo/whatever", // This is an error but not detected by the parser.
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
RunParserSyncTest(context_data, statement_data, kSuccess);
|
|
|
|
}
|
2014-02-14 11:24:26 +00:00
|
|
|
|
|
|
|
|
|
|
|
TEST(Intrinsics) {
|
|
|
|
const char* context_data[][2] = {
|
|
|
|
{"", ""},
|
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
|
|
|
|
|
|
|
const char* statement_data[] = {
|
|
|
|
"%someintrinsic(arg)",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
2014-06-20 12:28:13 +00:00
|
|
|
// This test requires kAllowNativesSyntax to succeed.
|
|
|
|
static const ParserFlag always_true_flags[] = {
|
|
|
|
kAllowNativesSyntax
|
|
|
|
};
|
|
|
|
|
|
|
|
RunParserSyncTest(context_data, statement_data, kSuccess, NULL, 0,
|
|
|
|
always_true_flags, 1);
|
2014-02-14 11:24:26 +00:00
|
|
|
}
|
2014-02-17 15:40:51 +00:00
|
|
|
|
|
|
|
|
|
|
|
TEST(NoErrorsNewExpression) {
|
|
|
|
const char* context_data[][2] = {
|
|
|
|
{"", ""},
|
|
|
|
{"var f =", ""},
|
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
|
|
|
|
|
|
|
const char* statement_data[] = {
|
|
|
|
"new foo",
|
|
|
|
"new foo();",
|
|
|
|
"new foo(1);",
|
|
|
|
"new foo(1, 2);",
|
|
|
|
// The first () will be processed as a part of the NewExpression and the
|
|
|
|
// second () will be processed as part of LeftHandSideExpression.
|
|
|
|
"new foo()();",
|
|
|
|
// The first () will be processed as a part of the inner NewExpression and
|
|
|
|
// the second () will be processed as a part of the outer NewExpression.
|
|
|
|
"new new foo()();",
|
|
|
|
"new foo.bar;",
|
|
|
|
"new foo.bar();",
|
|
|
|
"new foo.bar.baz;",
|
|
|
|
"new foo.bar().baz;",
|
|
|
|
"new foo[bar];",
|
|
|
|
"new foo[bar]();",
|
|
|
|
"new foo[bar][baz];",
|
|
|
|
"new foo[bar]()[baz];",
|
|
|
|
"new foo[bar].baz(baz)()[bar].baz;",
|
|
|
|
"new \"foo\"", // Runtime error
|
|
|
|
"new 1", // Runtime error
|
|
|
|
// This even runs:
|
|
|
|
"(new new Function(\"this.x = 1\")).x;",
|
|
|
|
"new new Test_Two(String, 2).v(0123).length;",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
RunParserSyncTest(context_data, statement_data, kSuccess);
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
TEST(ErrorsNewExpression) {
|
|
|
|
const char* context_data[][2] = {
|
|
|
|
{"", ""},
|
|
|
|
{"var f =", ""},
|
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
|
|
|
|
|
|
|
const char* statement_data[] = {
|
|
|
|
"new foo bar",
|
|
|
|
"new ) foo",
|
|
|
|
"new ++foo",
|
2014-03-21 09:46:18 +00:00
|
|
|
"new foo ++",
|
2014-02-17 15:40:51 +00:00
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
RunParserSyncTest(context_data, statement_data, kError);
|
|
|
|
}
|
2014-03-10 11:42:17 +00:00
|
|
|
|
|
|
|
|
|
|
|
TEST(StrictObjectLiteralChecking) {
|
|
|
|
const char* strict_context_data[][2] = {
|
|
|
|
{"\"use strict\"; var myobject = {", "};"},
|
2014-06-25 10:13:10 +00:00
|
|
|
{"\"use strict\"; var myobject = {", ",};"},
|
2014-03-10 11:42:17 +00:00
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
|
|
|
const char* non_strict_context_data[][2] = {
|
|
|
|
{"var myobject = {", "};"},
|
2014-06-25 10:13:10 +00:00
|
|
|
{"var myobject = {", ",};"},
|
2014-03-10 11:42:17 +00:00
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
|
|
|
|
|
|
|
// These are only errors in strict mode.
|
|
|
|
const char* statement_data[] = {
|
|
|
|
"foo: 1, foo: 2",
|
|
|
|
"\"foo\": 1, \"foo\": 2",
|
|
|
|
"foo: 1, \"foo\": 2",
|
|
|
|
"1: 1, 1: 2",
|
|
|
|
"1: 1, \"1\": 2",
|
|
|
|
"get: 1, get: 2", // Not a getter for real, just a property called get.
|
|
|
|
"set: 1, set: 2", // Not a setter for real, just a property called set.
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
RunParserSyncTest(non_strict_context_data, statement_data, kSuccess);
|
|
|
|
RunParserSyncTest(strict_context_data, statement_data, kError);
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
TEST(ErrorsObjectLiteralChecking) {
|
|
|
|
const char* context_data[][2] = {
|
|
|
|
{"\"use strict\"; var myobject = {", "};"},
|
|
|
|
{"var myobject = {", "};"},
|
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
|
|
|
|
|
|
|
const char* statement_data[] = {
|
2014-06-25 10:13:10 +00:00
|
|
|
",",
|
2014-03-10 11:42:17 +00:00
|
|
|
"foo: 1, get foo() {}",
|
2014-06-17 07:23:26 +00:00
|
|
|
"foo: 1, set foo(v) {}",
|
2014-03-10 11:42:17 +00:00
|
|
|
"\"foo\": 1, get \"foo\"() {}",
|
2014-06-17 07:23:26 +00:00
|
|
|
"\"foo\": 1, set \"foo\"(v) {}",
|
2014-03-10 11:42:17 +00:00
|
|
|
"1: 1, get 1() {}",
|
|
|
|
"1: 1, set 1() {}",
|
|
|
|
// It's counter-intuitive, but these collide too (even in classic
|
|
|
|
// mode). Note that we can have "foo" and foo as properties in classic mode,
|
|
|
|
// but we cannot have "foo" and get foo, or foo and get "foo".
|
|
|
|
"foo: 1, get \"foo\"() {}",
|
2014-06-17 07:23:26 +00:00
|
|
|
"foo: 1, set \"foo\"(v) {}",
|
2014-03-10 11:42:17 +00:00
|
|
|
"\"foo\": 1, get foo() {}",
|
2014-06-17 07:23:26 +00:00
|
|
|
"\"foo\": 1, set foo(v) {}",
|
2014-03-10 11:42:17 +00:00
|
|
|
"1: 1, get \"1\"() {}",
|
|
|
|
"1: 1, set \"1\"() {}",
|
|
|
|
"\"1\": 1, get 1() {}"
|
2014-06-17 07:23:26 +00:00
|
|
|
"\"1\": 1, set 1(v) {}"
|
|
|
|
// Wrong number of parameters
|
|
|
|
"get bar(x) {}",
|
|
|
|
"get bar(x, y) {}",
|
|
|
|
"set bar() {}",
|
|
|
|
"set bar(x, y) {}",
|
2014-03-10 11:42:17 +00:00
|
|
|
// Parsing FunctionLiteral for getter or setter fails
|
|
|
|
"get foo( +",
|
|
|
|
"get foo() \"error\"",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
RunParserSyncTest(context_data, statement_data, kError);
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
TEST(NoErrorsObjectLiteralChecking) {
|
|
|
|
const char* context_data[][2] = {
|
|
|
|
{"var myobject = {", "};"},
|
2014-06-25 10:13:10 +00:00
|
|
|
{"var myobject = {", ",};"},
|
2014-03-10 11:42:17 +00:00
|
|
|
{"\"use strict\"; var myobject = {", "};"},
|
2014-06-25 10:13:10 +00:00
|
|
|
{"\"use strict\"; var myobject = {", ",};"},
|
2014-03-10 11:42:17 +00:00
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
|
|
|
|
|
|
|
const char* statement_data[] = {
|
|
|
|
"foo: 1, bar: 2",
|
|
|
|
"\"foo\": 1, \"bar\": 2",
|
|
|
|
"1: 1, 2: 2",
|
|
|
|
// Syntax: IdentifierName ':' AssignmentExpression
|
|
|
|
"foo: bar = 5 + baz",
|
2014-06-17 07:23:26 +00:00
|
|
|
// Syntax: 'get' PropertyName '(' ')' '{' FunctionBody '}'
|
2014-03-10 11:42:17 +00:00
|
|
|
"get foo() {}",
|
|
|
|
"get \"foo\"() {}",
|
|
|
|
"get 1() {}",
|
2014-06-17 07:23:26 +00:00
|
|
|
// Syntax: 'set' PropertyName '(' PropertySetParameterList ')'
|
|
|
|
// '{' FunctionBody '}'
|
|
|
|
"set foo(v) {}",
|
|
|
|
"set \"foo\"(v) {}",
|
|
|
|
"set 1(v) {}",
|
2014-03-10 11:42:17 +00:00
|
|
|
// Non-colliding getters and setters -> no errors
|
|
|
|
"foo: 1, get bar() {}",
|
2014-06-17 07:23:26 +00:00
|
|
|
"foo: 1, set bar(v) {}",
|
2014-03-10 11:42:17 +00:00
|
|
|
"\"foo\": 1, get \"bar\"() {}",
|
2014-06-17 07:23:26 +00:00
|
|
|
"\"foo\": 1, set \"bar\"(v) {}",
|
2014-03-10 11:42:17 +00:00
|
|
|
"1: 1, get 2() {}",
|
2014-06-17 07:23:26 +00:00
|
|
|
"1: 1, set 2(v) {}",
|
2014-03-10 11:42:17 +00:00
|
|
|
// Keywords, future reserved and strict future reserved are also allowed as
|
|
|
|
// property names.
|
|
|
|
"if: 4",
|
|
|
|
"interface: 5",
|
|
|
|
"super: 6",
|
|
|
|
"eval: 7",
|
|
|
|
"arguments: 8",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
RunParserSyncTest(context_data, statement_data, kSuccess);
|
|
|
|
}
|
2014-03-11 16:30:47 +00:00
|
|
|
|
|
|
|
|
|
|
|
TEST(TooManyArguments) {
|
|
|
|
const char* context_data[][2] = {
|
|
|
|
{"foo(", "0)"},
|
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
|
|
|
|
|
|
|
using v8::internal::Code;
|
2014-03-13 15:55:38 +00:00
|
|
|
char statement[Code::kMaxArguments * 2 + 1];
|
2014-03-11 16:30:47 +00:00
|
|
|
for (int i = 0; i < Code::kMaxArguments; ++i) {
|
|
|
|
statement[2 * i] = '0';
|
|
|
|
statement[2 * i + 1] = ',';
|
|
|
|
}
|
2014-03-13 15:55:38 +00:00
|
|
|
statement[Code::kMaxArguments * 2] = 0;
|
2014-03-11 16:30:47 +00:00
|
|
|
|
|
|
|
const char* statement_data[] = {
|
|
|
|
statement,
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
// The test is quite slow, so run it with a reduced set of flags.
|
|
|
|
static const ParserFlag empty_flags[] = {kAllowLazy};
|
|
|
|
RunParserSyncTest(context_data, statement_data, kError, empty_flags, 1);
|
|
|
|
}
|
2014-03-19 14:08:47 +00:00
|
|
|
|
|
|
|
|
|
|
|
TEST(StrictDelete) {
|
|
|
|
// "delete <Identifier>" is not allowed in strict mode.
|
|
|
|
const char* strict_context_data[][2] = {
|
|
|
|
{"\"use strict\"; ", ""},
|
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
|
|
|
|
|
|
|
const char* sloppy_context_data[][2] = {
|
|
|
|
{"", ""},
|
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
|
|
|
|
|
|
|
// These are errors in the strict mode.
|
|
|
|
const char* sloppy_statement_data[] = {
|
|
|
|
"delete foo;",
|
|
|
|
"delete foo + 1;",
|
|
|
|
"delete (foo);",
|
|
|
|
"delete eval;",
|
|
|
|
"delete interface;",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
// These are always OK
|
|
|
|
const char* good_statement_data[] = {
|
|
|
|
"delete this;",
|
|
|
|
"delete 1;",
|
|
|
|
"delete 1 + 2;",
|
|
|
|
"delete foo();",
|
|
|
|
"delete foo.bar;",
|
|
|
|
"delete foo[bar];",
|
|
|
|
"delete foo--;",
|
|
|
|
"delete --foo;",
|
|
|
|
"delete new foo();",
|
|
|
|
"delete new foo(bar);",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
// These are always errors
|
|
|
|
const char* bad_statement_data[] = {
|
|
|
|
"delete if;",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
RunParserSyncTest(strict_context_data, sloppy_statement_data, kError);
|
|
|
|
RunParserSyncTest(sloppy_context_data, sloppy_statement_data, kSuccess);
|
|
|
|
|
|
|
|
RunParserSyncTest(strict_context_data, good_statement_data, kSuccess);
|
|
|
|
RunParserSyncTest(sloppy_context_data, good_statement_data, kSuccess);
|
|
|
|
|
|
|
|
RunParserSyncTest(strict_context_data, bad_statement_data, kError);
|
|
|
|
RunParserSyncTest(sloppy_context_data, bad_statement_data, kError);
|
|
|
|
}
|
2014-03-20 13:18:15 +00:00
|
|
|
|
|
|
|
|
2014-03-21 09:46:18 +00:00
|
|
|
TEST(InvalidLeftHandSide) {
|
2014-03-20 13:18:15 +00:00
|
|
|
const char* assignment_context_data[][2] = {
|
2014-03-24 12:49:26 +00:00
|
|
|
{"", " = 1;"},
|
|
|
|
{"\"use strict\"; ", " = 1;"},
|
2014-03-20 13:18:15 +00:00
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
|
|
|
|
|
|
|
const char* prefix_context_data[][2] = {
|
|
|
|
{"++", ";"},
|
|
|
|
{"\"use strict\"; ++", ";"},
|
|
|
|
{NULL, NULL},
|
|
|
|
};
|
|
|
|
|
|
|
|
const char* postfix_context_data[][2] = {
|
|
|
|
{"", "++;"},
|
|
|
|
{"\"use strict\"; ", "++;"},
|
|
|
|
{ NULL, NULL }
|
|
|
|
};
|
|
|
|
|
|
|
|
// Good left hand sides for assigment or prefix / postfix operations.
|
|
|
|
const char* good_statement_data[] = {
|
|
|
|
"foo",
|
|
|
|
"foo.bar",
|
|
|
|
"foo[bar]",
|
|
|
|
"foo()[bar]",
|
|
|
|
"foo().bar",
|
|
|
|
"this.foo",
|
|
|
|
"this[foo]",
|
|
|
|
"new foo()[bar]",
|
|
|
|
"new foo().bar",
|
2014-04-02 11:03:05 +00:00
|
|
|
"foo()",
|
|
|
|
"foo(bar)",
|
|
|
|
"foo[bar]()",
|
|
|
|
"foo.bar()",
|
|
|
|
"this()",
|
|
|
|
"this.foo()",
|
|
|
|
"this[foo].bar()",
|
|
|
|
"this.foo[foo].bar(this)(bar)[foo]()",
|
2014-03-20 13:18:15 +00:00
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
// Bad left hand sides for assigment or prefix / postfix operations.
|
|
|
|
const char* bad_statement_data_common[] = {
|
|
|
|
"2",
|
2014-04-02 11:03:05 +00:00
|
|
|
"new foo",
|
|
|
|
"new foo()",
|
2014-03-20 13:18:15 +00:00
|
|
|
"null",
|
|
|
|
"if", // Unexpected token
|
|
|
|
"{x: 1}", // Unexpected token
|
|
|
|
"this",
|
|
|
|
"\"bar\"",
|
|
|
|
"(foo + bar)",
|
|
|
|
"new new foo()[bar]", // means: new (new foo()[bar])
|
|
|
|
"new new foo().bar", // means: new (new foo()[bar])
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
// These are not okay for assignment, but okay for prefix / postix.
|
|
|
|
const char* bad_statement_data_for_assignment[] = {
|
|
|
|
"++foo",
|
|
|
|
"foo++",
|
|
|
|
"foo + bar",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
RunParserSyncTest(assignment_context_data, good_statement_data, kSuccess);
|
|
|
|
RunParserSyncTest(assignment_context_data, bad_statement_data_common, kError);
|
|
|
|
RunParserSyncTest(assignment_context_data, bad_statement_data_for_assignment,
|
|
|
|
kError);
|
|
|
|
|
|
|
|
RunParserSyncTest(prefix_context_data, good_statement_data, kSuccess);
|
|
|
|
RunParserSyncTest(prefix_context_data, bad_statement_data_common, kError);
|
|
|
|
|
|
|
|
RunParserSyncTest(postfix_context_data, good_statement_data, kSuccess);
|
2014-03-21 09:46:18 +00:00
|
|
|
RunParserSyncTest(postfix_context_data, bad_statement_data_common, kError);
|
2014-03-20 13:18:15 +00:00
|
|
|
}
|
2014-06-03 15:02:16 +00:00
|
|
|
|
|
|
|
|
|
|
|
TEST(FuncNameInferrerBasic) {
|
|
|
|
// Tests that function names are inferred properly.
|
|
|
|
i::FLAG_allow_natives_syntax = true;
|
|
|
|
v8::Isolate* isolate = CcTest::isolate();
|
|
|
|
v8::HandleScope scope(isolate);
|
|
|
|
LocalContext env;
|
|
|
|
CompileRun("var foo1 = function() {}; "
|
|
|
|
"var foo2 = function foo3() {}; "
|
|
|
|
"function not_ctor() { "
|
|
|
|
" var foo4 = function() {}; "
|
|
|
|
" return %FunctionGetInferredName(foo4); "
|
|
|
|
"} "
|
|
|
|
"function Ctor() { "
|
|
|
|
" var foo5 = function() {}; "
|
|
|
|
" return %FunctionGetInferredName(foo5); "
|
|
|
|
"} "
|
|
|
|
"var obj1 = { foo6: function() {} }; "
|
|
|
|
"var obj2 = { 'foo7': function() {} }; "
|
|
|
|
"var obj3 = {}; "
|
|
|
|
"obj3[1] = function() {}; "
|
|
|
|
"var obj4 = {}; "
|
|
|
|
"obj4[1] = function foo8() {}; "
|
|
|
|
"var obj5 = {}; "
|
|
|
|
"obj5['foo9'] = function() {}; "
|
|
|
|
"var obj6 = { obj7 : { foo10: function() {} } };");
|
|
|
|
ExpectString("%FunctionGetInferredName(foo1)", "foo1");
|
|
|
|
// foo2 is not unnamed -> its name is not inferred.
|
|
|
|
ExpectString("%FunctionGetInferredName(foo2)", "");
|
|
|
|
ExpectString("not_ctor()", "foo4");
|
|
|
|
ExpectString("Ctor()", "Ctor.foo5");
|
|
|
|
ExpectString("%FunctionGetInferredName(obj1.foo6)", "obj1.foo6");
|
|
|
|
ExpectString("%FunctionGetInferredName(obj2.foo7)", "obj2.foo7");
|
|
|
|
ExpectString("%FunctionGetInferredName(obj3[1])",
|
|
|
|
"obj3.(anonymous function)");
|
|
|
|
ExpectString("%FunctionGetInferredName(obj4[1])", "");
|
|
|
|
ExpectString("%FunctionGetInferredName(obj5['foo9'])", "obj5.foo9");
|
|
|
|
ExpectString("%FunctionGetInferredName(obj6.obj7.foo10)", "obj6.obj7.foo10");
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
TEST(FuncNameInferrerTwoByte) {
|
|
|
|
// Tests function name inferring in cases where some parts of the inferred
|
|
|
|
// function name are two-byte strings.
|
|
|
|
i::FLAG_allow_natives_syntax = true;
|
|
|
|
v8::Isolate* isolate = CcTest::isolate();
|
|
|
|
v8::HandleScope scope(isolate);
|
|
|
|
LocalContext env;
|
|
|
|
uint16_t* two_byte_source = AsciiToTwoByteString(
|
|
|
|
"var obj1 = { oXj2 : { foo1: function() {} } }; "
|
|
|
|
"%FunctionGetInferredName(obj1.oXj2.foo1)");
|
|
|
|
uint16_t* two_byte_name = AsciiToTwoByteString("obj1.oXj2.foo1");
|
|
|
|
// Make it really non-ASCII (replace the Xs with a non-ASCII character).
|
|
|
|
two_byte_source[14] = two_byte_source[78] = two_byte_name[6] = 0x010d;
|
|
|
|
v8::Local<v8::String> source =
|
|
|
|
v8::String::NewFromTwoByte(isolate, two_byte_source);
|
|
|
|
v8::Local<v8::Value> result = CompileRun(source);
|
|
|
|
CHECK(result->IsString());
|
|
|
|
v8::Local<v8::String> expected_name =
|
|
|
|
v8::String::NewFromTwoByte(isolate, two_byte_name);
|
|
|
|
CHECK(result->Equals(expected_name));
|
2014-06-03 15:39:47 +00:00
|
|
|
i::DeleteArray(two_byte_source);
|
|
|
|
i::DeleteArray(two_byte_name);
|
2014-06-03 15:02:16 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
TEST(FuncNameInferrerEscaped) {
|
|
|
|
// The same as FuncNameInferrerTwoByte, except that we express the two-byte
|
|
|
|
// character as a unicode escape.
|
|
|
|
i::FLAG_allow_natives_syntax = true;
|
|
|
|
v8::Isolate* isolate = CcTest::isolate();
|
|
|
|
v8::HandleScope scope(isolate);
|
|
|
|
LocalContext env;
|
|
|
|
uint16_t* two_byte_source = AsciiToTwoByteString(
|
|
|
|
"var obj1 = { o\\u010dj2 : { foo1: function() {} } }; "
|
|
|
|
"%FunctionGetInferredName(obj1.o\\u010dj2.foo1)");
|
|
|
|
uint16_t* two_byte_name = AsciiToTwoByteString("obj1.oXj2.foo1");
|
|
|
|
// Fix to correspond to the non-ASCII name in two_byte_source.
|
|
|
|
two_byte_name[6] = 0x010d;
|
|
|
|
v8::Local<v8::String> source =
|
|
|
|
v8::String::NewFromTwoByte(isolate, two_byte_source);
|
|
|
|
v8::Local<v8::Value> result = CompileRun(source);
|
|
|
|
CHECK(result->IsString());
|
|
|
|
v8::Local<v8::String> expected_name =
|
|
|
|
v8::String::NewFromTwoByte(isolate, two_byte_name);
|
|
|
|
CHECK(result->Equals(expected_name));
|
2014-06-03 15:39:47 +00:00
|
|
|
i::DeleteArray(two_byte_source);
|
|
|
|
i::DeleteArray(two_byte_name);
|
2014-06-03 15:02:16 +00:00
|
|
|
}
|
2014-06-24 14:03:24 +00:00
|
|
|
|
|
|
|
|
|
|
|
TEST(RegressionLazyFunctionWithErrorWithArg) {
|
|
|
|
// The bug occurred when a lazy function had an error which requires a
|
|
|
|
// parameter (such as "unknown label" here). The error message was processed
|
|
|
|
// before the AstValueFactory containing the error message string was
|
|
|
|
// internalized.
|
|
|
|
v8::Isolate* isolate = CcTest::isolate();
|
|
|
|
v8::HandleScope scope(isolate);
|
|
|
|
LocalContext env;
|
|
|
|
i::FLAG_lazy = true;
|
|
|
|
i::FLAG_min_preparse_length = 0;
|
|
|
|
CompileRun("function this_is_lazy() {\n"
|
|
|
|
" break p;\n"
|
|
|
|
"}\n"
|
|
|
|
"this_is_lazy();\n");
|
|
|
|
}
|
2014-06-26 11:59:42 +00:00
|
|
|
|
|
|
|
|
|
|
|
TEST(InnerAssignment) {
|
|
|
|
i::Isolate* isolate = CcTest::i_isolate();
|
|
|
|
i::Factory* factory = isolate->factory();
|
|
|
|
i::HandleScope scope(isolate);
|
|
|
|
LocalContext env;
|
|
|
|
|
|
|
|
const char* prefix = "function f() {";
|
|
|
|
const char* midfix = " function g() {";
|
|
|
|
const char* suffix = "}}";
|
|
|
|
struct { const char* source; bool assigned; bool strict; } outers[] = {
|
|
|
|
// Actual assignments.
|
|
|
|
{ "var x; var x = 5;", true, false },
|
|
|
|
{ "var x; { var x = 5; }", true, false },
|
|
|
|
{ "'use strict'; let x; x = 6;", true, true },
|
|
|
|
{ "var x = 5; function x() {}", true, false },
|
|
|
|
// Actual non-assignments.
|
|
|
|
{ "var x;", false, false },
|
|
|
|
{ "var x = 5;", false, false },
|
|
|
|
{ "'use strict'; let x;", false, true },
|
|
|
|
{ "'use strict'; let x = 6;", false, true },
|
|
|
|
{ "'use strict'; var x = 0; { let x = 6; }", false, true },
|
|
|
|
{ "'use strict'; var x = 0; { let x; x = 6; }", false, true },
|
|
|
|
{ "'use strict'; let x = 0; { let x = 6; }", false, true },
|
|
|
|
{ "'use strict'; let x = 0; { let x; x = 6; }", false, true },
|
|
|
|
{ "var x; try {} catch (x) { x = 5; }", false, false },
|
|
|
|
{ "function x() {}", false, false },
|
|
|
|
// Eval approximation.
|
|
|
|
{ "var x; eval('');", true, false },
|
|
|
|
{ "eval(''); var x;", true, false },
|
|
|
|
{ "'use strict'; let x; eval('');", true, true },
|
|
|
|
{ "'use strict'; eval(''); let x;", true, true },
|
|
|
|
// Non-assignments not recognized, because the analysis is approximative.
|
|
|
|
{ "var x; var x;", true, false },
|
|
|
|
{ "var x = 5; var x;", true, false },
|
|
|
|
{ "var x; { var x; }", true, false },
|
|
|
|
{ "var x; function x() {}", true, false },
|
|
|
|
{ "function x() {}; var x;", true, false },
|
|
|
|
{ "var x; try {} catch (x) { var x = 5; }", true, false },
|
|
|
|
};
|
|
|
|
struct { const char* source; bool assigned; bool with; } inners[] = {
|
|
|
|
// Actual assignments.
|
|
|
|
{ "x = 1;", true, false },
|
|
|
|
{ "x++;", true, false },
|
|
|
|
{ "++x;", true, false },
|
|
|
|
{ "x--;", true, false },
|
|
|
|
{ "--x;", true, false },
|
|
|
|
{ "{ x = 1; }", true, false },
|
|
|
|
{ "'use strict'; { let x; }; x = 0;", true, false },
|
|
|
|
{ "'use strict'; { const x = 1; }; x = 0;", true, false },
|
|
|
|
{ "'use strict'; { function x() {} }; x = 0;", true, false },
|
|
|
|
{ "with ({}) { x = 1; }", true, true },
|
|
|
|
{ "eval('');", true, false },
|
|
|
|
{ "'use strict'; { let y; eval('') }", true, false },
|
|
|
|
{ "function h() { x = 0; }", true, false },
|
|
|
|
{ "(function() { x = 0; })", true, false },
|
|
|
|
{ "(function() { x = 0; })", true, false },
|
|
|
|
{ "with ({}) (function() { x = 0; })", true, true },
|
|
|
|
// Actual non-assignments.
|
|
|
|
{ "", false, false },
|
|
|
|
{ "x;", false, false },
|
|
|
|
{ "var x;", false, false },
|
|
|
|
{ "var x = 8;", false, false },
|
|
|
|
{ "var x; x = 8;", false, false },
|
|
|
|
{ "'use strict'; let x;", false, false },
|
|
|
|
{ "'use strict'; let x = 8;", false, false },
|
|
|
|
{ "'use strict'; let x; x = 8;", false, false },
|
|
|
|
{ "'use strict'; const x = 8;", false, false },
|
|
|
|
{ "function x() {}", false, false },
|
|
|
|
{ "function x() { x = 0; }", false, false },
|
|
|
|
{ "function h(x) { x = 0; }", false, false },
|
|
|
|
{ "'use strict'; { let x; x = 0; }", false, false },
|
|
|
|
{ "{ var x; }; x = 0;", false, false },
|
|
|
|
{ "with ({}) {}", false, true },
|
|
|
|
{ "var x; { with ({}) { x = 1; } }", false, true },
|
|
|
|
{ "try {} catch(x) { x = 0; }", false, false },
|
|
|
|
{ "try {} catch(x) { with ({}) { x = 1; } }", false, true },
|
|
|
|
// Eval approximation.
|
|
|
|
{ "eval('');", true, false },
|
|
|
|
{ "function h() { eval(''); }", true, false },
|
|
|
|
{ "(function() { eval(''); })", true, false },
|
|
|
|
// Shadowing not recognized because of eval approximation.
|
|
|
|
{ "var x; eval('');", true, false },
|
|
|
|
{ "'use strict'; let x; eval('');", true, false },
|
|
|
|
{ "try {} catch(x) { eval(''); }", true, false },
|
|
|
|
{ "function x() { eval(''); }", true, false },
|
|
|
|
{ "(function(x) { eval(''); })", true, false },
|
|
|
|
};
|
|
|
|
|
|
|
|
int prefix_len = Utf8LengthHelper(prefix);
|
|
|
|
int midfix_len = Utf8LengthHelper(midfix);
|
|
|
|
int suffix_len = Utf8LengthHelper(suffix);
|
|
|
|
for (unsigned i = 0; i < ARRAY_SIZE(outers); ++i) {
|
|
|
|
const char* outer = outers[i].source;
|
|
|
|
int outer_len = Utf8LengthHelper(outer);
|
|
|
|
for (unsigned j = 0; j < ARRAY_SIZE(inners); ++j) {
|
|
|
|
if (outers[i].strict && inners[j].with) continue;
|
|
|
|
const char* inner = inners[j].source;
|
|
|
|
int inner_len = Utf8LengthHelper(inner);
|
|
|
|
int len = prefix_len + outer_len + midfix_len + inner_len + suffix_len;
|
|
|
|
i::ScopedVector<char> program(len + 1);
|
|
|
|
i::SNPrintF(program, "%s%s%s%s%s", prefix, outer, midfix, inner, suffix);
|
|
|
|
i::Handle<i::String> source =
|
|
|
|
factory->InternalizeUtf8String(program.start());
|
|
|
|
source->PrintOn(stdout);
|
|
|
|
printf("\n");
|
|
|
|
|
|
|
|
i::Handle<i::Script> script = factory->NewScript(source);
|
|
|
|
i::CompilationInfoWithZone info(script);
|
|
|
|
i::Parser parser(&info);
|
|
|
|
parser.set_allow_harmony_scoping(true);
|
|
|
|
CHECK(parser.Parse());
|
|
|
|
CHECK(i::Rewriter::Rewrite(&info));
|
|
|
|
CHECK(i::Scope::Analyze(&info));
|
|
|
|
CHECK(info.function() != NULL);
|
|
|
|
|
|
|
|
i::Scope* scope = info.function()->scope();
|
|
|
|
CHECK_EQ(scope->inner_scopes()->length(), 1);
|
|
|
|
i::Scope* inner_scope = scope->inner_scopes()->at(0);
|
|
|
|
const i::AstRawString* var_name =
|
|
|
|
info.ast_value_factory()->GetOneByteString("x");
|
|
|
|
i::Variable* var = inner_scope->Lookup(var_name);
|
|
|
|
bool expected = outers[i].assigned || inners[j].assigned;
|
|
|
|
CHECK(var != NULL);
|
|
|
|
CHECK(var->is_used() || !expected);
|
|
|
|
CHECK(var->maybe_assigned() == expected);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
2014-06-30 13:35:16 +00:00
|
|
|
|
|
|
|
namespace {
|
|
|
|
|
|
|
|
int* global_use_counts = NULL;
|
|
|
|
|
|
|
|
void MockUseCounterCallback(v8::Isolate* isolate,
|
|
|
|
v8::Isolate::UseCounterFeature feature) {
|
|
|
|
++global_use_counts[feature];
|
|
|
|
}
|
|
|
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
TEST(UseAsmUseCount) {
|
|
|
|
i::Isolate* isolate = CcTest::i_isolate();
|
|
|
|
i::HandleScope scope(isolate);
|
|
|
|
LocalContext env;
|
|
|
|
int use_counts[v8::Isolate::kUseCounterFeatureCount] = {};
|
|
|
|
global_use_counts = use_counts;
|
|
|
|
CcTest::isolate()->SetUseCounterCallback(MockUseCounterCallback);
|
|
|
|
CompileRun("\"use asm\";\n"
|
|
|
|
"var foo = 1;\n"
|
|
|
|
"\"use asm\";\n" // Only the first one counts.
|
|
|
|
"function bar() { \"use asm\"; var baz = 1; }");
|
|
|
|
CHECK_EQ(2, use_counts[v8::Isolate::kUseAsm]);
|
|
|
|
}
|
Implement handling of arrow functions in the parser
Arrow functions are parsed from ParseAssignmentExpression(). Handling the
parameter list is done by letting ParseConditionalExpression() parse a comma
separated list of identifiers, and it returns a tree of BinaryOperation nodes
with VariableProxy leaves, or a single VariableProxy if there is only one
parameter. When the arrow token "=>" is found, the VariableProxy nodes are
passed to ParseArrowFunctionLiteral(), which will then skip parsing the
paramaeter list. This avoids having to rewind when the arrow is found and
restart parsing the parameter list.
Note that the empty parameter list "()" is handled directly in
ParsePrimaryExpression(): after is has consumed the opening parenthesis,
if a closing parenthesis follows, then the only valid input is an arrow
function. In this case, ParsePrimaryExpression() directly calls
ParseArrowFunctionLiteral(), to avoid needing to return a sentinel value
to signal the empty parameter list. Because it will consume the body of
the arrow function, ParseAssignmentExpression() will not see the arrow
"=>" token as next, and return the already-parser expression.
The implementation is done in ParserBase, so it was needed to do some
additions to ParserBase, ParserTraits and PreParserTraits. Some of the
glue code can be removed later on when more more functionality is moved
to ParserBase.
Additionally, this adds a runtime flag "harmony_arrow_functions"
(disabled by default); enabling "harmony" will enable it as well.
BUG=v8:2700
LOG=N
R=marja@chromium.org
Review URL: https://codereview.chromium.org/383983002
Patch from Adrián Pérez de Castro <aperez@igalia.com>.
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22366 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-14 07:55:45 +00:00
|
|
|
|
|
|
|
|
|
|
|
TEST(ErrorsArrowFunctions) {
|
|
|
|
// Tests that parser and preparser generate the same kind of errors
|
|
|
|
// on invalid arrow function syntax.
|
|
|
|
const char* context_data[][2] = {
|
|
|
|
{"", ";"},
|
|
|
|
{"v = ", ";"},
|
|
|
|
{"bar ? (", ") : baz;"},
|
|
|
|
{"bar ? baz : (", ");"},
|
|
|
|
{"bar[", "];"},
|
|
|
|
{"bar, ", ";"},
|
|
|
|
{"", ", bar;"},
|
|
|
|
{NULL, NULL}
|
|
|
|
};
|
|
|
|
|
|
|
|
const char* statement_data[] = {
|
|
|
|
"=> 0",
|
|
|
|
"=>",
|
|
|
|
"() =>",
|
|
|
|
"=> {}",
|
|
|
|
") => {}",
|
|
|
|
", => {}",
|
|
|
|
"(,) => {}",
|
|
|
|
"return => {}",
|
|
|
|
"() => {'value': 42}",
|
|
|
|
|
|
|
|
// Check that the early return introduced in ParsePrimaryExpression
|
|
|
|
// does not accept stray closing parentheses.
|
|
|
|
")",
|
|
|
|
") => 0",
|
|
|
|
"foo[()]",
|
|
|
|
"()",
|
|
|
|
|
|
|
|
// Parameter lists with extra parens should be recognized as errors.
|
|
|
|
"(()) => 0",
|
|
|
|
"((x)) => 0",
|
|
|
|
"((x, y)) => 0",
|
|
|
|
"(x, (y)) => 0",
|
|
|
|
"((x, y, z)) => 0",
|
|
|
|
"(x, (y, z)) => 0",
|
|
|
|
"((x, y), z) => 0",
|
|
|
|
|
|
|
|
// Parameter lists are always validated as strict, so those are errors.
|
|
|
|
"eval => {}",
|
|
|
|
"arguments => {}",
|
|
|
|
"yield => {}",
|
|
|
|
"interface => {}",
|
|
|
|
"(eval) => {}",
|
|
|
|
"(arguments) => {}",
|
|
|
|
"(yield) => {}",
|
|
|
|
"(interface) => {}",
|
|
|
|
"(eval, bar) => {}",
|
|
|
|
"(bar, eval) => {}",
|
|
|
|
"(bar, arguments) => {}",
|
|
|
|
"(bar, yield) => {}",
|
|
|
|
"(bar, interface) => {}",
|
|
|
|
// TODO(aperez): Detecting duplicates does not work in PreParser.
|
|
|
|
// "(bar, bar) => {}",
|
|
|
|
|
|
|
|
// The parameter list is parsed as an expression, but only
|
|
|
|
// a comma-separated list of identifier is valid.
|
|
|
|
"32 => {}",
|
|
|
|
"(32) => {}",
|
|
|
|
"(a, 32) => {}",
|
|
|
|
"if => {}",
|
|
|
|
"(if) => {}",
|
|
|
|
"(a, if) => {}",
|
|
|
|
"a + b => {}",
|
|
|
|
"(a + b) => {}",
|
|
|
|
"(a + b, c) => {}",
|
|
|
|
"(a, b - c) => {}",
|
|
|
|
"\"a\" => {}",
|
|
|
|
"(\"a\") => {}",
|
|
|
|
"(\"a\", b) => {}",
|
|
|
|
"(a, \"b\") => {}",
|
|
|
|
"-a => {}",
|
|
|
|
"(-a) => {}",
|
|
|
|
"(-a, b) => {}",
|
|
|
|
"(a, -b) => {}",
|
|
|
|
"{} => {}",
|
|
|
|
"({}) => {}",
|
|
|
|
"(a, {}) => {}",
|
|
|
|
"({}, a) => {}",
|
|
|
|
"a++ => {}",
|
|
|
|
"(a++) => {}",
|
|
|
|
"(a++, b) => {}",
|
|
|
|
"(a, b++) => {}",
|
|
|
|
"[] => {}",
|
|
|
|
"([]) => {}",
|
|
|
|
"(a, []) => {}",
|
|
|
|
"([], a) => {}",
|
|
|
|
"(a = b) => {}",
|
|
|
|
"(a = b, c) => {}",
|
|
|
|
"(a, b = c) => {}",
|
|
|
|
"(foo ? bar : baz) => {}",
|
|
|
|
"(a, foo ? bar : baz) => {}",
|
|
|
|
"(foo ? bar : baz, a) => {}",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
2014-07-15 08:10:26 +00:00
|
|
|
// The test is quite slow, so run it with a reduced set of flags.
|
|
|
|
static const ParserFlag flags[] = {
|
|
|
|
kAllowLazy, kAllowHarmonyScoping, kAllowGenerators
|
|
|
|
};
|
|
|
|
static const ParserFlag always_flags[] = { kAllowArrowFunctions };
|
|
|
|
RunParserSyncTest(context_data, statement_data, kError, flags,
|
|
|
|
ARRAY_SIZE(flags), always_flags, ARRAY_SIZE(always_flags));
|
Implement handling of arrow functions in the parser
Arrow functions are parsed from ParseAssignmentExpression(). Handling the
parameter list is done by letting ParseConditionalExpression() parse a comma
separated list of identifiers, and it returns a tree of BinaryOperation nodes
with VariableProxy leaves, or a single VariableProxy if there is only one
parameter. When the arrow token "=>" is found, the VariableProxy nodes are
passed to ParseArrowFunctionLiteral(), which will then skip parsing the
paramaeter list. This avoids having to rewind when the arrow is found and
restart parsing the parameter list.
Note that the empty parameter list "()" is handled directly in
ParsePrimaryExpression(): after is has consumed the opening parenthesis,
if a closing parenthesis follows, then the only valid input is an arrow
function. In this case, ParsePrimaryExpression() directly calls
ParseArrowFunctionLiteral(), to avoid needing to return a sentinel value
to signal the empty parameter list. Because it will consume the body of
the arrow function, ParseAssignmentExpression() will not see the arrow
"=>" token as next, and return the already-parser expression.
The implementation is done in ParserBase, so it was needed to do some
additions to ParserBase, ParserTraits and PreParserTraits. Some of the
glue code can be removed later on when more more functionality is moved
to ParserBase.
Additionally, this adds a runtime flag "harmony_arrow_functions"
(disabled by default); enabling "harmony" will enable it as well.
BUG=v8:2700
LOG=N
R=marja@chromium.org
Review URL: https://codereview.chromium.org/383983002
Patch from Adrián Pérez de Castro <aperez@igalia.com>.
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22366 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
2014-07-14 07:55:45 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
TEST(NoErrorsArrowFunctions) {
|
|
|
|
// Tests that parser and preparser accept valid arrow functions syntax.
|
|
|
|
const char* context_data[][2] = {
|
|
|
|
{"", ";"},
|
|
|
|
{"bar ? (", ") : baz;"},
|
|
|
|
{"bar ? baz : (", ");"},
|
|
|
|
{"bar, ", ";"},
|
|
|
|
{"", ", bar;"},
|
|
|
|
{NULL, NULL}
|
|
|
|
};
|
|
|
|
|
|
|
|
const char* statement_data[] = {
|
|
|
|
"() => {}",
|
|
|
|
"() => { return 42 }",
|
|
|
|
"x => { return x; }",
|
|
|
|
"(x) => { return x; }",
|
|
|
|
"(x, y) => { return x + y; }",
|
|
|
|
"(x, y, z) => { return x + y + z; }",
|
|
|
|
"(x, y) => { x.a = y; }",
|
|
|
|
"() => 42",
|
|
|
|
"x => x",
|
|
|
|
"x => x * x",
|
|
|
|
"(x) => x",
|
|
|
|
"(x) => x * x",
|
|
|
|
"(x, y) => x + y",
|
|
|
|
"(x, y, z) => x, y, z",
|
|
|
|
"(x, y) => x.a = y",
|
|
|
|
"() => ({'value': 42})",
|
|
|
|
"x => y => x + y",
|
|
|
|
"(x, y) => (u, v) => x*u + y*v",
|
|
|
|
"(x, y) => z => z * (x + y)",
|
|
|
|
"x => (y, z) => z * (x + y)",
|
|
|
|
|
|
|
|
// Those are comma-separated expressions, with arrow functions as items.
|
|
|
|
// They stress the code for validating arrow function parameter lists.
|
|
|
|
"a, b => 0",
|
|
|
|
"a, b, (c, d) => 0",
|
|
|
|
"(a, b, (c, d) => 0)",
|
|
|
|
"(a, b) => 0, (c, d) => 1",
|
|
|
|
"(a, b => {}, a => a + 1)",
|
|
|
|
"((a, b) => {}, (a => a + 1))",
|
|
|
|
"(a, (a, (b, c) => 0))",
|
|
|
|
|
|
|
|
// Arrow has more precedence, this is the same as: foo ? bar : (baz = {})
|
|
|
|
"foo ? bar : baz => {}",
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
static const ParserFlag always_flags[] = {kAllowArrowFunctions};
|
|
|
|
RunParserSyncTest(context_data, statement_data, kSuccess, NULL, 0,
|
|
|
|
always_flags, ARRAY_SIZE(always_flags));
|
|
|
|
}
|