Always finalize blocks after parsing, also for do-expressions

src/parsing/rewriter.h

Index: src/parsing/rewriter.h
diff --git a/src/parsing/rewriter.h b/src/parsing/rewriter.h
index 477644a7567f92a09a74d67677d62059331dcfbf..f75d409669b5ab7d1c29fc9ad1c25db1cec4be01 100644
--- a/src/parsing/rewriter.h
+++ b/src/parsing/rewriter.h
@@ -12,6 +12,7 @@ class AstValueFactory;
 class DoExpression;
 class ParseInfo;
 class Parser;
+class Scope;
 
 class Rewriter {
  public:
@@ -24,8 +25,8 @@ class Rewriter {
   static bool Rewrite(ParseInfo* info);
 
   // Rewrite a list of statements, using the same rules as a top-level program,
-  // to  ensure identical behaviour of completion result.
-  static bool Rewrite(Parser* parser, DoExpression* expr,
+  // to ensure identical behaviour of completion result.
+  static bool Rewrite(Parser* parser, Scope* outer_scope, DoExpression* expr,

[adamk, 2016/07/21 17:15:41]

Please add documentation to the comment saying what the scope is used for, to
make it clear to future readers why it's safe to call this with the outer scope.

                       AstValueFactory* factory);
 };