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

Unified Diff: runtime/vm/parser_test.cc

Issue 2005723004: Fraction class prototype and test (not to be committed). (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: work in progress Created 4 years, 5 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
« no previous file with comments | « runtime/vm/parser.cc ('k') | runtime/vm/raw_object.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/parser_test.cc
diff --git a/runtime/vm/parser_test.cc b/runtime/vm/parser_test.cc
index 319b13deb482863d154a4a22d1199656ae684a35..06f9e08f8121a0e931fce7541d798d457bfbf038 100644
--- a/runtime/vm/parser_test.cc
+++ b/runtime/vm/parser_test.cc
@@ -372,7 +372,7 @@ TEST_CASE(Parser_AllocateVariables_TwoChains) {
EXPECT_STREQ(
// bb captures only value2 from aa. No others.
"a.b.aa.bb\n"
- " 0 ContextVar level=0 begin=33 end=43 name=value2\n"
+ " 0 ContextVar level=0 begin=34 end=44 name=value2\n"
" 1 CurrentCtx scope=0 begin=0 end=0"
" name=:current_context_var\n"
@@ -388,9 +388,9 @@ TEST_CASE(Parser_AllocateVariables_TwoChains) {
"a.b.aa\n"
" 0 CurrentCtx scope=0 begin=0 end=0"
" name=:current_context_var\n"
- " 1 ContextLevel level=1 scope=2 begin=22 end=52\n"
- " 2 ContextVar level=1 begin=28 end=52 name=value2\n"
- " 3 StackVar scope=2 begin=30 end=52 name=bb\n"
+ " 1 ContextLevel level=1 scope=2 begin=22 end=53\n"
+ " 2 ContextVar level=1 begin=29 end=53 name=value2\n"
+ " 3 StackVar scope=2 begin=31 end=53 name=bb\n"
// Closure call saves current context.
"_Closure.call\n"
@@ -400,10 +400,10 @@ TEST_CASE(Parser_AllocateVariables_TwoChains) {
// b captures value1 from a.
"a.b\n"
- " 0 ContextVar level=0 begin=14 end=62 name=value1\n"
+ " 0 ContextVar level=0 begin=14 end=63 name=value1\n"
" 1 CurrentCtx scope=0 begin=0 end=0"
" name=:current_context_var\n"
- " 2 StackVar scope=2 begin=18 end=62 name=aa\n"
+ " 2 StackVar scope=2 begin=18 end=63 name=aa\n"
// Closure call saves current context.
"_Closure.call\n"
@@ -415,9 +415,9 @@ TEST_CASE(Parser_AllocateVariables_TwoChains) {
"a\n"
" 0 CurrentCtx scope=0 begin=0 end=0"
" name=:current_context_var\n"
- " 1 ContextLevel level=1 scope=2 begin=4 end=70\n"
- " 2 ContextVar level=1 begin=10 end=70 name=value1\n"
- " 3 StackVar scope=2 begin=12 end=70 name=b\n",
+ " 1 ContextLevel level=1 scope=2 begin=4 end=71\n"
+ " 2 ContextVar level=1 begin=10 end=71 name=value1\n"
+ " 3 StackVar scope=2 begin=12 end=71 name=b\n",
CaptureVarsAtLine(lib, "a", 7));
}
@@ -453,8 +453,8 @@ TEST_CASE(Parser_AllocateVariables_Issue7681) {
EXPECT_STREQ(
// This frame saves the entry context instead of chaining. Good.
"doIt.<anonymous closure>\n"
- " 0 ContextLevel level=1 scope=1 begin=41 end=62\n"
- " 1 ContextVar level=1 begin=42 end=62 name=y\n"
+ " 0 ContextLevel level=1 scope=1 begin=41 end=65\n"
+ " 1 ContextVar level=1 begin=42 end=65 name=y\n"
" 2 CurrentCtx scope=0 begin=0 end=0"
" name=:current_context_var\n"
@@ -473,7 +473,7 @@ TEST_CASE(Parser_AllocateVariables_Issue7681) {
"doIt\n"
" 0 CurrentCtx scope=0 begin=0 end=0"
" name=:current_context_var\n"
- " 1 StackVar scope=2 begin=35 end=77 name=x\n",
+ " 1 StackVar scope=2 begin=35 end=80 name=x\n",
CaptureVarsAtLine(lib, "doIt", 12));
}
@@ -499,7 +499,7 @@ TEST_CASE(Parser_AllocateVariables_CaptureLoopVar) {
EXPECT_STREQ(
// inner function captures variable value. That's fine.
"outer.inner\n"
- " 0 ContextVar level=0 begin=32 end=42 name=value\n"
+ " 0 ContextVar level=0 begin=33 end=43 name=value\n"
" 1 CurrentCtx scope=0 begin=0 end=0"
" name=:current_context_var\n"
@@ -514,10 +514,10 @@ TEST_CASE(Parser_AllocateVariables_CaptureLoopVar) {
"outer\n"
" 0 CurrentCtx scope=0 begin=0 end=0"
" name=:current_context_var\n"
- " 1 StackVar scope=3 begin=12 end=50 name=i\n"
- " 2 ContextLevel level=1 scope=4 begin=20 end=50\n"
- " 3 ContextVar level=1 begin=28 end=50 name=value\n"
- " 4 StackVar scope=4 begin=30 end=50 name=inner\n",
+ " 1 StackVar scope=3 begin=12 end=52 name=i\n"
+ " 2 ContextLevel level=1 scope=4 begin=20 end=52\n"
+ " 3 ContextVar level=1 begin=28 end=52 name=value\n"
+ " 4 StackVar scope=4 begin=30 end=52 name=inner\n",
CaptureVarsAtLine(lib, "outer", 5));
}
@@ -542,7 +542,7 @@ TEST_CASE(Parser_AllocateVariables_MiddleChain) {
EXPECT_VALID(lib);
EXPECT_STREQ(
"a.b.c\n"
- " 0 ContextVar level=0 begin=50 end=62 name=x\n"
+ " 0 ContextVar level=0 begin=51 end=64 name=x\n"
" 1 CurrentCtx scope=0 begin=0 end=0"
" name=:current_context_var\n"
"_Closure.call\n"
@@ -552,13 +552,13 @@ TEST_CASE(Parser_AllocateVariables_MiddleChain) {
// Doesn't save the entry context. Chains to parent instead.
"a.b\n"
- " 0 ContextVar level=0 begin=12 end=71 name=x\n"
+ " 0 ContextVar level=0 begin=12 end=73 name=x\n"
" 1 CurrentCtx scope=0 begin=0 end=0"
" name=:current_context_var\n"
- " 2 StackVar scope=2 begin=47 end=71 name=c\n"
- " 3 ContextLevel level=1 scope=3 begin=18 end=47\n"
- " 4 ContextVar level=1 begin=22 end=47 name=i\n"
- " 5 StackVar scope=4 begin=32 end=47 name=d\n"
+ " 2 StackVar scope=2 begin=48 end=73 name=c\n"
+ " 3 ContextLevel level=1 scope=3 begin=18 end=48\n"
+ " 4 ContextVar level=1 begin=22 end=48 name=i\n"
+ " 5 StackVar scope=4 begin=33 end=48 name=d\n"
"_Closure.call\n"
" 0 StackVar scope=1 begin=0 end=4 name=this\n"
@@ -568,9 +568,9 @@ TEST_CASE(Parser_AllocateVariables_MiddleChain) {
"a\n"
" 0 CurrentCtx scope=0 begin=0 end=0"
" name=:current_context_var\n"
- " 1 ContextLevel level=1 scope=2 begin=3 end=79\n"
- " 2 ContextVar level=1 begin=9 end=79 name=x\n"
- " 3 StackVar scope=2 begin=11 end=79 name=b\n",
+ " 1 ContextLevel level=1 scope=2 begin=3 end=81\n"
+ " 2 ContextVar level=1 begin=9 end=81 name=x\n"
+ " 3 StackVar scope=2 begin=11 end=81 name=b\n",
CaptureVarsAtLine(lib, "a", 10));
}
« no previous file with comments | « runtime/vm/parser.cc ('k') | runtime/vm/raw_object.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698