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

Unified Diff: runtime/observatory/lib/src/cli/command.dart

Issue 2345023003: Use dartfmt on Observatory code (Closed)
Patch Set: merge Created 4 years, 3 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 side-by-side diff with in-line comments
Download patch
Index: runtime/observatory/lib/src/cli/command.dart
diff --git a/runtime/observatory/lib/src/cli/command.dart b/runtime/observatory/lib/src/cli/command.dart
index b9eebb02173de66b2928900d91903e77120c8c35..28ffbff326aec0df7f3ddf13913a7b169ac2588c 100644
--- a/runtime/observatory/lib/src/cli/command.dart
+++ b/runtime/observatory/lib/src/cli/command.dart
@@ -84,7 +84,7 @@ abstract class _CommandBase {
if (matches.isEmpty) {
return [];
} else if (matches.length == 1) {
- var childMatches = matches[0]._match(args.sublist(1), preferExact);
+ var childMatches = matches[0]._match(args.sublist(1), preferExact);
if (childMatches.isEmpty) {
return matches;
} else {
@@ -96,32 +96,31 @@ abstract class _CommandBase {
}
// Builds a list of completions for this command.
- Future<List<String>> _buildCompletions(List<String> args,
- bool addEmptyString) {
- return complete(args.sublist(_depth, args.length))
- .then((completions) {
- if (addEmptyString && completions.isEmpty &&
- args[args.length - 1] == '') {
- // Special case allowance for an empty particle at the end of
- // the command.
- completions = [''];
- }
- var prefix = _concatArgs(args, _depth);
- return completions.map((str) => '${prefix}${str}').toList();
- });
+ Future<List<String>> _buildCompletions(
+ List<String> args, bool addEmptyString) {
+ return complete(args.sublist(_depth, args.length)).then((completions) {
+ if (addEmptyString &&
+ completions.isEmpty &&
+ args[args.length - 1] == '') {
+ // Special case allowance for an empty particle at the end of
+ // the command.
+ completions = [''];
+ }
+ var prefix = _concatArgs(args, _depth);
+ return completions.map((str) => '${prefix}${str}').toList();
+ });
}
-
}
// The root of a tree of commands.
class RootCommand extends _CommandBase {
RootCommand(List<Command> children, [List<String> history])
- : this._(children, history ?? ['']);
+ : this._(children, history ?? ['']);
- RootCommand._(List<Command> children, List<String> history)
+ RootCommand._(List<Command> children, List<String> history)
: history = history,
historyPos = history.length - 1,
- super(children);
+ super(children);
// Provides a list of possible completions for a line of text.
Future<List<String>> completeCommand(String line) {
@@ -132,7 +131,7 @@ class RootCommand extends _CommandBase {
// subcommands of the last command.
args.add('');
}
- var commands = _match(args, false);
+ var commands = _match(args, false);
if (commands.isEmpty) {
// No matching commands.
return new Future.value([]);
@@ -160,11 +159,13 @@ class RootCommand extends _CommandBase {
if (matchLen == args.length) {
// If we are showing all possiblities, also include local
// completions for the parent command.
- return commands[0]._parent._buildCompletions(args, false)
- .then((localCompletions) {
- completions.addAll(localCompletions);
- return completions;
- });
+ return commands[0]
+ ._parent
+ ._buildCompletions(args, false)
+ .then((localCompletions) {
+ completions.addAll(localCompletions);
+ return completions;
+ });
}
return new Future.value(completions);
}
@@ -173,7 +174,7 @@ class RootCommand extends _CommandBase {
Future runCommand(String line) {
_historyAdvance(line);
var args = _splitLine(line);
- var commands = _match(args, true);
+ var commands = _match(args, true);
if (commands.isEmpty) {
// TODO(turnidge): Add a proper exception class for this.
return new Future.error('No such command');
« no previous file with comments | « runtime/observatory/lib/src/app/view_model.dart ('k') | runtime/observatory/lib/src/cpu_profile/cpu_profile.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698