Chromium Code Reviews

Side by Side Diff: CHANGELOG.md

Issue 1298093002: Support external implementations of FileSpan. (Closed) Base URL: git@github.com:dart-lang/source_span@master
Patch Set: Code review changes Created 5 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff |
« no previous file with comments | « no previous file | lib/src/file.dart » ('j') | lib/src/file.dart » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # 1.1.3
2
3 * `FileSpan.compareTo`, `FileSpan.==`, `FileSpan.union`, and `FileSpan.expand`
4 no longer throw exceptions for external implementations of `FileSpan`.
5
6 * `FileSpan.hashCode` now fully agrees with `FileSpan.==`.
Siggi Cherem (dart-lang) 2015/08/18 21:38:36 I think before that was true too, maybe it's more
nweiz 2015/08/18 22:01:25 Before, a FileSpan == a SourceSpan with the same s
7
1 # 1.1.2 8 # 1.1.2
2 9
3 * Fixed validation in `SourceSpanWithContext` to allow multiple occurrences of 10 * Fixed validation in `SourceSpanWithContext` to allow multiple occurrences of
4 `text` within `context`. 11 `text` within `context`.
5 12
6 # 1.1.1 13 # 1.1.1
7 14
8 * Fixed `FileSpan`'s context to include the full span text, not just the first 15 * Fixed `FileSpan`'s context to include the full span text, not just the first
9 line of it. 16 line of it.
10 17
(...skipping 44 matching lines...)
55 `SourceSpan.message` and `SourceFile.message`, respectively. Rather than 62 `SourceSpan.message` and `SourceFile.message`, respectively. Rather than
56 taking both a `useColor` and a `color` parameter, they now take a single 63 taking both a `useColor` and a `color` parameter, they now take a single
57 `color` parameter that controls both whether and which color is used. 64 `color` parameter that controls both whether and which color is used.
58 65
59 * `Span.isIdentifier` has been removed. This property doesn't make sense outside 66 * `Span.isIdentifier` has been removed. This property doesn't make sense outside
60 of a source map context. 67 of a source map context.
61 68
62 * `SourceFileSegment` has been removed. This class wasn't widely used and was 69 * `SourceFileSegment` has been removed. This class wasn't widely used and was
63 inconsistent in its choice of which parameters were considered relative and 70 inconsistent in its choice of which parameters were considered relative and
64 which absolute. 71 which absolute.
OLDNEW
« no previous file with comments | « no previous file | lib/src/file.dart » ('j') | lib/src/file.dart » ('J')

Powered by Google App Engine