Chromium Code Reviews| Index: pkg/analyzer/lib/src/services/writer.dart |
| =================================================================== |
| --- pkg/analyzer/lib/src/services/writer.dart (revision 32131) |
| +++ pkg/analyzer/lib/src/services/writer.dart (working copy) |
| @@ -232,7 +232,7 @@ |
| } |
| } |
| - bool fits(LineText text) => length + text.length < maxLength; |
| + bool fits(LineText text) => length + text.length <= maxLength; |
| int get length => start.value.length + buffer.length; |
| @@ -257,11 +257,15 @@ |
| String toString() => value; |
| - int get length => value.length; |
| + int get length => lengthLessNewlines(value); |
| void addTo(Chunk chunk) { |
| chunk.buffer.write(value); |
| } |
| + |
| + int lengthLessNewlines(String str) => |
| + str.endsWith('\n') ? str.length - 1 : str.length; |
|
Brian Wilkerson
2014/01/30 00:51:20
Do we also need to test for "\r\n"?
|
| + |
| } |