Clean up some code in plugins

pkg/analyzer_plugin/lib/src/utilities/change_builder/change_builder_dart.dart

 // Copyright (c) 2017, 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.
 
 import 'dart:async';
 
 import 'package:analyzer/dart/ast/ast.dart';
 import 'package:analyzer/dart/ast/token.dart';
 import 'package:analyzer/dart/element/element.dart';
 import 'package:analyzer/dart/element/type.dart';
@@ skipping 339 matching lines @@
     // parameters + body
     if (isGetter) {
       writeln(' => null;');
     } else {
       List<ParameterElement> parameters = member.parameters;
       writeParameters(parameters, methodBeingCopied: member);
       writeln(' {');
       // TO-DO
       write(prefix2);
       writeln('// TODO: implement ${member.displayName}');
-      // REVIEW: Added return statement.
       if (shouldReturn) {
         write(prefix2);
         writeln('return null;');
       }
       // close method
       write(prefix);
       writeln('}');
     }
   }
 
@@ skipping 600 matching lines @@
    * The compilation unit to which the code will be added.
    */
   CompilationUnit unit;
 
   /**
    * A set containing the sources of the libraries that need to be imported in
    * order to make visible the names used in generated code.
    */
   Set<Source> librariesToImport = new Set<Source>();
 
-//  /**
-//   * The content of the file being edited.
-//   */
-//  String _content;
-
   /**
    * Initialize a newly created builder to build a source file edit within the
    * change being built by the given [changeBuilder]. The file being edited has
    * the given [source] and [timeStamp], and the given fully resolved [unit].
    */
   DartFileEditBuilderImpl(DartChangeBuilderImpl changeBuilder, String path,
       int timeStamp, this.unit)
       : super(changeBuilder, path, timeStamp);
 
   @override
@@ skipping 270 matching lines @@
       int nextLineOffset = lineInfo.getOffsetOfLine(currentLine + 1);
       String insertLine = source.substring(offset, nextLineOffset);
       if (!insertLine.trim().isEmpty) {
         insertEmptyLineAfter = true;
       }
     }
     return new _InsertionDescription(
         offset, insertEmptyLineBefore, insertEmptyLineAfter);
   }
 
-//  /**
-//   * Return the content of the file being edited.
-//   */
-//  String getContent() {
-//    if (_content == null) {
-//      CompilationUnitElement unitElement = unit.element;
-//      AnalysisContext context = unitElement.context;
-//      if (context == null) {
-//        throw new CancelCorrectionException();
-//      }
-//      _content = context.getContents(unitElement.source).data;
-//    }
-//    return _content;
-//  }
-
   /**
    * Computes the best URI to import [what] into [from].
    */
   String _getLibrarySourceUri(LibraryElement from, Source what) {
     String whatPath = what.fullName;
     // check if an absolute URI (such as 'dart:' or 'package:')
     Uri whatUri = what.uri;
     String whatUriScheme = whatUri.scheme;
     if (whatUriScheme != '' && whatUriScheme != 'file') {
       return whatUri.toString();
     }
     // compute a relative URI
     String fromFolder = path.dirname(from.source.fullName);
     String relativeFile = path.relative(whatPath, from: fromFolder);
     return path.split(relativeFile).join('/');
   }
 
   /**
    * Returns the text of the given range in the unit.
    */
   String _getText(String content, int offset, int length) {
     return content.substring(offset, offset + length);
   }
 
-//  /**
-//   * Returns the text of the given [AstNode] in the unit.
-//   */
-//  String _getNodeText(AstNode node) {
-//    return _getText(node.offset, node.length);
-//  }
-//
-//  /**
-//   * Returns the text of the given range in the unit.
-//   */
-//  String _getText(int offset, int length) {
-//    return getContent().substring(offset, offset + length);
-//  }
-
   /**
    * Create an edit to replace the return type of the innermost function
    * containing the given [node] with the type `Future`. The [typeProvider] is
    * used to check the current return type, because if it is already `Future` no
    * edit will be added.
    */
   void _replaceReturnTypeWithFuture(AstNode node, TypeProvider typeProvider) {
     while (node != null) {
       node = node.parent;
       if (node is FunctionDeclaration) {
@@ skipping 62 matching lines @@
   }
 }
 
 class _InsertionDescription {
   final int offset;
   final bool insertEmptyLineBefore;
   final bool insertEmptyLineAfter;
   _InsertionDescription(
       this.offset, this.insertEmptyLineBefore, this.insertEmptyLineAfter);
 }