stdout/stderr fixes

Author: kevmoo

bin/dartdoc.dart

@@ -25,15 +25,15 @@ main(List<String> arguments) async {
   try {
     args = parser.parse(arguments);
   } on FormatException catch (e) {
-    print(e.message);
-    print('');
+    stderr.writeln(e.message);
+    stderr.writeln('');
     // http://linux.die.net/include/sysexits.h
     // #define EX_USAGE	64	/* command line usage error */
     _printUsageAndExit(parser, exitCode: 64);
   }
 
   if (args['help']) {
-    _printHelp(parser);
+    _printHelpAndExit(parser);
   }
 
   if (args['version']) {
@@ -135,7 +135,7 @@ main(List<String> arguments) async {
     if (record.level == progressLevel) {
       if (showProgress && stopwatch.elapsed.inMilliseconds > 250) {
         writingProgress = true;
-        stderr.write('.');
+        stdout.write('.');
         stopwatch.reset();
       }
       return;
@@ -151,13 +151,17 @@ main(List<String> arguments) async {
     assert(message == message.trimRight());
     assert(message.isNotEmpty);
 
-    if (message.endsWith('...')) {
-      // Assume there may be more progress to print, so omit the trailing
-      // newline
-      writingProgress = true;
-      stdout.write(message);
+    if (record.level < logging.Level.WARNING) {
+      if (message.endsWith('...')) {
+        // Assume there may be more progress to print, so omit the trailing
+        // newline
+        writingProgress = true;
+        stdout.write(message);
+      } else {
+        print(message);
+      }
     } else {
-      print(message);
+      stderr.writeln(message);
     }
   });
 
@@ -359,7 +363,7 @@ ArgParser _createArgsParser() {
 }
 
 /// Print help if we are passed the help option.
-void _printHelp(ArgParser parser, {int exitCode: 0}) {
+void _printHelpAndExit(ArgParser parser, {int exitCode: 0}) {
   print('Generate HTML documentation for Dart libraries.\n');
   _printUsageAndExit(parser, exitCode: exitCode);
 }