java2dart improvement - use field if getter/setter are trivial.

pkg/analyzer_experimental/lib/src/generated/source.dart

 // This code was auto-generated, is not intended to be edited, and is subject to
 // significant change. Please see the README file for more information.
 library engine.source;
 import 'java_core.dart';
 import 'sdk.dart' show DartSdk;
 import 'engine.dart' show AnalysisContext;
 /**
  * Instances of the class `SourceFactory` resolve possibly relative URI's against an existing
  * [Source].
  *
  * @coverage dart.engine.source
  */
 class SourceFactory {
 
   /**
    * The analysis context that this source factory is associated with.
    */
-  AnalysisContext _context;
+  AnalysisContext context;
 
   /**
    * The resolvers used to resolve absolute URI's.
    */
   List<UriResolver> _resolvers;
 
   /**
    * A cache of content used to override the default content of a source.
    */
-  ContentCache _contentCache;
+  ContentCache contentCache;
 
   /**
    * Initialize a newly created source factory.
    *
    * @param contentCache the cache holding content used to override the default content of a source
    * @param resolvers the resolvers used to resolve absolute URI's
    */
   SourceFactory.con1(ContentCache contentCache, List<UriResolver> resolvers) {
-    this._contentCache = contentCache;
+    this.contentCache = contentCache;
     this._resolvers = resolvers;
   }
 
   /**
    * Initialize a newly created source factory.
    *
    * @param resolvers the resolvers used to resolve absolute URI's
    */
   SourceFactory.con2(List<UriResolver> resolvers) : this.con1(new ContentCache(), resolvers);
 
@@ skipping 27 matching lines @@
     if (encoding.length < 2) {
       throw new IllegalArgumentException("Invalid encoding length");
     }
     UriKind kind = UriKind.fromEncoding(encoding.codeUnitAt(0));
     if (kind == null) {
       throw new IllegalArgumentException("Invalid source kind in encoding: ${kind}");
     }
     try {
       Uri uri = parseUriWithException(encoding.substring(1));
       for (UriResolver resolver in _resolvers) {
-        Source result = resolver.fromEncoding(_contentCache, kind, uri);
+        Source result = resolver.fromEncoding(contentCache, kind, uri);
         if (result != null) {
           return result;
         }
       }
       throw new IllegalArgumentException("No resolver for kind: ${kind}");
     } catch (exception) {
       throw new IllegalArgumentException("Invalid URI in encoding");
     }
   }
 
   /**
-   * Return a cache of content used to override the default content of a source.
-   *
-   * @return a cache of content used to override the default content of a source
-   */
-  ContentCache get contentCache => _contentCache;
-
-  /**
-   * Return the analysis context that this source factory is associated with.
-   *
-   * @return the analysis context that this source factory is associated with
-   */
-  AnalysisContext get context => _context;
-
-  /**
    * Return the [DartSdk] associated with this [SourceFactory], or `null` if there
    * is no such SDK.
    *
    * @return the [DartSdk] associated with this [SourceFactory], or `null` if
    *         there is no such SDK
    */
   DartSdk get dartSdk {
     for (UriResolver resolver in _resolvers) {
       if (resolver is DartUriResolver) {
         DartUriResolver dartUriResolver = resolver as DartUriResolver;
@@ skipping 42 matching lines @@
 
   /**
    * Set the contents of the given source to the given contents. This has the effect of overriding
    * the default contents of the source. If the contents are `null` the override is removed so
    * that the default contents will be returned.
    *
    * @param source the source whose contents are being overridden
    * @param contents the new contents of the source
    * @return `true` if the new cached contents are different from the old, else `false`
    */
-  bool setContents(Source source, String contents) => _contentCache.setContents(source, contents);
-
-  /**
-   * Set the analysis context that this source factory is associated with to the given context.
-   *
-   * <b>Note:</b> This method should only be invoked by
-   * [AnalysisContextImpl#setSourceFactory] and is only public out of
-   * necessity.
-   *
-   * @param context the analysis context that this source factory is associated with
-   */
-  void set context(AnalysisContext context2) {
-    this._context = context2;
-  }
+  bool setContents(Source source, String contents) => contentCache.setContents(source, contents);
 
   /**
    * Return the contents of the given source, or `null` if this factory does not override the
    * contents of the source.
    *
    * <b>Note:</b> This method is not intended to be used except by
    * [FileBasedSource#getContents].
    *
    * @param source the source whose content is to be returned
    * @return the contents of the given source
    */
-  String getContents(Source source) => _contentCache.getContents(source);
+  String getContents(Source source) => contentCache.getContents(source);
 
   /**
    * Return the modification stamp of the given source, or `null` if this factory does not
    * override the contents of the source.
    *
    * <b>Note:</b> This method is not intended to be used except by
    * [FileBasedSource#getModificationStamp].
    *
    * @param source the source whose modification stamp is to be returned
    * @return the modification stamp of the given source
    */
-  int getModificationStamp(Source source) => _contentCache.getModificationStamp(source);
+  int getModificationStamp(Source source) => contentCache.getModificationStamp(source);
 
   /**
    * Return a source object representing the URI that results from resolving the given (possibly
    * relative) contained URI against the URI associated with an existing source object, or
    * `null` if either the contained URI is invalid or if it cannot be resolved against the
    * source object's URI.
    *
    * @param containingSource the source containing the given URI
    * @param containedUri the (possibly relative) URI to be resolved against the containing source
    * @return the source representing the contained URI
    */
   Source resolveUri2(Source containingSource, Uri containedUri) {
     if (containedUri.isAbsolute) {
       for (UriResolver resolver in _resolvers) {
-        Source result = resolver.resolveAbsolute(_contentCache, containedUri);
+        Source result = resolver.resolveAbsolute(contentCache, containedUri);
         if (result != null) {
           return result;
         }
       }
       return null;
     } else {
       return containingSource.resolveRelative(containedUri);
     }
   }
 }
@@ skipping 233 matching lines @@
 
   /**
    * A 'package:' URI.
    */
   static final UriKind PACKAGE_URI = new UriKind('PACKAGE_URI', 2, 0x70);
   static final List<UriKind> values = [DART_URI, FILE_URI, PACKAGE_URI];
 
   /**
    * The single character encoding used to identify this kind of URI.
    */
-  int _encoding = 0;
+  int encoding = 0;
 
   /**
    * Initialize a newly created URI kind to have the given encoding.
    *
    * @param encoding the single character encoding used to identify this kind of URI.
    */
   UriKind(String name, int ordinal, int encoding) : super(name, ordinal) {
-    this._encoding = encoding;
+    this.encoding = encoding;
   }
 
   /**
    * Return the URI kind represented by the given encoding, or `null` if there is no kind with
    * the given encoding.
    *
    * @param encoding the single character encoding used to identify the URI kind to be returned
    * @return the URI kind represented by the given encoding
    */
   static UriKind fromEncoding(int encoding) {
     while (true) {
       if (encoding == 0x64) {
         return DART_URI;
       } else if (encoding == 0x66) {
         return FILE_URI;
       } else if (encoding == 0x70) {
         return PACKAGE_URI;
       }
       break;
     }
     return null;
   }
-
-  /**
-   * Return the single character encoding used to identify this kind of URI.
-   *
-   * @return the single character encoding used to identify this kind of URI
-   */
-  int get encoding => _encoding;
 }
 /**
  * A source range defines an [Element]'s source coordinates relative to its [Source].
  *
  * @coverage dart.engine.utilities
  */
 class SourceRange {
 
   /**
    * The 0-based index of the first character of the source code for this element, relative to the
    * source buffer in which this element is contained.
    */
-  int _offset = 0;
+  int offset = 0;
 
   /**
    * The number of characters of the source code for this element, relative to the source buffer in
    * which this element is contained.
    */
-  int _length = 0;
+  int length = 0;
 
   /**
    * Initialize a newly created source range using the given offset and the given length.
    *
    * @param offset the given offset
    * @param length the given length
    */
   SourceRange(int offset, int length) {
-    this._offset = offset;
-    this._length = length;
+    this.offset = offset;
+    this.length = length;
   }
 
   /**
    * @return `true` if <code>x</code> is in [offset, offset + length) interval.
    */
-  bool contains(int x) => _offset <= x && x < _offset + _length;
+  bool contains(int x) => offset <= x && x < offset + length;
 
   /**
    * @return `true` if <code>x</code> is in (offset, offset + length) interval.
    */
-  bool containsExclusive(int x) => _offset < x && x < _offset + _length;
+  bool containsExclusive(int x) => offset < x && x < offset + length;
 
   /**
    * @return `true` if <code>otherRange</code> covers this [SourceRange].
    */
   bool coveredBy(SourceRange otherRange) => otherRange.covers(this);
 
   /**
    * @return `true` if this [SourceRange] covers <code>otherRange</code>.
    */
   bool covers(SourceRange otherRange) => offset <= otherRange.offset && otherRange.end <= end;
 
   /**
    * @return `true` if this [SourceRange] ends in <code>otherRange</code>.
    */
   bool endsIn(SourceRange otherRange) {
     int thisEnd = end;
     return otherRange.contains(thisEnd);
   }
   bool operator ==(Object obj) {
     if (obj is! SourceRange) {
       return false;
     }
     SourceRange sourceRange = obj as SourceRange;
-    return sourceRange.offset == _offset && sourceRange.length == _length;
+    return sourceRange.offset == offset && sourceRange.length == length;
   }
 
   /**
    * @return the 0-based index of the after-last character of the source code for this element,
    *         relative to the source buffer in which this element is contained.
    */
-  int get end => _offset + _length;
+  int get end => offset + length;
 
   /**
    * @return the expanded instance of [SourceRange], which has the same center.
    */
-  SourceRange getExpanded(int delta) => new SourceRange(_offset - delta, delta + _length + delta);
-
-  /**
-   * Returns the number of characters of the source code for this element, relative to the source
-   * buffer in which this element is contained.
-   *
-   * @return the number of characters of the source code for this element, relative to the source
-   *         buffer in which this element is contained
-   */
-  int get length => _length;
+  SourceRange getExpanded(int delta) => new SourceRange(offset - delta, delta + length + delta);
 
   /**
    * @return the instance of [SourceRange] with end moved on "delta".
    */
-  SourceRange getMoveEnd(int delta) => new SourceRange(_offset, _length + delta);
-
-  /**
-   * Returns the 0-based index of the first character of the source code for this element, relative
-   * to the source buffer in which this element is contained.
-   *
-   * @return the 0-based index of the first character of the source code for this element, relative
-   *         to the source buffer in which this element is contained
-   */
-  int get offset => _offset;
+  SourceRange getMoveEnd(int delta) => new SourceRange(offset, length + delta);
 
   /**
    * @return the expanded translated of [SourceRange], with moved start and the same length.
    */
-  SourceRange getTranslated(int delta) => new SourceRange(_offset + delta, _length);
-  int get hashCode => 31 * _offset + _length;
+  SourceRange getTranslated(int delta) => new SourceRange(offset + delta, length);
+  int get hashCode => 31 * offset + length;
 
   /**
    * @return `true` if this [SourceRange] intersects with given.
    */
   bool intersects(SourceRange other) {
     if (other == null) {
       return false;
     }
     if (end <= other.offset) {
       return false;
     }
     if (offset >= other.end) {
       return false;
     }
     return true;
   }
 
   /**
    * @return `true` if this [SourceRange] starts in <code>otherRange</code>.
    */
-  bool startsIn(SourceRange otherRange) => otherRange.contains(_offset);
+  bool startsIn(SourceRange otherRange) => otherRange.contains(offset);
   String toString() {
     JavaStringBuilder builder = new JavaStringBuilder();
     builder.append("[offset=");
-    builder.append(_offset);
+    builder.append(offset);
     builder.append(", length=");
-    builder.append(_length);
+    builder.append(length);
     builder.append("]");
     return builder.toString();
   }
 }
 /**
  * The interface `SourceContainer` is used by clients to define a collection of sources
  *
  * Source containers are not used within analysis engine, but can be used by clients to group
  * sources for the purposes of accessing composite dependency information. For example, the Eclipse
  * client uses source containers to represent Eclipse projects, which allows it to easily compute
@@ skipping 14 matching lines @@
 /**
  * Instances of the class `DartUriResolver` resolve `dart` URI's.
  *
  * @coverage dart.engine.source
  */
 class DartUriResolver extends UriResolver {
 
   /**
    * The Dart SDK against which URI's are to be resolved.
    */
-  DartSdk _sdk;
+  DartSdk dartSdk;
 
   /**
    * The name of the `dart` scheme.
    */
   static String _DART_SCHEME = "dart";
 
   /**
    * Return `true` if the given URI is a `dart:` URI.
    *
    * @param uri the URI being tested
    * @return `true` if the given URI is a `dart:` URI
    */
   static bool isDartUri(Uri uri) => _DART_SCHEME == uri.scheme;
 
   /**
    * Initialize a newly created resolver to resolve Dart URI's against the given platform within the
    * given Dart SDK.
    *
    * @param sdk the Dart SDK against which URI's are to be resolved
    */
   DartUriResolver(DartSdk sdk) {
-    this._sdk = sdk;
+    this.dartSdk = sdk;
   }
   Source fromEncoding(ContentCache contentCache, UriKind kind, Uri uri) {
     if (identical(kind, UriKind.DART_URI)) {
-      return _sdk.fromEncoding(contentCache, kind, uri);
+      return dartSdk.fromEncoding(contentCache, kind, uri);
     }
     return null;
   }
-
-  /**
-   * Return the [DartSdk] against which URIs are to be resolved.
-   *
-   * @return the [DartSdk] against which URIs are to be resolved.
-   */
-  DartSdk get dartSdk => _sdk;
   Source resolveAbsolute(ContentCache contentCache, Uri uri) {
     if (!isDartUri(uri)) {
       return null;
     }
-    return _sdk.mapDartUri(uri.toString());
+    return dartSdk.mapDartUri(uri.toString());
   }
 }
 /**
  * Instances of the class `LineInfo` encapsulate information about line and column information
  * within a source file.
  *
  * @coverage dart.engine.utilities
  */
 class LineInfo {
 
@@ skipping 35 matching lines @@
 }
 /**
  * Instances of the class `Location` represent the location of a character as a line and
  * column pair.
  */
 class LineInfo_Location {
 
   /**
    * The one-based index of the line containing the character.
    */
-  int _lineNumber = 0;
+  int lineNumber = 0;
 
   /**
    * The one-based index of the column containing the character.
    */
-  int _columnNumber = 0;
+  int columnNumber = 0;
 
   /**
    * Initialize a newly created location to represent the location of the character at the given
    * line and column position.
    *
    * @param lineNumber the one-based index of the line containing the character
    * @param columnNumber the one-based index of the column containing the character
    */
   LineInfo_Location(int lineNumber, int columnNumber) {
-    this._lineNumber = lineNumber;
-    this._columnNumber = columnNumber;
+    this.lineNumber = lineNumber;
+    this.columnNumber = columnNumber;
   }
-
-  /**
-   * Return the one-based index of the column containing the character.
-   *
-   * @return the one-based index of the column containing the character
-   */
-  int get columnNumber => _columnNumber;
-
-  /**
-   * Return the one-based index of the line containing the character.
-   *
-   * @return the one-based index of the line containing the character
-   */
-  int get lineNumber => _lineNumber;
 }
 /**
  * Instances of class `ContentCache` hold content used to override the default content of a
  * [Source].
  *
  * @coverage dart.engine.source
  */
 class ContentCache {
 
   /**
@@ skipping 45 matching lines @@
     if (contents == null) {
       _stampMap.remove(source);
       return _contentMap.remove(source) != null;
     } else {
       _stampMap[source] = JavaSystem.currentTimeMillis();
       String originalContents = javaMapPut(_contentMap, source, contents);
       return contents != originalContents;
     }
   }
 }