formatting the formatter

Author: kevmoo

benchmark/directory.dart

@@ -63,10 +63,9 @@ void main(List<String> arguments) async {
 void _runFormatter(String source) {
   try {
     var formatter = DartFormatter(
-      languageVersion:
-          _isShort
-              ? DartFormatter.latestShortStyleLanguageVersion
-              : DartFormatter.latestLanguageVersion,
+      languageVersion: _isShort
+          ? DartFormatter.latestShortStyleLanguageVersion
+          : DartFormatter.latestLanguageVersion,
     );
 
     var result = formatter.format(source);

benchmark/run.dart

@@ -137,10 +137,9 @@ List<double> _runTrials(String verb, Benchmark benchmark, int trials) {
   );
 
   var formatter = DartFormatter(
-    languageVersion:
-        _isShort
-            ? DartFormatter.latestShortStyleLanguageVersion
-            : DartFormatter.latestLanguageVersion,
+    languageVersion: _isShort
+        ? DartFormatter.latestShortStyleLanguageVersion
+        : DartFormatter.latestLanguageVersion,
     pageWidth: benchmark.pageWidth,
     lineEnding: '\n',
   );

lib/src/back_end/code_writer.dart

@@ -370,11 +370,10 @@ final class CodeWriter {
       // indication into account which may vary based on the surrounding pieces
       // when we get here.
       Profile.begin('CodeWriter try to bind by page width');
-      isUnsolved =
-          !_solution.tryBindByPageWidth(
-            piece,
-            _pageWidth - _indentStack.first.spaces,
-          );
+      isUnsolved = !_solution.tryBindByPageWidth(
+        piece,
+        _pageWidth - _indentStack.first.spaces,
+      );
       Profile.end('CodeWriter try to bind by page width');
     }
 

lib/src/cli/format_command.dart

@@ -183,20 +183,18 @@ final class FormatCommand extends Command<int> {
       return 0;
     }
 
-    var show =
-        const {
-          'all': Show.all,
-          'changed': Show.changed,
-          'none': Show.none,
-        }[argResults['show']]!;
-
-    var output =
-        const {
-          'write': Output.write,
-          'show': Output.show,
-          'none': Output.none,
-          'json': Output.json,
-        }[argResults['output']]!;
+    var show = const {
+      'all': Show.all,
+      'changed': Show.changed,
+      'none': Show.none,
+    }[argResults['show']]!;
+
+    var output = const {
+      'write': Output.write,
+      'show': Output.show,
+      'none': Output.none,
+      'json': Output.json,
+    }[argResults['output']]!;
 
     var summary = Summary.none;
     switch (argResults['summary'] as String) {

lib/src/dart_formatter.dart

@@ -140,10 +140,9 @@ final class DartFormatter {
         text,
         uri: source.uri,
         isCompilationUnit: false,
-        selectionStart:
-            source.selectionStart != null
-                ? source.selectionStart! + inputOffset
-                : null,
+        selectionStart: source.selectionStart != null
+            ? source.selectionStart! + inputOffset
+            : null,
         selectionLength: source.selectionLength,
       );
     }
@@ -176,10 +175,9 @@ final class DartFormatter {
     }
 
     // Throw if there are syntactic errors.
-    var syntacticErrors =
-        parseResult.errors.where((error) {
-          return error.diagnosticCode.type == DiagnosticType.SYNTACTIC_ERROR;
-        }).toList();
+    var syntacticErrors = parseResult.errors.where((error) {
+      return error.diagnosticCode.type == DiagnosticType.SYNTACTIC_ERROR;
+    }).toList();
     if (syntacticErrors.isNotEmpty) {
       throw FormatterException(syntacticErrors);
     }

lib/src/front_end/ast_node_visitor.dart

@@ -1794,10 +1794,9 @@ final class AstNodeVisitor extends ThrowingAstVisitor<void> with PieceFactory {
     var isSinglePositional =
         positionalFields.length == 1 && namedFields == null;
 
-    var listStyle =
-        isSinglePositional
-            ? const ListStyle(commas: Commas.alwaysTrailing)
-            : const ListStyle(commas: Commas.trailing);
+    var listStyle = isSinglePositional
+        ? const ListStyle(commas: Commas.alwaysTrailing)
+        : const ListStyle(commas: Commas.trailing);
     var builder = DelimitedListBuilder(this, listStyle);
 
     // If all parameters are optional, put the `{` right after `(`.
@@ -1963,10 +1962,9 @@ final class AstNodeVisitor extends ThrowingAstVisitor<void> with PieceFactory {
     for (var element in node.elements) {
       pieces.visit(
         element,
-        context:
-            node.isMultiline
-                ? NodeContext.multilineStringInterpolation
-                : NodeContext.none,
+        context: node.isMultiline
+            ? NodeContext.multilineStringInterpolation
+            : NodeContext.none,
       );
     }
   }

lib/src/front_end/delimited_list_builder.dart

@@ -269,7 +269,10 @@ final class DelimitedListBuilder {
       hanging: hangingComments,
       separate: separateComments,
       leading: leadingComments,
-    ) = _splitCommaComments(comments, hasElementAfter: hasElementAfter);
+    ) = _splitCommaComments(
+      comments,
+      hasElementAfter: hasElementAfter,
+    );
 
     // Add any hanging inline block comments to the previous element before the
     // subsequent ",".
@@ -401,10 +404,8 @@ final class DelimitedListBuilder {
       }
     }
 
-    var (
-      hangingComments,
-      separateCommentsBeforeComma,
-    ) = remainingCommentsBeforeComma.splitAt(hangingCommentCount);
+    var (hangingComments, separateCommentsBeforeComma) =
+        remainingCommentsBeforeComma.splitAt(hangingCommentCount);
 
     // Inline block comments on the same line as the next element lead at the
     // beginning of that line, as in:

lib/src/front_end/piece_factory.dart

@@ -856,10 +856,9 @@ mixin PieceFactory {
         pieces.token(question);
       }
 
-      var returnTypeModifiers =
-          parameter != null
-              ? [parameter.requiredKeyword, parameter.covariantKeyword]
-              : const <Token?>[];
+      var returnTypeModifiers = parameter != null
+          ? [parameter.requiredKeyword, parameter.covariantKeyword]
+          : const <Token?>[];
 
       // If the type is a function-typed parameter with a default value, then
       // grab the default value from the parent node and attach it to the

lib/src/front_end/piece_writer.dart

@@ -208,10 +208,9 @@ final class PieceWriter {
     var builtPieces = _pieces.removeLast();
     assert(builtPieces.isNotEmpty);
 
-    var builtPiece =
-        builtPieces.length == 1
-            ? builtPieces.first
-            : AdjacentPiece(builtPieces);
+    var builtPiece = builtPieces.length == 1
+        ? builtPieces.first
+        : AdjacentPiece(builtPieces);
 
     if (leadingPieces.isEmpty) {
       // No metadata, so return the content piece directly.

lib/src/short/line_splitting/solve_state.dart

@@ -407,8 +407,9 @@ final class SolveState {
       if (chunk is BlockChunk) {
         if (_splits.shouldSplitAt(i)) {
           // Include the cost of the nested block.
-          cost +=
-              _splitter.writer.formatBlock(chunk, _splits.getColumn(i)).cost;
+          cost += _splitter.writer
+              .formatBlock(chunk, _splits.getColumn(i))
+              .cost;
         } else {
           // Include the nested block inline, if any.
           length += chunk.unsplitBlockLength;