dartfmt line wrapping bypass option support.

pkg/analyzer/bin/formatter.dart

 #!/usr/bin/env dart
 
 // Copyright (c) 2013, 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:convert';
 import 'dart:io';
 
 import 'package:args/args.dart';
@@ skipping 45 matching lines @@
   }
 }
 
 _readOptions(options) {
   kind = _parseKind(options[KIND_FLAG]);
   machineFormat = options[MACHINE_FLAG];
   overwriteFileContents = options[WRITE_FLAG];
   selection = _parseSelection(options[SELECTION_FLAG]);
   formatterSettings =
       new FormatterOptions(codeTransforms: options[TRANSFORM_FLAG],
-          pageWidth: _toInt(options[MAX_LINE_FLAG]));
+          pageWidth: _parseLineLength(options[MAX_LINE_FLAG]));
 }
 
 CodeKind _parseKind(kindOption) {
   switch(kindOption) {
     case 'stmt' :
       return CodeKind.STATEMENT;
     default:
       return CodeKind.COMPILATION_UNIT;
   }
 }
 
+int _parseLineLength(String lengthOption) {
+  var length = _toInt(lengthOption);
+  if (length == null) {
+    if (lengthOption.toUpperCase() == 'INF') {

[lukechurch, 2014/01/27 20:39:44]

I would recommend also supporting INFINITY - to support the use case of people
using the expanded form of the command line args.

[pquitslund, 2014/01/27 22:23:37]

Done.

+      length = -1;
+    } else {
+      throw new FormatterException('Line length is specified as an Integer or '
+          'the value "Inf".');
+    }
+  }
+  return length;
+}
+
+
 Selection _parseSelection(selectionOption) {
   if (selectionOption != null) {
     var units = selectionOption.split(',');
     if (units.length == 2) {
       var offset = _toInt(units[0]);
       var length = _toInt(units[1]);
       if (offset != null && length != null) {
         return new Selection(offset, length);
       }
     }
@@ skipping 64 matching lines @@
 /// Initialize the arg parser instance.
 ArgParser _initArgParser() {
   // NOTE: these flags are placeholders only!
   var parser = new ArgParser();
   parser.addFlag(WRITE_FLAG, abbr: 'w', negatable: false,
       help: 'Write reformatted sources to files (overwriting contents).  '
             'Do not print reformatted sources to standard output.');
   parser.addFlag(TRANSFORM_FLAG, abbr: 't', negatable: false,
       help: 'Perform code transformations.');
   parser.addOption(MAX_LINE_FLAG, abbr: 'l', defaultsTo: '80',
-      help: 'Wrap lines longer than this length.');
+      help: 'Wrap lines longer than this length. '
+            'To never wrap, specify "Inf".');
   parser.addOption(KIND_FLAG, abbr: 'k', defaultsTo: 'cu',
-      help: 'Specify source snippet kind ("stmt" or "cu")'
-            ' --- [PROVISIONAL API].', hide: true);
+      help: 'Specify source snippet kind ("stmt" or "cu") '
+            '--- [PROVISIONAL API].', hide: true);
   parser.addOption(SELECTION_FLAG, abbr: 's',
       help: 'Specify selection information as an offset,length pair '
             '(e.g., -s "0,4").', hide: true);
   parser.addFlag(MACHINE_FLAG, abbr: 'm', negatable: false,
       help: 'Produce output in a format suitable for parsing.');
   parser.addFlag(HELP_FLAG, abbr: 'h', negatable: false,
       help: 'Print this usage information.');
   return parser;
 }
 
@@ skipping 36 matching lines @@
                      'offset': formatResult.selection.offset,
                      'length': formatResult.selection.length
                   }
     });
 
 /// Log the given [msg].
 _log(String msg) {
   //TODO(pquitslund): add proper log support
   print(msg);
 }