Bob Nystrom comments

Results 973 comments of


                                            Bob Nystrom

bizarre parameter wrapping when followed by block comment

Yeah, that's pretty wonky.

bizarre parameter wrapping when followed by block comment

I have no idea what the old formatter is thinking, but the [forthcoming](https://github.com/dart-lang/dart_style/issues/1403) [tall style](https://github.com/dart-lang/dart_style/issues/1253) formatter produces: ```dart int foo(List list) /* 1234567 14 21 28 35 42 49 */...

Deprecate "dartfmt" in favor of "dart format"

Timing note: We want dartfmt printing the deprecation notice before 2.14 ships on stable (though we don't plan to remove support in 2.14).

Deprecate "dartfmt" in favor of "dart format"

I think the cost of exposing an executable in the package is relatively low. And I have some vague anticipation that the new change process for dart format may open...

Deprecate "dartfmt" in favor of "dart format"

This is also tracking the deprecation and removal of the old CLI from the dart_style package itself, so I'm going to leave it open until that's complete. If that makes...

Deprecate "dartfmt" in favor of "dart format"

No, this issue doesn't affect the `dart` CLI at all now. All it pertains to is the `dartfmt` command exposed by the `dart_style` package itself if you `dart pub global...

Incorrect indentation of trailing comma argument list in switch expression

Also, this one is even worse: ```dart main() { { return TextFieldTapRegion( onLongPressMoveUpdate: (longPressMoveUpdateDetails) => switch (Theme.of(this.context).platform) { TargetPlatform.iOS || TargetPlatform.macOS => _renderEditable.selectPositionAt( from: longPressMoveUpdateDetails.globalPosition, cause: SelectionChangedCause.longPress, ), TargetPlatform.android ||...