Serialize and resynthesize variable initializers.

pkg/analyzer/lib/src/summary/idl.dart

 // Copyright (c) 2015, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 /**
  * This file is an "idl" style description of the summary format.  It
  * contains abstract classes which declare the interface for reading data from
  * summaries.  It is parsed and transformed into code that implements the
  * summary format.
  *
@@ skipping 362 matching lines @@
    * The `length` property access.
    */
   length,
 
   /**
    * The entity is a typedef.
    */
   typedef,
 
   /**
+   * The entity is a local function.
+   */
+  function,
+
+  /**
    * The entity is a top level function.
    */
   topLevelFunction,
 
   /**
    * The entity is a top level getter or setter.
    */
   topLevelPropertyAccessor,
 
   /**
@@ skipping 956 matching lines @@
   int get uriOffset;
 }
 
 /**
  * Unlinked summary information about a function parameter.
  */
 abstract class UnlinkedParam extends base.SummaryClass {
   /**
    * Annotations for this parameter.
    */
-  @Id(9)
+  @Id(10)

[Paul Berry, 2016/02/17 21:15:29]

Don't change the ids of existing getters--it messes up forward/backward
compatibility.  Instead, just assign the new field to the next unused id (12 in
this case).

[scheglov, 2016/02/17 21:27:51]

Done.

   List<UnlinkedConst> get annotations;
 
   /**
    * If the parameter has a default value, the constant expression in the
    * default value.  Note that the presence of this expression does not mean
    * that it is a valid, check [UnlinkedConst.isInvalid].
    */
   @Id(7)
   UnlinkedConst get defaultValue;
 
   /**
    * If this parameter's type is inferable, nonzero slot id identifying which
    * entry in [LinkedLibrary.types] contains the inferred type.  If there is no
    * matching entry in [LinkedLibrary.types], then no type was inferred for
    * this variable, so its static type is `dynamic`.
    *
    * Note that although strong mode considers initializing formals to be
    * inferable, they are not marked as such in the summary; if their type is
    * not specified, they always inherit the static type of the corresponding
    * field.
    */
   @Id(2)
   int get inferredTypeSlot;
 
   /**
+   * The synthetic initializer function of the parameter.  Absent if the variable
+   * does not have an initializer.
+   */
+  @Id(8)
+  UnlinkedExecutable get initializer;
+
+  /**
    * Indicates whether this is a function-typed parameter.
    */
   @Id(5)
   bool get isFunctionTyped;
 
   /**
    * Indicates whether this is an initializing formal parameter (i.e. it is
    * declared using `this.` syntax).
    */
   @Id(6)
@@ skipping 14 matching lines @@
   /**
    * Offset of the parameter name relative to the beginning of the file.
    */
   @informative
   @Id(1)
   int get nameOffset;
 
   /**
    * If [isFunctionTyped] is `true`, the parameters of the function type.
    */
-  @Id(8)
+  @Id(9)
   List<UnlinkedParam> get parameters;
 
   /**
    * If [isFunctionTyped] is `true`, the declared return type.  If
    * [isFunctionTyped] is `false`, the declared type.  Absent if the type is
    * implicit.
    */
   @Id(3)
   EntityRef get type;
 
   /**
    * The length of the visible range.
    */
-  @Id(10)
+  @Id(11)
   int get visibleLength;
 
   /**
    * The beginning of the visible range.
    */
-  @Id(11)
+  @Id(12)
   int get visibleOffset;
 }
 
 /**
  * Enum used to indicate the kind of a parameter.
  */
 enum UnlinkedParamKind {
   /**
    * Parameter is required.
    */
@@ skipping 326 matching lines @@
 }
 
 /**
  * Unlinked summary information about a top level variable, local variable, or
  * a field.
  */
 abstract class UnlinkedVariable extends base.SummaryClass {
   /**
    * Annotations for this variable.
    */
-  @Id(8)
+  @Id(9)

[Paul Berry, 2016/02/17 21:15:29]

Similar issue here.

[scheglov, 2016/02/17 21:27:51]

Done.

   List<UnlinkedConst> get annotations;
 
   /**
    * If [isConst] is true, and the variable has an initializer, the constant
    * expression in the initializer.  Note that the presence of this expression
    * does not mean that it is a valid, check [UnlinkedConst.isInvalid].
    */
   @Id(5)
   UnlinkedConst get constExpr;
 
   /**
    * Documentation comment for the variable, or `null` if there is no
    * documentation comment.
    */
   @informative
-  @Id(10)
+  @Id(11)
   UnlinkedDocumentationComment get documentationComment;
 
   /**
    * If this variable is inferable, nonzero slot id identifying which entry in
    * [LinkedLibrary.types] contains the inferred type for this variable.  If
    * there is no matching entry in [LinkedLibrary.types], then no type was
    * inferred for this variable, so its static type is `dynamic`.
    */
-  @Id(9)
+  @Id(10)
   int get inferredTypeSlot;
 
   /**
+   * The synthetic initializer function of the variable.  Absent if the variable
+   * does not have an initializer.
+   */
+  @Id(8)
+  UnlinkedExecutable get initializer;
+
+  /**
    * Indicates whether the variable is declared using the `const` keyword.
    */
   @Id(6)
   bool get isConst;
 
   /**
    * Indicates whether the variable is declared using the `final` keyword.
    */
   @Id(7)
   bool get isFinal;
@@ skipping 34 matching lines @@
 
   /**
    * Declared type of the variable.  Absent if the type is implicit.
    */
   @Id(3)
   EntityRef get type;
 
   /**
    * If a local variable, the length of the visible range; zero otherwise.
    */
-  @Id(11)
+  @Id(12)
   int get visibleLength;
 
   /**
    * If a local variable, the beginning of the visible range; zero otherwise.
    */
-  @Id(12)
+  @Id(13)
   int get visibleOffset;
 }