Skip to content

Commit efd2af0

Browse files
scheglovCommit Queue
authored andcommitted
Elements. Remove 'unitElement' from _readMetadata().
Change-Id: I3eb65eb69250f52df87c65f8466772141f7aba9c Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/445110 Commit-Queue: Konstantin Shcheglov <[email protected]> Reviewed-by: Johnni Winther <[email protected]>
1 parent 944ff08 commit efd2af0

File tree

1 file changed

+13
-14
lines changed

1 file changed

+13
-14
lines changed

pkg/analyzer/lib/src/summary2/bundle_reader.dart

Lines changed: 13 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -188,7 +188,7 @@ class LibraryReader {
188188
);
189189
reader.currentLibraryFragment = unitElement;
190190

191-
_libraryElement.metadata = reader._readMetadata(unitElement: unitElement);
191+
_libraryElement.metadata = reader._readMetadata();
192192

193193
_libraryElement.entryPoint =
194194
reader.readElement() as TopLevelFunctionElementImpl?;
@@ -729,7 +729,7 @@ class LibraryReader {
729729
List<FormalParameterFragmentImpl> parameters,
730730
) {
731731
for (var parameter in parameters) {
732-
parameter.metadata = reader._readMetadata(unitElement: unitElement);
732+
parameter.metadata = reader._readMetadata();
733733
_readTypeParameters2(unitElement, reader, parameter.typeParameters);
734734
_readFormalParameters2(unitElement, reader, parameter.formalParameters);
735735
parameter.element.inheritsCovariant = reader.readBool();
@@ -758,8 +758,8 @@ class LibraryReader {
758758
T fragment,
759759
ResolutionReader reader,
760760
) {
761-
var libraryFragment = fragment.libraryFragment as LibraryFragmentImpl;
762-
fragment.metadata = reader._readMetadata(unitElement: libraryFragment);
761+
// TODO(scheglov): inline it
762+
fragment.metadata = reader._readMetadata();
763763
}
764764

765765
String? _readFragmentName() {
@@ -1335,7 +1335,7 @@ class LibraryReader {
13351335
) {
13361336
reader._addTypeParameters(typeParameters);
13371337
for (var typeParameter in typeParameters) {
1338-
typeParameter.metadata = reader._readMetadata(unitElement: unitElement);
1338+
typeParameter.metadata = reader._readMetadata();
13391339
typeParameter.element.bound = reader.readType();
13401340
typeParameter.element.defaultType = reader.readType();
13411341
}
@@ -1363,23 +1363,23 @@ class LibraryReader {
13631363
reader.currentLibraryFragment = unitElement;
13641364

13651365
for (var import in unitElement.libraryImports) {
1366-
import.metadata = reader._readMetadata(unitElement: unitElement);
1366+
import.metadata = reader._readMetadata();
13671367
var uri = import.uri;
13681368
if (uri is DirectiveUriWithLibraryImpl) {
13691369
uri.library = reader.libraryOfUri(uri.source.uri);
13701370
}
13711371
}
13721372

13731373
for (var export in unitElement.libraryExports) {
1374-
export.metadata = reader._readMetadata(unitElement: unitElement);
1374+
export.metadata = reader._readMetadata();
13751375
var uri = export.uri;
13761376
if (uri is DirectiveUriWithLibraryImpl) {
13771377
uri.library = reader.libraryOfUri(uri.source.uri);
13781378
}
13791379
}
13801380

13811381
for (var part in unitElement.parts) {
1382-
part.metadata = reader._readMetadata(unitElement: unitElement);
1382+
part.metadata = reader._readMetadata();
13831383
}
13841384
});
13851385

@@ -1546,7 +1546,7 @@ class ResolutionReader {
15461546
}
15471547

15481548
MetadataImpl readMetadata() {
1549-
return _readMetadata(unitElement: currentLibraryFragment);
1549+
return _readMetadata();
15501550
}
15511551

15521552
List<T> readNodeList<T>() {
@@ -1794,7 +1794,7 @@ class ResolutionReader {
17941794
// TODO(scheglov): reuse for formal parameters
17951795
_localElements.length -= typeParameters.length;
17961796
if (unitElement != null) {
1797-
element.metadata = _readMetadata(unitElement: unitElement);
1797+
element.metadata = _readMetadata();
17981798
}
17991799
return element;
18001800
});
@@ -1831,11 +1831,10 @@ class ResolutionReader {
18311831
return readTypedList(_readInterfaceType);
18321832
}
18331833

1834-
MetadataImpl _readMetadata({required LibraryFragmentImpl unitElement}) {
1835-
currentLibraryFragment = unitElement;
1834+
MetadataImpl _readMetadata() {
18361835
var annotations = readTypedList(() {
18371836
var ast = _readRequiredNode() as AnnotationImpl;
1838-
return ElementAnnotationImpl(unitElement, ast);
1837+
return ElementAnnotationImpl(currentLibraryFragment, ast);
18391838
});
18401839

18411840
return MetadataImpl(annotations);
@@ -1905,7 +1904,7 @@ class ResolutionReader {
19051904
for (var typeParameter in typeParameters) {
19061905
typeParameter.element.bound = readType();
19071906
if (unitElement != null) {
1908-
typeParameter.metadata = _readMetadata(unitElement: unitElement);
1907+
typeParameter.metadata = _readMetadata();
19091908
}
19101909
}
19111910
return typeParameters;

0 commit comments

Comments
 (0)