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

Side by Side Diff: src/parsing/preparser.h

Issue 2690723002: [cleanup] Remove redundant block_scope argument from DeclareClassVariable (Closed)
Patch Set: Created 3 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 unified diff | Download patch
« no previous file with comments | « src/parsing/parser-base.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef V8_PARSING_PREPARSER_H 5 #ifndef V8_PARSING_PREPARSER_H
6 #define V8_PARSING_PREPARSER_H 6 #define V8_PARSING_PREPARSER_H
7 7
8 #include "src/ast/ast.h" 8 #include "src/ast/ast.h"
9 #include "src/ast/scopes.h" 9 #include "src/ast/scopes.h"
10 #include "src/parsing/parser-base.h" 10 #include "src/parsing/parser-base.h"
(...skipping 1120 matching lines...) Expand 10 before | Expand all | Expand 10 after
1131 int end_pos, bool* ok) { 1131 int end_pos, bool* ok) {
1132 // Preparser shouldn't be used in contexts where we need to track the names. 1132 // Preparser shouldn't be used in contexts where we need to track the names.
1133 DCHECK_NULL(names); 1133 DCHECK_NULL(names);
1134 if (variable_name.string_ != nullptr) { 1134 if (variable_name.string_ != nullptr) {
1135 DCHECK(track_unresolved_variables_); 1135 DCHECK(track_unresolved_variables_);
1136 scope()->DeclareVariableName(variable_name.string_, LET); 1136 scope()->DeclareVariableName(variable_name.string_, LET);
1137 } 1137 }
1138 return PreParserStatement::Default(); 1138 return PreParserStatement::Default();
1139 } 1139 }
1140 V8_INLINE void DeclareClassVariable(PreParserIdentifier name, 1140 V8_INLINE void DeclareClassVariable(PreParserIdentifier name,
1141 Scope* block_scope, ClassInfo* class_info, 1141 ClassInfo* class_info,
1142 int class_token_pos, bool* ok) {} 1142 int class_token_pos, bool* ok) {}
1143 V8_INLINE void DeclareClassProperty(PreParserIdentifier class_name, 1143 V8_INLINE void DeclareClassProperty(PreParserIdentifier class_name,
1144 PreParserExpression property, 1144 PreParserExpression property,
1145 ClassLiteralProperty::Kind kind, 1145 ClassLiteralProperty::Kind kind,
1146 bool is_static, bool is_constructor, 1146 bool is_static, bool is_constructor,
1147 ClassInfo* class_info, bool* ok) { 1147 ClassInfo* class_info, bool* ok) {
1148 } 1148 }
1149 V8_INLINE PreParserExpression RewriteClassLiteral(PreParserIdentifier name, 1149 V8_INLINE PreParserExpression RewriteClassLiteral(PreParserIdentifier name,
1150 ClassInfo* class_info, 1150 ClassInfo* class_info,
1151 int pos, bool* ok) { 1151 int pos, bool* ok) {
(...skipping 548 matching lines...) Expand 10 before | Expand all | Expand 10 after
1700 function_state_->NextMaterializedLiteralIndex(); 1700 function_state_->NextMaterializedLiteralIndex();
1701 function_state_->NextMaterializedLiteralIndex(); 1701 function_state_->NextMaterializedLiteralIndex();
1702 } 1702 }
1703 return EmptyExpression(); 1703 return EmptyExpression();
1704 } 1704 }
1705 1705
1706 } // namespace internal 1706 } // namespace internal
1707 } // namespace v8 1707 } // namespace v8
1708 1708
1709 #endif // V8_PARSING_PREPARSER_H 1709 #endif // V8_PARSING_PREPARSER_H
OLDNEW
« no previous file with comments | « src/parsing/parser-base.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698