Skip to content

Commit e22f0e8

Browse files
scheglovCommit Queue
authored andcommitted
Fine. Rename BundleRequirementsManifest to RequirementsManifest.
Also rename `linkingBundleManifest` to `globalResultRequirements`. Change-Id: Ia2ce5186d1be6eb97f3782ae2b08d1f4f4ddf52e Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/422240 Commit-Queue: Konstantin Shcheglov <[email protected]> Reviewed-by: Paul Berry <[email protected]>
1 parent d057a33 commit e22f0e8

File tree

9 files changed

+210
-211
lines changed

9 files changed

+210
-211
lines changed

pkg/analyzer/lib/src/dart/analysis/driver.dart

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1397,10 +1397,10 @@ class AnalysisDriver {
13971397
libraryElement.typeSystem,
13981398
strictCasts: analysisOptions.strictCasts);
13991399

1400-
BundleRequirementsManifest? libraryRequirements;
1400+
RequirementsManifest? requirements;
14011401
if (withFineDependencies) {
1402-
libraryRequirements = BundleRequirementsManifest();
1403-
linkingBundleManifest = libraryRequirements;
1402+
requirements = RequirementsManifest();
1403+
globalResultRequirements = requirements;
14041404
}
14051405

14061406
var results = performance.run('LibraryAnalyzer', (performance) {
@@ -1418,7 +1418,7 @@ class AnalysisDriver {
14181418
});
14191419

14201420
if (withFineDependencies) {
1421-
linkingBundleManifest = null;
1421+
globalResultRequirements = null;
14221422
}
14231423

14241424
var isLibraryWithPriorityFile = _isLibraryWithPriorityFile(library);
@@ -1479,7 +1479,7 @@ class AnalysisDriver {
14791479
_updateHasErrorOrWarningFlag(unitFile, resolvedUnit.errors);
14801480
}
14811481

1482-
if (withFineDependencies && libraryRequirements != null) {
1482+
if (withFineDependencies && requirements != null) {
14831483
performance.run('writeResolvedLibrary', (_) {
14841484
var mapSink = BufferedSink();
14851485
mapSink.writeMap(
@@ -1490,12 +1490,12 @@ class AnalysisDriver {
14901490
var mapBytes = mapSink.takeBytes();
14911491

14921492
library.lastResolutionResult = LibraryResolutionResult(
1493-
requirements: libraryRequirements!,
1493+
requirements: requirements!,
14941494
bytes: mapBytes,
14951495
);
14961496

14971497
var sink = BufferedSink();
1498-
libraryRequirements.write(sink);
1498+
requirements.write(sink);
14991499
sink.writeUint8List(mapBytes);
15001500
var allBytes = sink.takeBytes();
15011501

@@ -1506,7 +1506,7 @@ class AnalysisDriver {
15061506
_scheduler.eventsController.add(
15071507
events.AnalyzedLibrary(
15081508
library: library,
1509-
requirements: libraryRequirements,
1509+
requirements: requirements,
15101510
),
15111511
);
15121512
}
@@ -1784,7 +1784,7 @@ class AnalysisDriver {
17841784

17851785
var reader = SummaryDataReader(reqAndUnitBytes);
17861786
var requirements = performance.run('readRequirements', (_) {
1787-
return BundleRequirementsManifest.read(reader);
1787+
return RequirementsManifest.read(reader);
17881788
});
17891789

17901790
var failure = requirements.isSatisfied(
@@ -2099,7 +2099,7 @@ class AnalysisDriver {
20992099
if (reqAndUnitBytes != null) {
21002100
var reader = SummaryDataReader(reqAndUnitBytes);
21012101
var requirements = performance.run('readRequirements', (_) {
2102-
return BundleRequirementsManifest.read(reader);
2102+
return RequirementsManifest.read(reader);
21032103
});
21042104

21052105
var failure = requirements.isSatisfied(

pkg/analyzer/lib/src/dart/analysis/driver_event.dart

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ sealed class AnalysisDriverEvent {}
1515
/// The event after [library] analysis.
1616
final class AnalyzedLibrary extends AnalysisDriverEvent {
1717
final LibraryFileKind library;
18-
final BundleRequirementsManifest? requirements;
18+
final RequirementsManifest? requirements;
1919

2020
AnalyzedLibrary({
2121
required this.library,
@@ -75,17 +75,16 @@ final class GetErrorsFromBytes extends AnalysisDriverEvent {
7575
}
7676

7777
/// The event that libraries for [cycle] were linked, and accumulated the
78-
/// [requirementsManifest] to be checked if we try to reuse the summary bundle
79-
/// later.
78+
/// [requirements] to be checked if we try to reuse the summary bundle later.
8079
final class LinkLibraryCycle extends AnalysisDriverEvent {
8180
final LinkedElementFactory elementFactory;
8281
final LibraryCycle cycle;
83-
final BundleRequirementsManifest? requirementsManifest;
82+
final RequirementsManifest? requirements;
8483

8584
LinkLibraryCycle({
8685
required this.elementFactory,
8786
required this.cycle,
88-
required this.requirementsManifest,
87+
required this.requirements,
8988
});
9089
}
9190

pkg/analyzer/lib/src/dart/analysis/file_state.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2135,7 +2135,7 @@ abstract class LibraryOrAugmentationFileKind extends FileKind {
21352135

21362136
/// The resolution result for a library.
21372137
class LibraryResolutionResult {
2138-
final BundleRequirementsManifest requirements;
2138+
final RequirementsManifest requirements;
21392139

21402140
/// Approximately serialized map of file URIs to diagnostics.
21412141
/// See uses for precise details.

pkg/analyzer/lib/src/dart/analysis/library_context.dart

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -207,8 +207,8 @@ class LibraryContext {
207207
Uint8List linkedBytes;
208208
try {
209209
if (withFineDependencies) {
210-
var requirementsManifest = BundleRequirementsManifest();
211-
linkingBundleManifest = requirementsManifest;
210+
var requirements = RequirementsManifest();
211+
globalResultRequirements = requirements;
212212

213213
var linkResult = performance.run('link', (performance) {
214214
return link(
@@ -219,7 +219,7 @@ class LibraryContext {
219219
inputLibraryManifests: inputLibraryManifests,
220220
);
221221
});
222-
linkingBundleManifest = null;
222+
globalResultRequirements = null;
223223
linkedBytes = linkResult.resolutionBytes;
224224

225225
var newLibraryManifests = <Uri, LibraryManifest>{};
@@ -234,16 +234,16 @@ class LibraryContext {
234234
elementFactory.libraryManifests.addAll(newLibraryManifests);
235235
});
236236

237-
requirementsManifest.addExports(
237+
requirements.addExports(
238238
elementFactory: elementFactory,
239239
libraryUriSet: cycle.libraryUris,
240240
);
241-
requirementsManifest.removeReqForLibs(cycle.libraryUris);
241+
requirements.removeReqForLibs(cycle.libraryUris);
242242

243243
bundleEntry = LinkedBundleEntry(
244244
apiSignature: cycle.nonTransitiveApiSignature,
245245
libraryManifests: newLibraryManifests,
246-
requirements: requirementsManifest,
246+
requirements: requirements,
247247
linkedBytes: linkedBytes,
248248
);
249249
linkedBundleProvider.put(
@@ -255,7 +255,7 @@ class LibraryContext {
255255
LinkLibraryCycle(
256256
elementFactory: elementFactory,
257257
cycle: cycle,
258-
requirementsManifest: requirementsManifest,
258+
requirements: requirements,
259259
),
260260
);
261261
} else {
@@ -273,7 +273,7 @@ class LibraryContext {
273273
bundleEntry = LinkedBundleEntry(
274274
apiSignature: cycle.nonTransitiveApiSignature,
275275
libraryManifests: {},
276-
requirements: BundleRequirementsManifest(),
276+
requirements: RequirementsManifest(),
277277
linkedBytes: linkedBytes,
278278
);
279279
linkedBundleProvider.put(
@@ -285,7 +285,7 @@ class LibraryContext {
285285
LinkLibraryCycle(
286286
elementFactory: elementFactory,
287287
cycle: cycle,
288-
requirementsManifest: null,
288+
requirements: null,
289289
),
290290
);
291291
}
@@ -472,7 +472,7 @@ class LinkedBundleEntry {
472472
/// These requirements are to the libraries in dependencies.
473473
///
474474
/// If [withFineDependencies] is `false`, the requirements are empty.
475-
final BundleRequirementsManifest requirements;
475+
final RequirementsManifest requirements;
476476

477477
/// The serialized libraries, for [BundleReader].
478478
final Uint8List linkedBytes;
@@ -523,7 +523,7 @@ class LinkedBundleProvider {
523523
readKey: () => reader.readUri(),
524524
readValue: () => LibraryManifest.read(reader),
525525
);
526-
var requirements = BundleRequirementsManifest.read(reader);
526+
var requirements = RequirementsManifest.read(reader);
527527
var linkedBytes = reader.readUint8List();
528528

529529
var result = LinkedBundleEntry(

pkg/analyzer/lib/src/dart/element/inheritance_manager3.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -328,7 +328,7 @@ class InheritanceManager3 {
328328
int forMixinIndex = -1,
329329
bool forSuper = false,
330330
}) {
331-
linkingBundleManifest?.notifyInterfaceRequest(
331+
globalResultRequirements?.notifyInterfaceRequest(
332332
element: element.asElement2, nameObj: name);
333333
var interface = getInterface(element);
334334
if (forSuper) {

pkg/analyzer/lib/src/dart/element/scope.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -580,9 +580,9 @@ class PrefixScope implements Scope {
580580
);
581581
}
582582

583-
if (linkingBundleManifest case var linkingBundleManifest?) {
583+
if (globalResultRequirements case var resultRequirements?) {
584584
for (var importedLibrary in _importedLibraries) {
585-
linkingBundleManifest.notifyRequest(
585+
resultRequirements.notifyRequest(
586586
importedLibrary: importedLibrary,
587587
nameStr: id,
588588
);

0 commit comments

Comments
 (0)