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

Unified Diff: pkg/kernel/lib/binary/ast_from_binary.dart

Issue 2750013002: [kernel] Debugging of switch statement (Closed)
Patch Set: Take #2: Update switch case to include list of positions Created 3 years, 9 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/lib/binary/ast_from_binary.dart
diff --git a/pkg/kernel/lib/binary/ast_from_binary.dart b/pkg/kernel/lib/binary/ast_from_binary.dart
index aa9641b672f5901a4a675b1089cd4fb6f5e31dbc..3c80743e35708744d413c2f648b4688936f7eb10 100644
--- a/pkg/kernel/lib/binary/ast_from_binary.dart
+++ b/pkg/kernel/lib/binary/ast_from_binary.dart
@@ -908,6 +908,10 @@ class BinaryBuilder {
for (int i = 0; i < cases.length; ++i) {
var caseNode = cases[i];
_fillTreeNodeList(caseNode.expressions, readExpression, caseNode);
+ caseNode.expressionsOffsets.length = readUInt();
Kevin Millikin (Google) 2017/03/20 12:29:47 I'd just change _fillNonTreeNodeList so it didn't
jensj 2017/03/21 10:06:19 I've changed it to not encode the size as well so
+ for (int i = 0; i < caseNode.expressionsOffsets.length; ++i) {
+ caseNode.expressionsOffsets[i] = readOffset();
+ }
caseNode.isDefault = readByte() == 1;
caseNode.body = readStatement()..parent = caseNode;
}

Powered by Google App Engine
This is Rietveld 408576698