Smarter splitting around named collection arguments.

lib/src/line_splitting/solve_state.dart

 // Copyright (c) 2015, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 library dart_style.src.line_splitting.solve_state;
 
 import '../debug.dart' as debug;
 import '../rule/rule.dart';
 import '../whitespace.dart';
 import 'line_splitter.dart';
@@ skipping 79 matching lines @@
   /// invalid constraints on the currently bound values.
   ///
   /// For example, say rule A is bound to 1 and B is unbound. If B has value
   /// 1, it constrains A to be 1. Likewise, if B is 2, it constrains A to be
   /// 2 as well. Since we already know A is 1, that means we know B cannot be
   /// bound to value 2. This means B is more limited in this state than it
   /// might be in another state that binds A to a different value.
   ///
   /// It's important to track this, because we can't allow to states to overlap
   /// if one permits more values for some unbound rule than the other does.
-  Map<Rule, List<int>> _unboundConstraints;
+  Map<Rule, Set<int>> _unboundConstraints;
 
   /// The bound rules that appear inside lines also containing unbound rules.
   ///
   /// By appearing in the same line, it means these bound rules may affect the
   /// results of binding those unbound rules. This is used to tell if two
   /// states may diverge by binding unbound rules or not.
   Set<Rule> _boundRulesInUnboundLines;
 
   SolveState(this._splitter, this._ruleValues) {
     _calculateSplits();
@@ skipping 474 matching lines @@
 
     buffer.write("   \$${splits.cost}");
 
     if (overflowChars > 0) buffer.write(" (${overflowChars} over)");
     if (!_isComplete) buffer.write(" (incomplete)");
     if (splits == null) buffer.write(" invalid");
 
     return buffer.toString();
   }
 }