stdout/stderr cleanup (#1427)

* Partial revert of #1413

* remove extra line

* dartfmt

Author: jcollins-g

lib/dartdoc.dart

@@ -187,22 +187,21 @@ class DartDoc {
       writtenFiles.addAll(generator.writtenFiles.map(path.normalize));
     }
 
-    double seconds = _stopwatch.elapsedMilliseconds / 1000.0;
-    print(
-        "documented ${package.libraries.length} librar${package.libraries.length == 1 ? 'y' : 'ies'} "
-        "in ${seconds.toStringAsFixed(1)} seconds");
-    print('');
-
     verifyLinks(package, outputDir.path);
     int warnings = package.packageWarningCounter.warningCount;
     int errors = package.packageWarningCounter.errorCount;
     if (warnings == 0 && errors == 0) {
-      print("no issues found");
+      print("\nno issues found");
     } else {
-      print("found ${warnings} ${pluralize('warning', warnings)} "
+      print("\nfound ${warnings} ${pluralize('warning', warnings)} "
           "and ${errors} ${pluralize('error', errors)}");
     }
 
+    double seconds = _stopwatch.elapsedMilliseconds / 1000.0;
+    print(
+        "\ndocumented ${package.libraries.length} librar${package.libraries.length == 1 ? 'y' : 'ies'} "
+        "in ${seconds.toStringAsFixed(1)} seconds");
+
     if (package.libraries.isEmpty) {
       throw new DartDocFailure(
           "dartdoc could not find any libraries to document. Run `pub get` and try again.");
@@ -356,7 +355,7 @@ class DartDoc {
     final Set<String> visited = new Set();
     final String start = 'index.html';
     visited.add(start);
-    print('validating docs...');
+    stdout.write('\nvalidating docs...');
     _doCheck(package, origin, visited, start);
     _doOrphanCheck(package, origin, visited);
   }
@@ -517,7 +516,6 @@ class DartDoc {
     double seconds = _stopwatch.elapsedMilliseconds / 1000.0;
     print("parsed ${libraries.length} ${pluralize('file', libraries.length)} "
         "in ${seconds.toStringAsFixed(1)} seconds");
-    print('');
     _stopwatch.reset();
 
     if (errors.isNotEmpty) {

lib/src/html/html_generator_instance.dart

@@ -4,7 +4,7 @@
 
 import 'dart:async' show Future, StreamController;
 import 'dart:convert' show JsonEncoder;
-import 'dart:io' show Directory, File;
+import 'dart:io' show Directory, File, stdout;
 import 'dart:typed_data' show Uint8List;
 
 import 'package:collection/collection.dart' show compareNatural;
@@ -181,12 +181,14 @@ class HtmlGeneratorInstance implements HtmlOptions {
 
   void generatePackage() {
     TemplateData data = new PackageTemplateData(this, package, useCategories);
+    stdout.write('\ndocumenting ${package.name}');
 
     _build('index.html', _templates.indexTemplate, data);
   }
 
   void generateLibrary(Package package, Library lib) {
-    print('generating docs for library ${lib.name} from ${lib.path}...');
+    stdout
+        .write('\ngenerating docs for library ${lib.name} from ${lib.path}...');
     if (!lib.isAnonymous && !lib.hasDocumentation) {
       package.warnOnElement(lib, PackageWarning.noLibraryLevelDocs);
     }

lib/src/model.dart

@@ -2850,7 +2850,7 @@ class PackageWarningCounter {
     } else {
       if (options.asErrors.contains(kind)) toWrite = "error: ${fullMessage}";
     }
-    if (toWrite != null) print(" ${toWrite}");
+    if (toWrite != null) stderr.write("\n ${toWrite}");
   }
 
   /// Returns true if we've already warned for this.