fix default warning level

lib/src/options.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.
 
 /// Set of flags and options passed to the compiler
 library dev_compiler.src.options;
 
 import 'dart:io';
 
 import 'package:args/args.dart';
@@ skipping 141 matching lines @@
       {this.strongOptions: const StrongModeOptions(),
       this.sourceOptions: const SourceResolverOptions(),
       this.codegenOptions: const CodegenOptions(),
       this.runnerOptions: const RunnerOptions(),
       this.checkSdk: false,
       this.dumpInfo: false,
       this.dumpInfoFile,
       this.useColors: true,
       this.help: false,
       this.version: false,
-      this.logLevel: Level.SEVERE,
+      this.logLevel: Level.WARNING,
       this.serverMode: false,
       this.enableHashing: false,
       this.widget: true,
       this.host: 'localhost',
       this.port: 8080,
       this.runtimeDir,
       this.inputs,
       this.inputBaseDir});
 }
 
 /// Parses options from the command-line
 CompilerOptions parseOptions(List<String> argv, {bool forceOutDir: false}) {
   ArgResults args = argParser.parse(argv);
   bool showUsage = args['help'];
   bool showVersion = args['version'];
 
   var serverMode = args['server'];
   var enableHashing = args['hashing'];
   if (enableHashing == null) {
     enableHashing = serverMode;
   }
-  // TODO(jmesserly): shouldn't level always default to warning?
-  var logLevel = serverMode ? Level.WARNING : Level.SEVERE;
+  var logLevel = Level.WARNING;
   var levelName = args['log'];
   if (levelName != null) {
     levelName = levelName.toUpperCase();
     logLevel = Level.LEVELS
         .firstWhere((l) => l.name == levelName, orElse: () => logLevel);
   }
   var useColors = stdioType(stdout) == StdioType.TERMINAL;
   var sdkPath = args['dart-sdk'];
   if (sdkPath == null && !args['mock-sdk']) {
     sdkPath = getSdkDir(argv).path;
@@ skipping 110 matching lines @@
           'to listen on all interfaces (used only when --serve is on)',
       defaultsTo: 'localhost')
   ..addOption('port',
       help: 'Port to serve files from (used only when --serve is on)',
       defaultsTo: '8080')
   ..addFlag('closure',
       help: 'Emit Closure Compiler-friendly code (experimental)',
       defaultsTo: _CLOSURE_DEFAULT)
   ..addFlag('force-compile',
       help: 'Compile code with static errors', defaultsTo: false)
-  ..addOption('log', abbr: 'l', help: 'Logging level (defaults to severe)')
+  ..addOption('log', abbr: 'l', help: 'Logging level (defaults to warning)')
   ..addFlag('dump-info',
       abbr: 'i', help: 'Dump summary information', defaultsTo: null)
   ..addOption('v8-binary',
       help: 'V8-based binary to run JavaScript output with (iojs, node, d8)',
       defaultsTo: 'iojs')
   ..addOption('dump-info-file',
       abbr: 'f',
       help: 'Dump info json file (requires dump-info)',
       defaultsTo: null));
 
@@ skipping 46 matching lines @@
     // The pub-cache directory is two levels up, but we verify that the layout
     // looks correct.
     if (path.basename(dir) != 'dev_compiler') return null;
     dir = path.dirname(dir);
     if (path.basename(dir) != 'global_packages') return null;
     dir = path.dirname(dir);
     return path.join(dir, cacheDir, 'lib', 'runtime');
   }
   return null;
 }