Serialize local functions and variables.

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

Index: pkg/analyzer/lib/src/summary/idl.dart
diff --git a/pkg/analyzer/lib/src/summary/idl.dart b/pkg/analyzer/lib/src/summary/idl.dart
index b671751c71b08b0c3e0ab61d4fe2a13f5d295791..903732e25ec2afcf6b2175b93c5dfb4cd76921ea 100644
--- a/pkg/analyzer/lib/src/summary/idl.dart
+++ b/pkg/analyzer/lib/src/summary/idl.dart
@@ -991,6 +991,11 @@ abstract class UnlinkedExecutable extends base.SummaryClass {
   UnlinkedDocumentationComment get documentationComment;
 
   /**
+   * The list of local functions.
+   */
+  List<UnlinkedExecutable> get executables;

[Paul Berry, 2016/02/11 15:29:19]

I'd prefer if we called this "localFunctions" for clarity.

[scheglov, 2016/02/11 15:50:08]

Done!
Actually I named them as localFunctions and localVariables initially, but then
thought that maybe they should be named as other similar fields :-)

+
+  /**
    * If this executable's return type is inferable, nonzero slot id
    * identifying which entry in [LinkedUnit.types] contains the inferred
    * return type.  If there is no matching entry in [LinkedUnit.types], then
@@ -1086,6 +1091,21 @@ abstract class UnlinkedExecutable extends base.SummaryClass {
    * method syntax is disabled.
    */
   List<UnlinkedTypeParam> get typeParameters;
+
+  /**
+   * The list of local variable.

[Paul Berry, 2016/02/11 15:29:19]

s/variable/variables/

[scheglov, 2016/02/11 15:50:08]

Done.

+   */
+  List<UnlinkedVariable> get variables;

[Paul Berry, 2016/02/11 15:29:19]

Similarly, this should be called "localVariables".

[scheglov, 2016/02/11 15:50:08]

Done.

+
+  /**
+   * If a local function, the length of the visible range; absent otherwise.

[Paul Berry, 2016/02/11 15:29:19]

s/absent/zero/ here and in 3 other places below.

(The fact that ints with a value of zero don't take up any space in the
serialized output is an implementation detail of Flatbuffers which shouldn't be
leaked to this level of abstraction)

[scheglov, 2016/02/11 15:50:08]

Done.

+   */
+  int get visibleLength;
+
+  /**
+   * If a local function, the beginning of the visible range; absent otherwise.
+   */
+  int get visibleOffset;
 }
 
 /**
@@ -1668,4 +1688,14 @@ abstract class UnlinkedVariable extends base.SummaryClass {
    * Declared type of the variable.  Absent if the type is implicit.
    */
   EntityRef get type;
+
+  /**
+   * If a local variable, the length of the visible range; absent otherwise.
+   */
+  int get visibleLength;
+
+  /**
+   * If a local variable, the beginning of the visible range; absent otherwise.
+   */
+  int get visibleOffset;
 }