Chromium Code Reviews| OLD | NEW |
|---|---|
| (Empty) | |
| 1 # 1.0.0 | |
| 2 | |
| 3 This package was extracted from the | |
| 4 [`source_maps`](http://pub.dartlang.org/packages/source_maps) package, but the | |
| 5 API has many differences. Among them: | |
| 6 | |
| 7 * `Span` has been renamed to `SourceSpan` and `Location` has been renamed to | |
| 8 `SourceLocation` to clarify their purpose and maintain consistency with the | |
| 9 package name. Likewise, `SpanException` is now `SourceSpanException` and | |
| 10 `SpanFormatException` is not `SourceSpanFormatException`. | |
| 11 | |
| 12 * `FixedSpan` and `FixedLocation` have been rolled into the `Span` and | |
| 13 `Location` classes, respectively. | |
| 14 | |
| 15 * `SourceFile` is more aggressive about validating its arguments. Out-of-bounds | |
| 16 lines, columns, and offsets will now throw errors rather than be silently | |
| 17 clamped. | |
| 18 | |
| 19 * `SourceSpan.sourceUrl`, `SourceLocation.sourceUrl`, and `SourceFile.url` now | |
| 20 return `Uri` objects rather than `String`s. The constructors allow either | |
| 21 `String`s or `Uri`s. | |
| 22 | |
| 23 * `Span.getLocationMessage` and `SourceFile.getLocationMessage` are now | |
| 24 `SourceSpan.message` and `SourceFile.message`, respectively. Rather than | |
| 25 taking both a `useColor` and a `color` parameter, they now take a single | |
| 26 `color` parameter that controls both whether and which color is used. | |
| 27 | |
| 28 * `Span.isIdentifier` has been removed. This property doesn't make sense outside | |
| 29 of a source map context. | |
| 30 | |
| 31 * `SourceFileSegment` has been removed. This class wasn't widely used and was | |
|
Siggi Cherem (dart-lang)
2014/07/11 17:01:32
I don't think we should remove this entirely. This
nweiz
2014/07/11 18:41:41
As it exists currently in source_maps, it's not a
| |
| 32 inconsistent in its choice of which parameters were considered relative and | |
| 33 which absolute. | |
| OLD | NEW |