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

Unified Diff: lib/src/line_splitting/solve_state.dart

Issue 1258203006: Handle function arguments inside function calls. (Closed) Base URL: https://github.com/dart-lang/dart_style.git@master
Patch Set: Reformat self. 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 | « lib/src/argument_list_visitor.dart ('k') | lib/src/rule/argument.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/src/line_splitting/solve_state.dart
diff --git a/lib/src/line_splitting/solve_state.dart b/lib/src/line_splitting/solve_state.dart
index 04a9b1142caa0dd8aee8bef4d5f6032c7b20459f..b4197561aa9fc9912e3018ef00391b0d09e9c78c 100644
--- a/lib/src/line_splitting/solve_state.dart
+++ b/lib/src/line_splitting/solve_state.dart
@@ -218,7 +218,7 @@ class SolveState {
// Lines that contain both bound and unbound rules must have the same
// bound values.
if (_boundRulesInUnboundLines.length !=
- other._boundRulesInUnboundLines.length) {
+ other._boundRulesInUnboundLines.length) {
return false;
}
@@ -415,7 +415,6 @@ class SolveState {
var hasUnbound = false;
for (var i = 0; i < _splitter.chunks.length - 1; i++) {
-
if (splits.shouldSplitAt(i)) {
if (hasUnbound) _boundRulesInUnboundLines.addAll(boundInLine);
@@ -439,25 +438,23 @@ class SolveState {
String toString() {
var buffer = new StringBuffer();
- buffer.writeAll(
- _splitter.rules.map((rule) {
- var valueLength = "${rule.fullySplitValue}".length;
+ buffer.writeAll(_splitter.rules.map((rule) {
+ var valueLength = "${rule.fullySplitValue}".length;
- var value = "?";
- if (_ruleValues.contains(rule)) {
- value = "${_ruleValues.getValue(rule)}";
- }
+ var value = "?";
+ if (_ruleValues.contains(rule)) {
+ value = "${_ruleValues.getValue(rule)}";
+ }
- value = value.padLeft(valueLength);
- if (_liveRules.contains(rule)) {
- value = debug.bold(value);
- } else {
- value = debug.gray(value);
- }
+ value = value.padLeft(valueLength);
+ if (_liveRules.contains(rule)) {
+ value = debug.bold(value);
+ } else {
+ value = debug.gray(value);
+ }
- return value;
- }),
- " ");
+ return value;
+ }), " ");
buffer.write(" \$${splits.cost}");
« no previous file with comments | « lib/src/argument_list_visitor.dart ('k') | lib/src/rule/argument.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698