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

Unified Diff: pkg/analyzer/lib/src/dart/element/element.dart

Issue 2330813002: Reapply abc5051854fd7c98c7ba6b593211e47dcbcec9e8: Remove 'Element.docRange'. (Closed)
Patch Set: Created 4 years, 3 months 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 side-by-side diff with in-line comments
Download patch
Index: pkg/analyzer/lib/src/dart/element/element.dart
diff --git a/pkg/analyzer/lib/src/dart/element/element.dart b/pkg/analyzer/lib/src/dart/element/element.dart
index 84d7523ced04269d27fd4e2fd58e17436735b50a..fffd3e23102706ad710c9a65be36c2a894a3aa50 100644
--- a/pkg/analyzer/lib/src/dart/element/element.dart
+++ b/pkg/analyzer/lib/src/dart/element/element.dart
@@ -584,18 +584,6 @@ class ClassElementImpl extends AbstractClassElementImpl
}
@override
- SourceRange get docRange {
- if (_unlinkedClass != null) {
- UnlinkedDocumentationComment comment =
- _unlinkedClass.documentationComment;
- return comment != null
- ? new SourceRange(comment.offset, comment.length)
- : null;
- }
- return super.docRange;
- }
-
- @override
String get documentationComment {
if (_unlinkedClass != null) {
return _unlinkedClass?.documentationComment?.text;
@@ -1821,18 +1809,6 @@ class ConstFieldElementImpl_EnumValue extends ConstFieldElementImpl_ofEnum {
: super(enumElement);
@override
- SourceRange get docRange {
- if (_unlinkedEnumValue != null) {
- UnlinkedDocumentationComment comment =
- _unlinkedEnumValue.documentationComment;
- return comment != null
- ? new SourceRange(comment.offset, comment.length)
- : null;
- }
- return super.docRange;
- }
-
- @override
String get documentationComment {
if (_unlinkedEnumValue != null) {
return _unlinkedEnumValue?.documentationComment?.text;
@@ -2700,17 +2676,6 @@ abstract class ElementImpl implements Element {
String _docComment;
/**
- * The offset to the beginning of the documentation comment,
- * or `null` if this element does not have a documentation comment.
- */
- int _docRangeOffset;
-
- /**
- * The length of the documentation comment range for this element.
- */
- int _docRangeLength;
-
- /**
* The offset of the beginning of the element's code in the file that contains
* the element, or `null` if the element is synthetic.
*/
@@ -2763,14 +2728,6 @@ abstract class ElementImpl implements Element {
String get displayName => _name;
@override
- SourceRange get docRange {
- if (_docRangeOffset != null && _docRangeLength != null) {
- return new SourceRange(_docRangeOffset, _docRangeLength);
- }
- return null;
- }
-
- @override
String get documentationComment => _docComment;
/**
@@ -3115,15 +3072,6 @@ abstract class ElementImpl implements Element {
}
/**
- * Set the documentation comment source range for this element.
- */
- void setDocRange(int offset, int length) {
- assert(!isResynthesized);
- _docRangeOffset = offset;
- _docRangeLength = length;
- }
-
- /**
* Set whether the given [modifier] is associated with this element to
* correspond to the given [value].
*/
@@ -3409,17 +3357,6 @@ class EnumElementImpl extends AbstractClassElementImpl {
}
@override
- SourceRange get docRange {
- if (_unlinkedEnum != null) {
- UnlinkedDocumentationComment comment = _unlinkedEnum.documentationComment;
- return comment != null
- ? new SourceRange(comment.offset, comment.length)
- : null;
- }
- return super.docRange;
- }
-
- @override
String get documentationComment {
if (_unlinkedEnum != null) {
return _unlinkedEnum?.documentationComment?.text;
@@ -3673,18 +3610,6 @@ abstract class ExecutableElementImpl extends ElementImpl
}
@override
- SourceRange get docRange {
- if (serializedExecutable != null) {
- UnlinkedDocumentationComment comment =
- serializedExecutable.documentationComment;
- return comment != null
- ? new SourceRange(comment.offset, comment.length)
- : null;
- }
- return super.docRange;
- }
-
- @override
String get documentationComment {
if (serializedExecutable != null) {
return serializedExecutable?.documentationComment?.text;
@@ -4651,18 +4576,6 @@ class FunctionTypeAliasElementImpl extends ElementImpl
String get displayName => name;
@override
- SourceRange get docRange {
- if (_unlinkedTypedef != null) {
- UnlinkedDocumentationComment comment =
- _unlinkedTypedef.documentationComment;
- return comment != null
- ? new SourceRange(comment.offset, comment.length)
- : null;
- }
- return super.docRange;
- }
-
- @override
String get documentationComment {
if (_unlinkedTypedef != null) {
return _unlinkedTypedef?.documentationComment?.text;
@@ -5420,18 +5333,6 @@ class LibraryElementImpl extends ElementImpl implements LibraryElement {
}
@override
- SourceRange get docRange {
- if (_unlinkedDefiningUnit != null) {
- UnlinkedDocumentationComment comment =
- _unlinkedDefiningUnit.libraryDocumentationComment;
- return comment != null
- ? new SourceRange(comment.offset, comment.length)
- : null;
- }
- return super.docRange;
- }
-
- @override
String get documentationComment {
if (_unlinkedDefiningUnit != null) {
return _unlinkedDefiningUnit?.libraryDocumentationComment?.text;
@@ -6490,9 +6391,6 @@ class MultiplyDefinedElementImpl implements MultiplyDefinedElement {
String get displayName => _name;
@override
- SourceRange get docRange => null;
-
- @override
String get documentationComment => null;
@override
@@ -6776,18 +6674,6 @@ abstract class NonParameterVariableElementImpl extends VariableElementImpl {
}
@override
- SourceRange get docRange {
- if (_unlinkedVariable != null) {
- UnlinkedDocumentationComment comment =
- _unlinkedVariable.documentationComment;
- return comment != null
- ? new SourceRange(comment.offset, comment.length)
- : null;
- }
- return super.docRange;
- }
-
- @override
String get documentationComment {
if (_unlinkedVariable != null) {
return _unlinkedVariable?.documentationComment?.text;

Powered by Google App Engine
This is Rietveld 408576698