dartfmt gets opinionated about 'gratuitous' linebreaks.

dartfmt gets opinionated about 'gratuitous' linebreaks.

IOW: we now collapse lines that are broken.

NOTE: EOL comments are sure to break this; CL to address to follow.

R=brianwilkerson@google.com

Committed: https://code.google.com/p/dart/source/detail?r=31243

[pquitslund, 2013-12-18 17:16:47]

More babysteps... :)

https://codereview.chromium.org/113693003/diff/20001/pkg/analyzer/lib/src/ser...
File pkg/analyzer/lib/src/services/formatter_impl.dart (right):

https://codereview.chromium.org/113693003/diff/20001/pkg/analyzer/lib/src/ser...
pkg/analyzer/lib/src/services/formatter_impl.dart:1324: //TODO(pquitslund):
handle this more neatly
EG: by introducing a const pointer to the space function.

[Brian Wilkerson, 2013-12-18 17:53:07]

LGTM

https://codereview.chromium.org/113693003/diff/20001/pkg/analyzer/lib/src/ser...
File pkg/analyzer/lib/src/services/formatter_impl.dart (right):

https://codereview.chromium.org/113693003/diff/20001/pkg/analyzer/lib/src/ser...
pkg/analyzer/lib/src/services/formatter_impl.dart:1324: //TODO(pquitslund):
handle this more neatly
> EG: by introducing a const pointer to the space function.

In order to do that, the space function will have to be a static function. But
if it's a static function, then it's already a constant, so you don't need an
additional const variable to point to it and should be able to write
"{followedBy(): space}".

[pquitslund, 2013-12-18 22:57:57]

Committed patchset #2 manually as r31243 (presubmit successful).