Silly little cleanup

Don't use triple-slashes except when documenting members - started
Spelling

Author: kevmoo

lib/src/asset/reader.dart

@@ -18,6 +18,6 @@ abstract class AssetReader {
   /// Gets a [Stream<AssetId>] of all assets available matching [inputSets].
   Stream<AssetId> listAssetIds(Iterable<InputSet> inputSets);
 
-  /// Asynchonously gets the last modified [DateTime] of [id].
+  /// Asynchronously gets the last modified [DateTime] of [id].
   Future<DateTime> lastModified(AssetId id);
 }

lib/src/builder/build_step_impl.dart

@@ -119,7 +119,7 @@ class BuildStepImpl implements BuildStep {
   }
 
   /// Checks that [asset] is a valid output, and throws an
-  /// [InvalidOutputException] or [UnexcpectedOutputException] if it's not.
+  /// [InvalidOutputException] or [UnexpectedOutputException] if it's not.
   void _checkOutput(Asset asset) {
     if (asset.id.package != _rootPackage) {
       throw new InvalidOutputException(asset);

lib/src/generate/build.dart

@@ -82,7 +82,7 @@ Future<BuildResult> build(PhaseGroup phaseGroup,
 /// The [directoryWatcherFactory] allows you to inject a way of creating custom
 /// [DirectoryWatcher]s. By default a normal [DirectoryWatcher] will be used.
 ///
-/// The [teminateEventStream] is a stream which can send termination events.
+/// The [terminateEventStream] is a stream which can send termination events.
 /// By default the [ProcessSignal.SIGINT] stream is used. In this mode, the
 /// first event will allow any ongoing builds to finish, and then the program
 ///  will complete normally. Subsequent events are not handled (and will
@@ -175,7 +175,7 @@ Stream<BuildResult> serve(PhaseGroup phaseGroup,
 }
 
 /// Given [terminateEventStream], call [onTerminate] the first time an event is
-/// seen. If a second event is recieved, simply exit.
+/// seen. If a second event is received, simply exit.
 StreamSubscription _setupTerminateLogic(
     Stream terminateEventStream, Future onTerminate(),
     {Future cancelWhen}) {

lib/src/generate/build_impl.dart

@@ -191,8 +191,8 @@ class BuildImpl {
       return new AssetGraph.deserialize(
           JSON.decode(await _reader.readAsString(_assetGraphId)));
     } on AssetGraphVersionException catch (_) {
-      /// Start fresh if the cached asset_graph version doesn't match up with
-      /// the current version. We don't currently support old graph versions.
+      // Start fresh if the cached asset_graph version doesn't match up with
+      // the current version. We don't currently support old graph versions.
       _logger.info('Throwing away cached asset graph due to version mismatch.');
       return new AssetGraph();
     }
@@ -207,7 +207,7 @@ class BuildImpl {
     var completer = new Completer<bool>();
     Future
         .wait(currentMirrorSystem().libraries.keys.map((Uri uri) async {
-      /// Short-circuit
+      // Short-circuit
       if (completer.isCompleted) return;
       var lastModified;
       switch (uri.scheme) {
@@ -223,14 +223,14 @@ class BuildImpl {
           break;
         case 'file':
 
-          /// TODO(jakemac): Probably shouldn't use dart:io directly, but its
-          /// definitely the easiest solution and should be fine.
+          // TODO(jakemac): Probably shouldn't use dart:io directly, but its
+          // definitely the easiest solution and should be fine.
           var file = new File.fromUri(uri);
           lastModified = await file.lastModified();
           break;
         case 'data':
 
-          /// Test runner uses a `data` scheme, don't invalidate for those.
+          // Test runner uses a `data` scheme, don't invalidate for those.
           if (uri.path.contains('package:test')) return;
           continue unknownUri;
         unknownUri: default:
@@ -252,7 +252,7 @@ class BuildImpl {
 
   /// Creates and returns a map of updates to assets based on [_assetGraph].
   Future<Map<AssetId, ChangeType>> _getUpdates() async {
-    /// Collect updates to the graph based on any changed assets.
+    // Collect updates to the graph based on any changed assets.
     var updates = <AssetId, ChangeType>{};
     await Future.wait(_assetGraph.allNodes
         .where((node) =>
@@ -294,11 +294,11 @@ class BuildImpl {
         (_reader as CachedAssetReader).evictFromCache(id);
       }
 
-      /// Update all ouputs of this asset as well.
+      // Update all outputs of this asset as well.
       await Future.wait(node.outputs.map((output) =>
           clearNodeAndDeps(output, rootChangeType, parent: node.id)));
 
-      /// For deletes, prune the graph.
+      // For deletes, prune the graph.
       if (parent == null && rootChangeType == ChangeType.REMOVE) {
         _assetGraph.remove(id);
       }
@@ -322,8 +322,8 @@ class BuildImpl {
       await _writer.delete(_assetGraphId);
       _inputsByPackage[_assetGraphId.package]?.remove(_assetGraphId);
 
-      /// Remove all output nodes from [_inputsByPackage], and delete all assets
-      /// that need updates.
+      // Remove all output nodes from [_inputsByPackage], and delete all assets
+      // that need updates.
       await Future.wait(_assetGraph.allNodes
           .where((node) => node is GeneratedAssetNode)
           .map((node) async {
@@ -361,16 +361,16 @@ class BuildImpl {
         }
       }
 
-      /// Once the group is done, add all outputs so they can be used in the next
-      /// phase.
+      // Once the group is done, add all outputs so they can be used in the next
+      // phase.
       for (var outputId in groupOutputIds) {
         tempInputsByPackage.putIfAbsent(
             outputId.package, () => new Set<AssetId>());
         tempInputsByPackage[outputId.package].add(outputId);
       }
     }
 
-    // Check conflictingOuputs, prompt user to delete files.
+    // Check conflictingOutputs, prompt user to delete files.
     if (conflictingOutputs.isEmpty) return;
 
     Future deleteConflictingOutputs() {
@@ -432,10 +432,10 @@ class BuildImpl {
   Future<BuildResult> _runPhases() async {
     final outputs = <Asset>[];
     for (var phase in _buildActions) {
-      /// Collects all the ids for files which are output by this stage. This
-      /// also includes files which didn't get regenerated because they weren't,
-      /// dirty unlike [outputs] which only gets files which were explicitly
-      /// generated in this build.
+      // Collects all the ids for files which are output by this stage. This
+      // also includes files which didn't get regenerated because they weren't,
+      // dirty unlike [outputs] which only gets files which were explicitly
+      // generated in this build.
       final phaseOutputIds = new Set<AssetId>();
 
       await Future.wait(phase.map((action) async {

lib/src/generate/options.dart

@@ -17,18 +17,18 @@ import 'directory_watcher_factory.dart';
 
 /// Manages setting up consistent defaults for all options and build modes.
 class BuildOptions {
-  /// Build mode options.
+  // Build mode options.
   StreamSubscription logListener;
   PackageGraph packageGraph;
   AssetReader reader;
   AssetWriter writer;
   bool deleteFilesByDefault;
 
-  /// Watch mode options.
+  // Watch mode options.
   Duration debounceDelay;
   DirectoryWatcherFactory directoryWatcherFactory;
 
-  /// Server options.
+  // Server options.
   int port;
   String address;
   String directory;

lib/src/logging/logging.dart

@@ -8,7 +8,7 @@ import 'package:logging/logging.dart';
 
 /// Calls [logger#info] with [description] before and after calling [action]
 /// and waiting for the returned future to complete. The 2nd log message will
-/// have the ellapsed time appended.
+/// have the elapsed time appended.
 ///
 /// This function also appends a newline after the 2nd log (unless on windows)
 /// to ensure that it does not get overwritten by later log messages.

test/generate/watch_test.dart

@@ -177,7 +177,7 @@ main() {
             ChangeType.MODIFY, path.absolute('a', 'web', 'a.txt')));
 
         result = await nextResult(results);
-        // Ignores the modifcation under the `b` package, even though it
+        // Ignores the modification under the `b` package, even though it
         // matches the input set.
         checkOutputs({'a|web/a.txt.copy': 'b',}, result, writer.assets);
       });