Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(825)

Unified Diff: src/preparser.h

Issue 177683002: Mode clean-up pt 1: rename classic/non-strict mode to sloppy mode (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 6 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« src/globals.h ('K') | « src/parser.cc ('k') | src/preparser.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/preparser.h
diff --git a/src/preparser.h b/src/preparser.h
index f392304be826d985e4df1c4e2e8ff4694b4b04ee..14b801a560f574d7e7798f7f3c2a907dffea978a 100644
--- a/src/preparser.h
+++ b/src/preparser.h
@@ -297,7 +297,7 @@ class ParserBase : public Traits {
return function_state_->factory();
}
- bool is_classic_mode() const { return scope_->is_classic_mode(); }
+ bool is_sloppy_mode() const { return scope_->is_sloppy_mode(); }
bool is_generator() const { return function_state_->is_generator(); }
@@ -568,13 +568,13 @@ class PreParserScope {
language_mode_ = outer_scope->language_mode();
} else {
scope_inside_with_ = is_with_scope();
- language_mode_ = CLASSIC_MODE;
+ language_mode_ = SLOPPY_MODE;
}
}
bool is_with_scope() const { return scope_type_ == WITH_SCOPE; }
- bool is_classic_mode() const {
- return language_mode() == CLASSIC_MODE;
+ bool is_sloppy_mode() const {
+ return language_mode() == SLOPPY_MODE;
}
bool is_extended_mode() {
return language_mode() == EXTENDED_MODE;
@@ -763,7 +763,7 @@ class PreParser : public ParserBase<PreParserTraits> {
if (stack_overflow()) return kPreParseStackOverflow;
if (!ok) {
ReportUnexpectedToken(scanner()->current_token());
- } else if (!scope_->is_classic_mode()) {
+ } else if (!scope_->is_sloppy_mode()) {
CheckOctalLiteral(start_position, scanner()->location().end_pos, &ok);
}
return kPreParseSuccess;
@@ -974,7 +974,7 @@ void ParserBase<Traits>::ReportUnexpectedToken(Token::Value token) {
case Token::YIELD:
case Token::FUTURE_STRICT_RESERVED_WORD:
return ReportMessageAt(source_location,
- is_classic_mode() ? "unexpected_token_identifier"
+ is_sloppy_mode() ? "unexpected_token_identifier"
: "unexpected_strict_reserved");
Michael Starzinger 2014/02/24 15:09:14 nit: Indentation is off.
default:
const char* name = Token::String(token);
@@ -993,12 +993,12 @@ typename Traits::Type::Identifier ParserBase<Traits>::ParseIdentifier(
if (next == Token::IDENTIFIER) {
typename Traits::Type::Identifier name = this->GetSymbol(scanner());
if (allow_eval_or_arguments == kDontAllowEvalOrArguments &&
- !is_classic_mode() && this->IsEvalOrArguments(name)) {
+ !is_sloppy_mode() && this->IsEvalOrArguments(name)) {
ReportMessageAt(scanner()->location(), "strict_eval_arguments");
*ok = false;
}
return name;
- } else if (is_classic_mode() && (next == Token::FUTURE_STRICT_RESERVED_WORD ||
+ } else if (is_sloppy_mode() && (next == Token::FUTURE_STRICT_RESERVED_WORD ||
(next == Token::YIELD && !is_generator()))) {
Michael Starzinger 2014/02/24 15:09:14 nit: Indentation is off.
return this->GetSymbol(scanner());
} else {
@@ -1258,7 +1258,7 @@ void ParserBase<Traits>::ObjectLiteralChecker::CheckProperty(
if (HasConflict(old_type, type)) {
if (IsDataDataConflict(old_type, type)) {
// Both are data properties.
- if (language_mode_ == CLASSIC_MODE) return;
+ if (language_mode_ == SLOPPY_MODE) return;
parser()->ReportMessageAt(scanner()->location(),
"strict_duplicate_property");
} else if (IsDataAccessorConflict(old_type, type)) {
« src/globals.h ('K') | « src/parser.cc ('k') | src/preparser.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698