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

Unified Diff: pkg/kernel/binary.md

Issue 2680303002: Kernel debugging; service tests (Closed)
Patch Set: New failing test 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 side-by-side diff with in-line comments
Download patch
Index: pkg/kernel/binary.md
diff --git a/pkg/kernel/binary.md b/pkg/kernel/binary.md
index a287dd6be83b9b2884a61d04c12ab347a3fb7af5..dbaf989751383e5feebfbaa0d3204a5f6691946a 100644
--- a/pkg/kernel/binary.md
+++ b/pkg/kernel/binary.md
@@ -435,6 +435,7 @@ type StaticGet extends Expression {
type StaticSet extends Expression {
Byte tag = 27;
+ FileOffset fileOffset;
MemberReference target;
Expression value;
}
@@ -545,6 +546,7 @@ type IsExpression extends Expression {
type AsExpression extends Expression {
Byte tag = 38;
+ FileOffset fileOffset;
Expression operand;
DartType type;
}
@@ -617,12 +619,14 @@ type Throw extends Expression {
type ListLiteral extends Expression {
Byte tag = 49;
+ FileOffset fileOffset;
DartType typeArgument;
List<Expression> values;
}
type ConstListLiteral extends Expression {
Byte tag = 58; // Note: tag is out of order.
+ FileOffset fileOffset;
DartType typeArgument;
List<Expression> values;
}
@@ -697,6 +701,7 @@ type LabeledStatement extends Statement {
type BreakStatement extends Statement {
Byte tag = 66;
+ FileOffset fileOffset;
// Reference to the Nth LabeledStatement in scope, with 0 being the
// outermost enclosing labeled statement within the same FunctionNode.
@@ -813,6 +818,7 @@ type VariableDeclarationStatement extends Statement {
type VariableDeclaration {
FileOffset fileOffset;
Kevin Millikin (Google) 2017/02/08 15:37:51 Document these two offsets here.
jensj 2017/02/13 14:04:15 Done.
+ FileOffset fileEqualsOffset;
Byte flags (isFinal, isConst);
// For named parameters, this is the parameter name.
// For other variables, the name is cosmetic, may be empty,
« no previous file with comments | « no previous file | pkg/kernel/lib/analyzer/ast_from_analyzer.dart » ('j') | pkg/kernel/lib/analyzer/ast_from_analyzer.dart » ('J')

Powered by Google App Engine
This is Rietveld 408576698