Kernel debugging; service tests

pkg/kernel/lib/ast.dart

 // Copyright (c) 2016, 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.
 
 /// -----------------------------------------------------------------------
 ///                           ERROR HANDLING
 /// -----------------------------------------------------------------------
 ///
 /// As a rule of thumb, errors that can be detected statically are handled by
 /// the frontend, typically by translating the erroneous code into a 'throw' or
@@ skipping 2982 matching lines @@
 
 /// Declaration of a local variable.
 ///
 /// This may occur as a statement, but is also used in several non-statement
 /// contexts, such as in [ForStatement], [Catch], and [FunctionNode].
 ///
 /// When this occurs as a statement, it must be a direct child of a [Block].
 //
 // DESIGN TODO: Should we remove the 'final' modifier from variables?
 class VariableDeclaration extends Statement {
+  /// Offset of the equals sign in the source file it comes from.

[Kevin Millikin (Google), 2017/02/08 15:37:51]

For dartdoc comments, I suggest following the style guide:
https://www.dartlang.org/guides/language/effective-dart/documentation

Namely, the first paragraph should be a short (one line) description, followed
by a blank line and then more details.  Putting it all on one line like this
makes the entire comment show up as the summary in some views, which looks
cluttered.

[jensj, 2017/02/13 14:04:15]

Done.

+  /// Valid values are from 0 and up, or -1 ([TreeNode.noOffset])
+  /// if the equals sign offset is not available (e.g. if not initialized)
+  /// (this is the default if none is specifically set).
+  int fileEqualsOffset = TreeNode.noOffset;
+
   /// For named parameters, this is the name of the parameter. No two named
   /// parameters (in the same parameter list) can have the same name.
   ///
   /// In all other cases, the name is cosmetic, may be empty or null,
   /// and is not necessarily unique.
   String name;
   int flags = 0;
   DartType type; // Not null, defaults to dynamic.
   InferredValue inferredValue; // May be null.
 
@@ skipping 655 matching lines @@
     }
   }
 }
 
 class Source {
   final List<int> lineStarts;
   final String source;
 
   Source(this.lineStarts, this.source);
 }