Fix for translation of \!= to \!identical(), but use == and \!= for Enum.

pkg/analyzer/lib/src/generated/source_io.dart

 // Copyright (c) 2014, 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 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.io;
 
 import 'source.dart';
@@ skipping 48 matching lines @@
   bool isLocal(Source source) => source.uriKind != UriKind.DART_URI;
 }
 
 /**
  * Instances of the class `FileBasedSource` implement a source that represents a file.
  */
 class FileBasedSource implements Source {
   /**
    * The file represented by this source.
    */
-  JavaFile _file;
+  final JavaFile file;
 
   /**
    * The cached encoding for this source.
    */
   String _encoding;
 
   /**
    * The kind of URI from which this source was originally derived.
    */
-  UriKind _uriKind;
+  final UriKind uriKind;
 
   /**
    * Initialize a newly created source object. The source object is assumed to not be in a system
    * library.
    *
    * @param file the file represented by this source
    */
   FileBasedSource.con1(JavaFile file) : this.con2(file, UriKind.FILE_URI);
 
   /**
    * Initialize a newly created source object.
    *
    * @param file the file represented by this source
    * @param flags `true` if this source is in one of the system libraries
    */
-  FileBasedSource.con2(JavaFile file, UriKind uriKind) {
-    this._file = file;
-    this._uriKind = uriKind;
-  }
+  FileBasedSource.con2(this.file, this.uriKind);
 
   @override
-  bool operator ==(Object object) => object != null && this.runtimeType == object.runtimeType && _file == (object as FileBasedSource)._file;
+  bool operator ==(Object object) => object != null && this.runtimeType == object.runtimeType && file == (object as FileBasedSource).file;
 
   @override
-  bool exists() => _file.isFile();
+  bool exists() => file.isFile();
 
   @override
   TimestampedData<String> get contents {
     TimeCounter_TimeCounterHandle handle = PerformanceStatistics.io.start();
     try {
       return contentsFromFile;
     } finally {
       handle.stop();
     }
   }
 
   @override
   String get encoding {
     if (_encoding == null) {
-      _encoding = "${_uriKind.encoding}${_file.toURI().toString()}";
+      _encoding = "${uriKind.encoding}${file.toURI().toString()}";
     }
     return _encoding;
   }
 
   @override
-  String get fullName => _file.getAbsolutePath();
+  String get fullName => file.getAbsolutePath();
 
   @override
-  int get modificationStamp => _file.lastModified();
+  int get modificationStamp => file.lastModified();
 
   @override
-  String get shortName => _file.getName();
+  String get shortName => file.getName();
 
   @override
-  UriKind get uriKind => _uriKind;
+  int get hashCode => file.hashCode;
 
   @override
-  int get hashCode => _file.hashCode;
-
-  @override
-  bool get isInSystemLibrary => identical(_uriKind, UriKind.DART_URI);
+  bool get isInSystemLibrary => uriKind == UriKind.DART_URI;
 
   @override
   Source resolveRelative(Uri containedUri) {
     try {
       Uri resolvedUri = file.toURI().resolveUri(containedUri);
-      return new FileBasedSource.con2(new JavaFile.fromUri(resolvedUri), _uriKind);
+      return new FileBasedSource.con2(new JavaFile.fromUri(resolvedUri), uriKind);
     } on JavaException catch (exception) {
     }
     return null;
   }
 
   @override
   String toString() {
-    if (_file == null) {
+    if (file == null) {
       return "<unknown source>";
     }
-    return _file.getAbsolutePath();
+    return file.getAbsolutePath();
   }
 
   /**
    * Get the contents and timestamp of the underlying file.
    *
    * Clients should consider using the the method [AnalysisContext#getContents]
    * because contexts can have local overrides of the content of a source that the source is not
    * aware of.
    *
    * @return the contents of the source paired with the modification stamp of the source
    * @throws Exception if the contents of this source could not be accessed
    * @see #getContents()
    */
   TimestampedData<String> get contentsFromFile {
-    return new TimestampedData<String>(_file.lastModified(), _file.readAsStringSync());
+    return new TimestampedData<String>(file.lastModified(), file.readAsStringSync());
   }
-
-  /**
-   * Return the file represented by this source. This is an internal method that is only intended to
-   * be used by subclasses of [UriResolver] that are designed to work with file-based sources.
-   *
-   * @return the file represented by this source
-   */
-  JavaFile get file => _file;
 }
 
 /**
  * Instances of the class `PackageUriResolver` resolve `package` URI's in the context of
  * an application.
  *
  * For the purposes of sharing analysis, the path to each package under the "packages" directory
  * should be canonicalized, but to preserve relative links within a package, the remainder of the
  * path from the package directory to the leaf should not.
  */
 class PackageUriResolver extends UriResolver {
   /**
    * The package directories that `package` URI's are assumed to be relative to.
    */
-  List<JavaFile> _packagesDirectories;
+  final List<JavaFile> _packagesDirectories;
 
   /**
    * The name of the `package` scheme.
    */
   static String PACKAGE_SCHEME = "package";
 
   /**
    * Log exceptions thrown with the message "Required key not available" only once.
    */
   static bool _CanLogRequiredKeyIoException = true;
 
   /**
    * Return `true` if the given URI is a `package` URI.
    *
    * @param uri the URI being tested
    * @return `true` if the given URI is a `package` URI
    */
   static bool isPackageUri(Uri uri) => PACKAGE_SCHEME == uri.scheme;
 
   /**
    * Initialize a newly created resolver to resolve `package` URI's relative to the given
    * package directories.
    *
    * @param packagesDirectories the package directories that `package` URI's are assumed to be
    *          relative to
    */
-  PackageUriResolver(List<JavaFile> packagesDirectories) {
-    if (packagesDirectories.length < 1) {
+  PackageUriResolver(this._packagesDirectories) {
+    if (_packagesDirectories.length < 1) {
       throw new IllegalArgumentException("At least one package directory must be provided");
     }
-    this._packagesDirectories = packagesDirectories;
   }
 
   @override
   Source fromEncoding(UriKind kind, Uri uri) {
-    if (identical(kind, UriKind.PACKAGE_SELF_URI) || identical(kind, UriKind.PACKAGE_URI)) {
+    if (kind == UriKind.PACKAGE_SELF_URI || kind == UriKind.PACKAGE_URI) {
       return new FileBasedSource.con2(new JavaFile.fromUri(uri), kind);
     }
     return null;
   }
 
   @override
   Source resolveAbsolute(Uri uri) {
     if (!isPackageUri(uri)) {
       return null;
     }
@@ skipping 168 matching lines @@
   /**
    * Return `true` if the given URI is a `file` URI.
    *
    * @param uri the URI being tested
    * @return `true` if the given URI is a `file` URI
    */
   static bool isFileUri(Uri uri) => uri.scheme == FILE_SCHEME;
 
   @override
   Source fromEncoding(UriKind kind, Uri uri) {
-    if (identical(kind, UriKind.FILE_URI)) {
+    if (kind == UriKind.FILE_URI) {
       return new FileBasedSource.con2(new JavaFile.fromUri(uri), kind);
     }
     return null;
   }
 
   @override
   Source resolveAbsolute(Uri uri) {
     if (!isFileUri(uri)) {
       return null;
     }
     return new FileBasedSource.con1(new JavaFile.fromUri(uri));
   }
 }