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

Unified Diff: test/splitting/function_arguments.stmt

Issue 1252323003: Allow arguments before and after block-formatted functions. (Closed) Base URL: https://github.com/dart-lang/dart_style.git@master
Patch Set: Update pubspec and changelog. Created 5 years, 5 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
« no previous file with comments | « test/splitting/arguments.stmt ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/splitting/function_arguments.stmt
diff --git a/test/splitting/function_arguments.stmt b/test/splitting/function_arguments.stmt
index 450b53a183411c37f1b1bbd8c4d0984b33aeb073..3da9e718b3f2adde1a17e554153ab04716d5903d 100644
--- a/test/splitting/function_arguments.stmt
+++ b/test/splitting/function_arguments.stmt
@@ -1,13 +1,10 @@
40 columns |
->>> args before and after function forces nesting
+>>> args before and after function do not force nesting
method(first,() {fn;},third);
<<<
-method(
- first,
- () {
- fn;
- },
- third);
+method(first, () {
+ fn;
+}, third);
>>> nothing but function args does not nest
longFunctionName(() {;}, () {;}, () {;});
<<<
@@ -168,14 +165,21 @@ obj.outer(
obj.inner(() {
body;
}));
->>> force named args to split on positional function
+>>> do not force named args to split on positional function
function(argument, () {;},
named: argument, another: argument);
<<<
-function(
- argument,
- () {
- ;
- },
- named: argument,
- another: argument);
+function(argument, () {
+ ;
+}, named: argument, another: argument);
+>>> args before and after functions split independently
+longFunction(argument, argument, argument, argument, argument,
+() {;}, () {;}, argument, argument, argument, argument, argument);
+<<<
+longFunction(argument, argument,
+ argument, argument, argument, () {
+ ;
+}, () {
+ ;
+}, argument, argument, argument,
+ argument, argument);
« no previous file with comments | « test/splitting/arguments.stmt ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698