Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(738)

Side by Side Diff: pkg/args/lib/src/usage.dart

Issue 12401002: Make List.from and Iterable.toList default to not growable. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « pkg/args/lib/src/parser.dart ('k') | pkg/pathos/lib/path.dart » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 library args.src.usage; 5 library args.src.usage;
6 6
7 import 'dart:math'; 7 import 'dart:math';
8 8
9 import '../args.dart'; 9 import '../args.dart';
10 10
(...skipping 55 matching lines...) Expand 10 before | Expand all | Expand 10 after
66 66
67 calculateColumnWidths(); 67 calculateColumnWidths();
68 68
69 args.options.forEach((name, option) { 69 args.options.forEach((name, option) {
70 write(0, getAbbreviation(option)); 70 write(0, getAbbreviation(option));
71 write(1, getLongOption(option)); 71 write(1, getLongOption(option));
72 72
73 if (option.help != null) write(2, option.help); 73 if (option.help != null) write(2, option.help);
74 74
75 if (option.allowedHelp != null) { 75 if (option.allowedHelp != null) {
76 var allowedNames = option.allowedHelp.keys.toList(); 76 var allowedNames = option.allowedHelp.keys.toList(growable: false);
77 allowedNames.sort(); 77 allowedNames.sort();
78 newline(); 78 newline();
79 for (var name in allowedNames) { 79 for (var name in allowedNames) {
80 write(1, getAllowedTitle(name)); 80 write(1, getAllowedTitle(name));
81 write(2, option.allowedHelp[name]); 81 write(2, option.allowedHelp[name]);
82 } 82 }
83 newline(); 83 newline();
84 } else if (option.allowed != null) { 84 } else if (option.allowed != null) {
85 write(2, buildAllowedList(option)); 85 write(2, buildAllowedList(option));
86 } else if (option.defaultValue != null) { 86 } else if (option.defaultValue != null) {
(...skipping 142 matching lines...) Expand 10 before | Expand all | Expand 10 after
229 String padRight(String source, int length) { 229 String padRight(String source, int length) {
230 final result = new StringBuffer(); 230 final result = new StringBuffer();
231 result.write(source); 231 result.write(source);
232 232
233 while (result.length < length) { 233 while (result.length < length) {
234 result.write(' '); 234 result.write(' ');
235 } 235 }
236 236
237 return result.toString(); 237 return result.toString();
238 } 238 }
OLDNEW
« no previous file with comments | « pkg/args/lib/src/parser.dart ('k') | pkg/pathos/lib/path.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698