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

Side by Side Diff: pkg/source_span/lib/src/span_mixin.dart

Issue 754463002: Avoid instantiating FileLocations where possible in source_span. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Code review changes Created 6 years, 1 month 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « pkg/source_span/lib/src/file.dart ('k') | pkg/source_span/pubspec.yaml » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 library source_span.span_mixin; 5 library source_span.span_mixin;
6 6
7 import 'package:path/path.dart' as p; 7 import 'package:path/path.dart' as p;
8 8
9 import 'colors.dart' as colors; 9 import 'colors.dart' as colors;
10 import 'span.dart'; 10 import 'span.dart';
11 import 'utils.dart'; 11 import 'utils.dart';
12 12
13 /// A mixin for easily implementing [SourceSpan]. 13 /// A mixin for easily implementing [SourceSpan].
14 /// 14 ///
15 /// This implements the [SourceSpan] methods in terms of [start], [end], and 15 /// This implements the [SourceSpan] methods in terms of [start], [end], and
16 /// [text]. This assumes that [start] and [end] have the same source URL, that 16 /// [text]. This assumes that [start] and [end] have the same source URL, that
17 /// [start] comes before [end], and that [text] has a number of characters equal 17 /// [start] comes before [end], and that [text] has a number of characters equal
18 /// to the distance between [start] and [end]. 18 /// to the distance between [start] and [end].
19 abstract class SourceSpanMixin implements SourceSpan { 19 abstract class SourceSpanMixin implements SourceSpan {
20 Uri get sourceUrl => start.sourceUrl; 20 Uri get sourceUrl => start.sourceUrl;
21 int get length => end.offset - start.offset; 21 int get length => end.offset - start.offset;
22 22
23 int compareTo(SourceSpan other) { 23 int compareTo(SourceSpan other) {
24 int d = start.compareTo(other.start); 24 var result = start.compareTo(other.start);
25 return d == 0 ? end.compareTo(other.end) : d; 25 return result == 0 ? end.compareTo(other.end) : result;
26 } 26 }
27 27
28 SourceSpan union(SourceSpan other) { 28 SourceSpan union(SourceSpan other) {
29 if (sourceUrl != other.sourceUrl) { 29 if (sourceUrl != other.sourceUrl) {
30 throw new ArgumentError("Source URLs \"${sourceUrl}\" and " 30 throw new ArgumentError("Source URLs \"${sourceUrl}\" and "
31 " \"${other.sourceUrl}\" don't match."); 31 " \"${other.sourceUrl}\" don't match.");
32 } 32 }
33 33
34 var start = min(this.start, other.start); 34 var start = min(this.start, other.start);
35 var end = max(this.end, other.end); 35 var end = max(this.end, other.end);
(...skipping 22 matching lines...) Expand all
58 buffer.write("\n"); 58 buffer.write("\n");
59 var textLine = text.split("\n").first; 59 var textLine = text.split("\n").first;
60 if (color != null) buffer.write(color); 60 if (color != null) buffer.write(color);
61 buffer.write(textLine); 61 buffer.write(textLine);
62 buffer.write("\n"); 62 buffer.write("\n");
63 buffer.write('^' * textLine.length); 63 buffer.write('^' * textLine.length);
64 if (color != null) buffer.write(colors.NONE); 64 if (color != null) buffer.write(colors.NONE);
65 return buffer.toString(); 65 return buffer.toString();
66 } 66 }
67 67
68 bool operator ==(SourceSpan other) => 68 bool operator ==(other) => other is SourceSpan &&
69 start == other.start && end == other.end; 69 start == other.start && end == other.end;
70 70
71 int get hashCode => start.hashCode + (31 * end.hashCode); 71 int get hashCode => start.hashCode + (31 * end.hashCode);
72 72
73 String toString() => '<$runtimeType: from $start to $end "$text">'; 73 String toString() => '<$runtimeType: from $start to $end "$text">';
74 } 74 }
OLDNEW
« no previous file with comments | « pkg/source_span/lib/src/file.dart ('k') | pkg/source_span/pubspec.yaml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698